compile hangs for a few minutes before firing proc.on('close') about gulp-elm HOT 8 OPEN

Thanks for the investigating and tracking down the cause of this issue, I'd love to be able to make gulp-elm much faster. The reason we're using cross-spawn is for Windows support (the built-in spawn can't find the elm-make.cmd file used on Windows). So we'll need to figure out if there's a way to fix the speed issue while also continuing Windows support.

I assume you're on macOS or Linux, right? I'll have to see if I can reproduce the issue on my Linux machine and find a solution that also keeps Windows support.

Thanks!

from gulp-elm.

@stephenhand, running gulp make-elm-app in the gulp-elm branch of that repository doesn't cause any delay for me on my Linux machine (it takes about 4 seconds). Let me know if there are different steps to reproduce.

from gulp-elm.

Thanks for looking into this.

I'm actually reproducing this issue on Windows 10 (64 bit) (sorry, should have mentioned that on the issue) and it seems to have no issues with the built-in spawn command.

I notice the plugin falls back to using the elm-make on the path if it can't find one in the directory it expects, and I do have elm-make on my path, so perhaps that's why it still works in my environment?

from gulp-elm.

FYI I'm running node v6.11.0

from gulp-elm.

I just ran through the build on my Windows 10 machine, using node v6.11.3, and didn't run into any delays. If you type where.exe elm-make into PowerShell, does it return anything different than C:\Program Files (x86)\Elm Platform\0.18\bin\elm-make.exe?

from gulp-elm.

If I can get this to reproduce, I'd like to try updating the cross-spawn dependency. So, in the meantime, you may want to get a local version of gulp-elm, update that dependency, and see if it makes any difference.

Also, if you could commit an npm or yarn lock file, that might help us see if we have any differences in how dependencies are being resolved.

Hope I can help!

from gulp-elm.

Sorry for taking a while to get back to you

My elm-make is in a slightly different location:

D:\Elm Platform\0.18\bin\elm-make.exe

And this is my package-lock.json file:
package-lock.zip

My resolved cross-spawn version is 3.0.1

I'll try updating the cross-spawn dep in a local version of gulp-elm like you suggested ASAP

from gulp-elm.

I updated a local gulp-elm to reference cross spawn v5.1.0 (latest version on npm) and it appeared to resolve the problem.

from gulp-elm.