Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot find elm-make from npm install #48

Open
rtfeldman opened this issue Sep 9, 2015 · 12 comments
Open

Cannot find elm-make from npm install #48

rtfeldman opened this issue Sep 9, 2015 · 12 comments

Comments

@rtfeldman
Copy link
Contributor

Just tried the beta and got:

[Errno 2] No such file or directory: 'elm-make'
[cmd: ['elm-make', '/Users/rtfeldman/code/elm-effects/src/Effects.elm', '--output=/dev/null', '--report=json', '--yes']]
[dir: /Users/rtfeldman/code/elm-effects]
[path: /usr/bin:/bin:/usr/sbin:/sbin]
[Finished]

This makes sense, considering it was only looking in /usr/bin:/bin:/usr/sbin:/sbin and given:

$ which elm-make
/usr/local/bin/elm-make

It would appear https://github.com/deadfoxygrandpa/Elm.tmLanguage/blob/master/Build%20Systems/Elm%20Make.sublime-build#L18-L21 is being ignored somehow?

@texastoland
Copy link
Contributor

Ah I forgot I replaced that with a elm_bin_dir setting inspired by the CoffeeScript package in https://github.com/dnalot/sublime-langauge-elm/commit/ca4c22074171e117d7a59f24c435cd77c76fe741.

The issue was Elm Make (or anything else depending on this hack) expanded PATH recursively i.e. 100 compiles would result in 100 /usr/local/bins prepended to PATH. It was mostly just annoying for debugging. Potential usability fixes include:

  1. Document or even prompt about the setting or another package called Fix Mac Path.
  2. Default setting to /usr/local/bin on OS X. Where does the standalone installer link?
  3. Find a tricky way to set PATH non-recursively.
  4. Stop being OCD and revert what we had because it "just worked."

Even though it works as designed it shouldn't degrade DX (dev exp). Thanks @rtfeldman CC @deadfoxygrandpa.

@deadfoxygrandpa
Copy link
Owner

Do you mean reverting back to my hacky way to fix the build command? If so, I'm ok with doing that even though it feels less elegant.

@texastoland
Copy link
Contributor

👍 I'll revisit next beta. @rtfeldman Workaround in the meantime is elm_bin_dir setting.

@rtfeldman
Copy link
Contributor Author

Gotcha, thanks! +1 to reverting for the next round...at least in my experience, that approach worked fine.

@rtfeldman
Copy link
Contributor Author

For what it's worth, I tried setting elm_bin_dir to /usr/local/bin and it errored out. I went back to the old Elm Build configuration to get things working again.

@texastoland
Copy link
Contributor

Going through a move but high on my list!!

@texastoland
Copy link
Contributor

@rtfeldman Just got finished with moving and new job stuff. Are you still interested in this? I can help finish the PR through but I've fully transitioned stacks (including editors) and think @deadfoxygrandpa has lost momentum too. Would you ask around for interested contributors?

@deadfoxygrandpa
Copy link
Owner

@rtfeldman Are you still experiencing this?

@rtfeldman
Copy link
Contributor Author

I still get this on the beta:

Running elm-make /Users/rtfeldman/code/NoRedInk/app/assets/javascripts/Quiz/Question.elm --output=/dev/null --report=json --yes
error: Error loading syntax file "Packages/sublime-language-elm/Syntaxes/Elm Compile Messages.hidden-tmLanguage": Unable to open Packages/sublime-language-elm/Syntaxes/Elm Compile Messages.hidden-tmLanguage

@deadfoxygrandpa
Copy link
Owner

@rtfeldman I'm creating a new way to distribute beta versions, by adding a custom repository to Package Control which overrides the normal Elm Language Support package to point at the beta branch instead. It should fix some of these beta-only issues.

Right now the public release is ahead of the previous beta release, but once I have a new beta version ready I'll let you know.

@rtfeldman
Copy link
Contributor Author

Awesome, please do! :D

On Fri, Dec 25, 2015, 1:00 AM Alex Neslusan [email protected]
wrote:

@rtfeldman https://github.com/rtfeldman I'm creating a new way to
distribute beta versions, by adding a custom repository to Package Control
which overrides the normal Elm Language Support package to point at the
beta branch instead. It should fix some of these beta-only issues.

Right now the public release is ahead of the previous beta release, but
once I have a new beta version ready I'll let you know.


Reply to this email directly or view it on GitHub
#48 (comment)
.

@DaveSanders
Copy link

I'm having a similar issue today. npm installed elm, and am getting the "[Errno 2] No such file or directory: 'elm-make'" error on a build.

I have updated my user preferences to include the path to the elm bins (elm-oracle and elm-format are working), but elm-make is not... Any tips?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants