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

Add a temporary env var to enable hashes in filenames #3022

Merged
merged 1 commit into from
Aug 19, 2016

Conversation

alexcrichton
Copy link
Member

For rustbuild we need the hashes to exist for all deps, even if they're path
deps, because we care about the actual file names. For example we don't want to
install /usr/lib/libstd.so!

This adds a "secret" environment variable, __CARGO_DEFAULT_LIB_METADATA which
re-enables the old behavior of just putting hashes in filenames.

Closes #3005

For rustbuild we need the hashes to exist for all deps, even if they're path
deps, because we care about the actual file names. For example we don't want to
install /usr/lib/libstd.so!

This adds a "secret" environment variable, `__CARGO_DEFAULT_LIB_METADATA` which
re-enables the old behavior of just putting hashes in filenames.

Closes rust-lang#3005
@rust-highfive
Copy link

@alexcrichton: no appropriate reviewer found, use r? to override

@alexcrichton
Copy link
Member Author

r? @brson

@brson
Copy link
Contributor

brson commented Aug 19, 2016

@bors r+

@bors
Copy link
Contributor

bors commented Aug 19, 2016

📌 Commit 7e40439 has been approved by brson

@bors
Copy link
Contributor

bors commented Aug 19, 2016

⌛ Testing commit 7e40439 with merge 109cb7c...

bors added a commit that referenced this pull request Aug 19, 2016
Add a temporary env var to enable hashes in filenames

For rustbuild we need the hashes to exist for all deps, even if they're path
deps, because we care about the actual file names. For example we don't want to
install /usr/lib/libstd.so!

This adds a "secret" environment variable, `__CARGO_DEFAULT_LIB_METADATA` which
re-enables the old behavior of just putting hashes in filenames.

Closes #3005
@bors
Copy link
Contributor

bors commented Aug 19, 2016

☀️ Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: brson
Pushing 109cb7c to master...

@bors bors merged commit 7e40439 into rust-lang:master Aug 19, 2016
alexcrichton added a commit to rust-lang-deprecated/rust-packaging that referenced this pull request Aug 22, 2016
alexcrichton added a commit to rust-lang-deprecated/rust-packaging that referenced this pull request Aug 22, 2016
@alexcrichton alexcrichton deleted the add-more-metadata branch September 6, 2016 20:54
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.

4 participants