Skip to content

Conversation

lgeiger
Copy link
Collaborator

@lgeiger lgeiger commented Oct 13, 2017

For some reason Atom only detects the correct env if started from the command line on Mac.

Strangely process.env.PATH is correct when evaluating it in the dev tools. Unfortunately in this file process.env has the wrong value. atom.updateProcessEnv() doesn't help either. 😕

This looks like a bug in Atom.

@kbrose Could you test if this fixes the CPU issues on Linux?

@kbrose
Copy link

kbrose commented Oct 13, 2017

Works great! +1 from me!

@lgeiger
Copy link
Collaborator Author

lgeiger commented Oct 13, 2017

Wow that's great! Thanks for the super fast testing 👍 🎉

@lgeiger lgeiger merged commit 5434cc0 into master Oct 13, 2017
@lgeiger lgeiger deleted the only-use-shell-env-on-darwin branch October 13, 2017 03:53
@lgeiger
Copy link
Collaborator Author

lgeiger commented Oct 13, 2017

@kbrose Would you be interested in helping maintaining this package? I'd be happy to add you to the repo.

@kbrose
Copy link

kbrose commented Oct 13, 2017

Sure, I'd be happy to help out

@lgeiger
Copy link
Collaborator Author

lgeiger commented Oct 13, 2017

That's awesome! 🎉

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants