Skip to content

Don't ship duplicate sets of libraries #39875

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

Closed
alexcrichton opened this issue Feb 16, 2017 · 2 comments
Closed

Don't ship duplicate sets of libraries #39875

alexcrichton opened this issue Feb 16, 2017 · 2 comments
Labels
C-feature-request Category: A feature request, i.e: not implemented / a PR. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Comments

@alexcrichton
Copy link
Member

Today when you install Rust by default you get duplicate sets of libraries at $sysroot/lib and $sysroot/lib/rustlib/$host/lib, for example:

$ find . -name libproc_macro-* | xargs shasum
2453aab91f0cd5e77b9f514d5df15be7b4ec965e  ./lib/libproc_macro-2641322a715c6559.dylib
2453aab91f0cd5e77b9f514d5df15be7b4ec965e  ./lib/rustlib/x86_64-apple-darwin/lib/libproc_macro-2641322a715c6559.dylib

We should avoid duplicating these libraries! Unfortunately I'm not sure of the best method of doing so.

@brson
Copy link
Contributor

brson commented Feb 16, 2017

Moving to an on disk scheme that doesn't have duplicate paths to the bins seems fairly challenging, but I can easily imagine installers being smartened to remove the duplication as best they can.

@Mark-Simulacrum Mark-Simulacrum added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) C-feature-request Category: A feature request, i.e: not implemented / a PR. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. and removed A-build labels Jul 27, 2017
@Mark-Simulacrum Mark-Simulacrum removed the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label Sep 15, 2017
@Mark-Simulacrum
Copy link
Member

closing in favor of #42645

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature-request Category: A feature request, i.e: not implemented / a PR. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

3 participants