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

CARGO: Don't explicitely set RUSTC env var #2078

Merged
merged 1 commit into from Nov 27, 2017

Conversation

Projects
None yet
1 participant
@matklad
Copy link
Member

matklad commented Nov 27, 2017

It helps tools like clippy, which use their own RUSTC. Historically,
this was implemented because not everyone was using rustup, and, thus,
not everybody had rustc in PATH, but hopefully this is not a problem
anymore.

closes #1941

@matklad

This comment has been minimized.

Copy link
Member Author

matklad commented Nov 27, 2017

bors r+

bors bot added a commit that referenced this pull request Nov 27, 2017

Merge #2078
2078: CARGO: Don't explicitely set RUSTC env var r=matklad a=matklad

It helps tools like clippy, which use their own RUSTC. Historically,
this was implemented because not everyone was using rustup, and, thus,
not everybody had `rustc` in PATH, but hopefully this is not a problem
anymore.

closes #1941
@bors

This comment has been minimized.

Copy link
Contributor

bors bot commented Nov 27, 2017

CARGO: Don't explicitely set RUSTC env var
It helps tools like clippy, which use their own RUSTC. Historically,
this was implemented because not everyone was using rustup, and, thus,
not everybody had `rustc` in PATH, but hopefully this is not a problem
anymore.

@matklad matklad force-pushed the no-path branch from 737b745 to 9414904 Nov 27, 2017

@matklad

This comment has been minimized.

Copy link
Member Author

matklad commented Nov 27, 2017

bors r+

bors bot added a commit that referenced this pull request Nov 27, 2017

Merge #2078
2078: CARGO: Don't explicitely set RUSTC env var r=matklad a=matklad

It helps tools like clippy, which use their own RUSTC. Historically,
this was implemented because not everyone was using rustup, and, thus,
not everybody had `rustc` in PATH, but hopefully this is not a problem
anymore.

closes #1941
@bors

This comment has been minimized.

Copy link
Contributor

bors bot commented Nov 27, 2017

@bors bors bot merged commit 9414904 into master Nov 27, 2017

2 checks passed

bors Build succeeded
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bors bors bot deleted the no-path branch Nov 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.