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

process.env alteration? #94

Closed
steak3 opened this issue Jan 11, 2016 · 4 comments
Closed

process.env alteration? #94

steak3 opened this issue Jan 11, 2016 · 4 comments
Labels

Comments

@steak3
Copy link

steak3 commented Jan 11, 2016

Hello,

I'm investigating why I have this error when I try to perform a child_process.exec with linters activated, here:
AtomLinter/linter-jshint#191

I made a diff of process.env in Atom, before and after linters activation, and this one seems very suspicious:
image

So the OS is deleted somewhere. I just fast-explored the commits and maybe this one is related because there is some env management modification:
42b68ca

What do you think? Is that normal?

Thanks!

@steak3
Copy link
Author

steak3 commented Jan 11, 2016

Also, in other conditions (after clicking on a file in the tree) I have this strange vartype difference (but it might not be related and I don't know where it come from):
image

@steelbrain
Copy link
Owner

Good find, fixing now

@steelbrain
Copy link
Owner

Fixed by #95

@Arcanemagus
Copy link
Collaborator

A fix is published in v4.3.2 of this package, you should be able to run apm install from inside the linter-jshint folder to install the new dependency version. You might need to check Atom's developer tools to make sure it is loading atom-linter from the linter-jshint folder, if it is loading it from another folder you would need to update the dependency there as well.

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

No branches or pull requests

3 participants