Skip to content

Conversation

pietroalbini
Copy link
Member

Merged and approved:

r? @ghost

petrochenkov and others added 4 commits September 1, 2018 10:21
The former code used `target.contains("64")` to detect Elf64 targets,
but this is inaccurate in a few cases:

- `s390x-unknown-linux-gnu` is 64-bit
- `sparcv9-sun-solaris` is 64-bit
- `x86_64-unknown-linux-gnux32` is 32-bit

Instead the `std` build script can use `CARGO_CFG_TARGET_POINTER_WIDTH`
to reliably detect 64-bit targets for libbacktrace.

Also update to backtrace-sys 0.1.24 for rust-lang/backtrace-rs#122.
If we detect a local rebuild (e.g. bootstrap compiler is the same version as target compiler), we set stage to 1.
When trying to build e.g. UnstableBook, we use Mode::ToolBootstrap and stage is 1.
Just allow Mode::ToolBootstrap and stagge != 0 if we are in a local_rebuild

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>
@pietroalbini
Copy link
Member Author

@bors r+ p=10

@bors
Copy link
Collaborator

bors commented Sep 1, 2018

📌 Commit 1aa4bbb has been approved by pietroalbini

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Sep 1, 2018
@bors
Copy link
Collaborator

bors commented Sep 1, 2018

⌛ Testing commit 1aa4bbb with merge b4ec8d4...

bors added a commit that referenced this pull request Sep 1, 2018
[beta] Rollup backports

Merged and approved:

* #53653: Address two regressions
* #53377: std: Use target_pointer_width for BACKTRACE_ELF_SIZE
* #52969: rustbuild: fix local_rebuild

r? @ghost
@bors
Copy link
Collaborator

bors commented Sep 1, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: pietroalbini
Pushing b4ec8d4 to beta...

@bors bors merged commit 1aa4bbb into rust-lang:beta Sep 1, 2018
@pietroalbini pietroalbini deleted the beta-backports branch September 1, 2018 11:42
@cuviper
Copy link
Member

cuviper commented Sep 4, 2018

@pietroalbini - The Cargo.lock change in #53377 was important too, as it updates for a similar fix in backtrace-sys 0.1.24. Do you have any more backports planned where you can pull this in? If not, I can go ahead and submit this to beta separately.

@pietroalbini
Copy link
Member Author

@cuviper please submit another PR, I don't know if/when the other backports will be approved.

bors added a commit that referenced this pull request Sep 4, 2018
[beta] update backtrace-sys to 0.1.24

This is the other part of #53377 that was missed in the #53880 backport.

r? @pietroalbini
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants