Skip to content

mk: Fix rpath on cross compile builds #13462

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

Merged
merged 1 commit into from
Apr 13, 2014

Conversation

alexcrichton
Copy link
Member

After removing absolute rpaths, cross compile builds (notably the nightly
builders) broke. This is because the RPATH was pointing at an empty directory
because only the rustc binary is copied over, not all of the target libraries.
This modifies the cross compile logic to fixup the rpath of the stage0
cross-compiled rustc to point to where it came from.

After removing absolute rpaths, cross compile builds (notably the nightly
builders) broke. This is because the RPATH was pointing at an empty directory
because only the rustc binary is copied over, not all of the target libraries.
This modifies the cross compile logic to fixup the rpath of the stage0
cross-compiled rustc to point to where it came from.
bors added a commit that referenced this pull request Apr 13, 2014
After removing absolute rpaths, cross compile builds (notably the nightly
builders) broke. This is because the RPATH was pointing at an empty directory
because only the rustc binary is copied over, not all of the target libraries.
This modifies the cross compile logic to fixup the rpath of the stage0
cross-compiled rustc to point to where it came from.
@bors bors closed this Apr 13, 2014
@bors bors merged commit c60d9ad into rust-lang:master Apr 13, 2014
@alexcrichton alexcrichton deleted the fix-cross-rpath branch April 13, 2014 17:19
flip1995 pushed a commit to flip1995/rust that referenced this pull request Oct 3, 2024
`zombie_processes`: consider `wait()` calls in nested bodies

Fixes rust-lang#13459

Small oversight. We weren't considering uses of the local in closures.

changelog: none
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.

3 participants