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

[babel] don't override BABEL_CACHE_PATH #5372

Merged
merged 1 commit into from Nov 11, 2015

Conversation

Projects
None yet
2 participants
@spalger
Member

spalger commented Nov 11, 2015

#5244 set the BABEL_CACHE_PATH so that babel didn't try to write to the home directory. This prevents people from setting the cache path, which shouldn't be the case.

@epixa

This comment has been minimized.

Show comment
Hide comment
@epixa

epixa Nov 11, 2015

Member

LGTM

Member

epixa commented Nov 11, 2015

LGTM

@epixa epixa assigned spalger and unassigned epixa Nov 11, 2015

spalger added a commit that referenced this pull request Nov 11, 2015

Merge pull request #5372 from spalger/fix/overrideBabelCachePath
[babel] don't override BABEL_CACHE_PATH

@spalger spalger merged commit aaeaa16 into elastic:master Nov 11, 2015

3 checks passed

CLA Commit author has signed the CLA
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
default Merged build finished.
Details

@spalger spalger deleted the spalger:fix/overrideBabelCachePath branch Feb 25, 2016

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