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

Bump backtrace to 0.3.74~ish #129642

Merged
merged 2 commits into from
Aug 31, 2024
Merged

Conversation

workingjubilee
Copy link
Member

@workingjubilee workingjubilee commented Aug 27, 2024

Commit: rust-lang/backtrace-rs@230570f

This should help with backtraces on Android, QNX NTO 7.0, and Windows.
It addresses a case of backtrace incurring undefined behavior on Android.

r? @ghost

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 27, 2024
It should be 0.3.74~ish.

This should help with backtraces on Android, QNX NTO 7.0, and Windows.
@workingjubilee workingjubilee force-pushed the bump-backtrace-fc37b22 branch from 1e375a0 to 0763a3a Compare August 27, 2024 03:16
@workingjubilee
Copy link
Member Author

Specifically this brings in rust-lang/backtrace-rs@6a2e7a1

cc @saethlin

@workingjubilee
Copy link
Member Author

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Aug 27, 2024

📌 Commit 0763a3a has been approved by workingjubilee

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 27, 2024
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 27, 2024
…b22, r=workingjubilee

Bump backtrace to rust-lang/backtrace@fc37b22

It should be 0.3.74~ish.

This should help with backtraces on Android, QNX NTO 7.0, and Windows.
@workingjubilee workingjubilee changed the title Bump backtrace to rust-lang/backtrace@fc37b22 Bump backtrace to rust-lang/backtrace-rs@fc37b22 Aug 27, 2024
@workingjubilee
Copy link
Member Author

Meh, doesn't much matter.

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 27, 2024
…kingjubilee

Rollup of 9 pull requests

Successful merges:

 - rust-lang#126985 (Implement `-Z embed-source` (DWARFv5 source code embedding extension))
 - rust-lang#127922 (Add unsafe to extern blocks in style guide)
 - rust-lang#128731 (simd_shuffle intrinsic: allow argument to be passed as vector)
 - rust-lang#128935 (More work on `zstd` compression)
 - rust-lang#128942 (miri weak memory emulation: put previous value into initial store buffer)
 - rust-lang#129418 (rustc: Simplify getting sysroot library directory)
 - rust-lang#129490 (Add Trusty OS as tier 3 target)
 - rust-lang#129559 (float types: document NaN bit pattern guarantees)
 - rust-lang#129642 (Bump backtrace to rust-lang/backtrace@fc37b22)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors
Copy link
Contributor

bors commented Aug 27, 2024

⌛ Testing commit 0763a3a with merge 306c605...

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 27, 2024
…2, r=workingjubilee

Bump backtrace to rust-lang/backtrace-rs@fc37b22

It should be 0.3.74~ish.

This should help with backtraces on Android, QNX NTO 7.0, and Windows.
@tgross35
Copy link
Contributor

I think this caused the error in #129643 (comment), needs some tests blessed. Only a few minutes away from that point so we can let it go to be sure.

@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Aug 27, 2024

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 27, 2024
@workingjubilee
Copy link
Member Author

Can't bless tests for a single architecture. Either have to ignore it or fix the problem. What's going on that this code is causing an unsafe precondition to be violated?

@workingjubilee
Copy link
Member Author

cc @maurer @mgeisler @chriswailes It feels like either our test suite uses an incorrect emulator or some target-specific code is unsound.

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 27, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
@rustbot rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Aug 27, 2024
@workingjubilee
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 27, 2024
…2, r=<try>

Bump backtrace to rust-lang/backtrace-rs@fc37b22

It should be 0.3.74~ish.

This should help with backtraces on Android, QNX NTO 7.0, and Windows.

try-job: arm-android
@bors
Copy link
Contributor

bors commented Aug 27, 2024

⌛ Trying commit 8919cb4 with merge 64c470f...

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@workingjubilee
Copy link
Member Author

er... wait, I don't trust that.
@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 31, 2024
@workingjubilee
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Aug 31, 2024

📌 Commit 518b41c has been approved by workingjubilee

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Aug 31, 2024

🌲 The tree is currently closed for pull requests below priority 100. This pull request will be tested once the tree is reopened.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 31, 2024
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 31, 2024
…b22, r=workingjubilee

Bump backtrace to 0.3.74~ish

Commit: rust-lang/backtrace-rs@230570f

This should help with backtraces on Android, QNX NTO 7.0, and Windows.
It addresses a case of backtrace incurring undefined behavior on Android.
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 31, 2024
…kingjubilee

Rollup of 3 pull requests

Successful merges:

 - rust-lang#129642 (Bump backtrace to 0.3.74~ish)
 - rust-lang#129733 (Subtree update of `rust-analyzer`)
 - rust-lang#129785 (Miri subtree update)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 31, 2024
…kingjubilee

Rollup of 3 pull requests

Successful merges:

 - rust-lang#129642 (Bump backtrace to 0.3.74~ish)
 - rust-lang#129733 (Subtree update of `rust-analyzer`)
 - rust-lang#129785 (Miri subtree update)

r? `@ghost`
`@rustbot` modify labels: rollup
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 31, 2024
…b22, r=workingjubilee

Bump backtrace to 0.3.74~ish

Commit: rust-lang/backtrace-rs@230570f

This should help with backtraces on Android, QNX NTO 7.0, and Windows.
It addresses a case of backtrace incurring undefined behavior on Android.
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 31, 2024
…iaskrgr

Rollup of 15 pull requests

Successful merges:

 - rust-lang#120221 (Don't make statement nonterminals match pattern nonterminals)
 - rust-lang#126183 (Separate core search logic with search ui)
 - rust-lang#129123 (rustdoc-json: Add test for `Self` type)
 - rust-lang#129366 (linker: Synchronize native library search in rustc and linker)
 - rust-lang#129527 (Don't use `TyKind` in a lint)
 - rust-lang#129534 (Deny `wasm_c_abi` lint to nudge the last 25%)
 - rust-lang#129640 (Re-enable android tests/benches in alloc/core)
 - rust-lang#129642 (Bump backtrace to 0.3.74~ish)
 - rust-lang#129675 (allow BufReader::peek to be called on unsized types)
 - rust-lang#129723 (Simplify some extern providers)
 - rust-lang#129724 (Remove `Option<!>` return types.)
 - rust-lang#129725 (Stop using `ty::GenericPredicates` for non-predicates_of queries)
 - rust-lang#129731 (Allow running `./x.py test compiler`)
 - rust-lang#129751 (interpret/visitor: make memory order iteration slightly more efficient)
 - rust-lang#129754 (wasi: Fix sleeping for `Duration::MAX`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit fbf74df into rust-lang:master Aug 31, 2024
6 of 7 checks passed
@rustbot rustbot added this to the 1.82.0 milestone Aug 31, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Aug 31, 2024
Rollup merge of rust-lang#129642 - workingjubilee:bump-backtrace-fc37b22, r=workingjubilee

Bump backtrace to 0.3.74~ish

Commit: rust-lang/backtrace-rs@230570f

This should help with backtraces on Android, QNX NTO 7.0, and Windows.
It addresses a case of backtrace incurring undefined behavior on Android.
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 1, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 1, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
@workingjubilee workingjubilee deleted the bump-backtrace-fc37b22 branch September 3, 2024 13:22
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 4, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 4, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 20, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 8, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 29, 2024
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 14, 2025
Add some track_caller info to precondition panics

r? `@ghost`

Thought of this while looking at rust-lang#129642 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants