Skip to content

Fix a compile error on the ESP-IDF target - #162049

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
ivmarkov:fix-espidf-thread-stack-size
Aug 31, 2026
Merged

Fix a compile error on the ESP-IDF target#162049
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
ivmarkov:fix-espidf-thread-stack-size

Conversation

@ivmarkov

@ivmarkov ivmarkov commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

PR #160848 accidentally broke the Tier-3 ESP-IDF target by leaving out one reference to attr.as_ptr() which should've been changed to attr.get().

This PR does the absolute minimum to restore correct behavior for the ESP-IDF target (and Nuttx).

CC
@joboet
@nia-e

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Aug 31, 2026
@rustbot

rustbot commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

r? @ChrisDenton

rustbot has assigned @ChrisDenton.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ChrisDenton, libs
  • @ChrisDenton, libs expanded to 13 candidates
  • Random selection from 6 candidates

@joboet

joboet commented Aug 31, 2026

Copy link
Copy Markdown
Member

Whoops, sorry about that!
@bors r+ rollup
r? joboet

@rust-bors

rust-bors Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 6508e60 has been approved by joboet

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 31, 2026
Zalathar added a commit to Zalathar/rust that referenced this pull request Aug 31, 2026
…ize, r=joboet

Fix a compile error on the ESP-IDF target

PR rust-lang#160848 accidentally broke the Tier-3 ESP-IDF target by leaving out one reference to `attr.as_ptr()` which should've been changed to `attr.get()`.

This PR does the absolute minimum to restore correct behavior for the ESP-IDF target (and Nuttx).

CC
@joboet
@nia-e
rust-bors Bot pushed a commit that referenced this pull request Aug 31, 2026
Rollup of 5 pull requests

Successful merges:

 - #161556 (document some rustc_dump attributes)
 - #161353 (Add test for parallel compiler reproducible build)
 - #161776 (Add a couple polonius tests and move universal region liveness generation)
 - #162002 (Output `tests/debuginfo` raw JSON data on failure in CI)
 - #162049 (Fix a compile error on the ESP-IDF target)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 31, 2026
…ize, r=joboet

Fix a compile error on the ESP-IDF target

PR rust-lang#160848 accidentally broke the Tier-3 ESP-IDF target by leaving out one reference to `attr.as_ptr()` which should've been changed to `attr.get()`.

This PR does the absolute minimum to restore correct behavior for the ESP-IDF target (and Nuttx).

CC
@joboet
@nia-e
rust-bors Bot pushed a commit that referenced this pull request Aug 31, 2026
…uwer

Rollup of 6 pull requests

Successful merges:

 - #162058 (Port 1.98.1 release notes into main)
 - #161556 (document some rustc_dump attributes)
 - #161776 (Add a couple polonius tests and move universal region liveness generation)
 - #162002 (Output `tests/debuginfo` raw JSON data on failure in CI)
 - #162049 (Fix a compile error on the ESP-IDF target)
 - #162054 (Refine InferCtxtInner docs)
rust-bors Bot pushed a commit that referenced this pull request Aug 31, 2026
…uwer

Rollup of 6 pull requests

Successful merges:

 - #162058 (Port 1.98.1 release notes into main)
 - #161556 (document some rustc_dump attributes)
 - #161776 (Add a couple polonius tests and move universal region liveness generation)
 - #162002 (Output `tests/debuginfo` raw JSON data on failure in CI)
 - #162049 (Fix a compile error on the ESP-IDF target)
 - #162054 (Refine InferCtxtInner docs)
rust-bors Bot pushed a commit that referenced this pull request Aug 31, 2026
…uwer

Rollup of 6 pull requests

Successful merges:

 - #162058 (Port 1.98.1 release notes into main)
 - #161556 (document some rustc_dump attributes)
 - #161776 (Add a couple polonius tests and move universal region liveness generation)
 - #162002 (Output `tests/debuginfo` raw JSON data on failure in CI)
 - #162049 (Fix a compile error on the ESP-IDF target)
 - #162054 (Refine InferCtxtInner docs)
@rust-bors
rust-bors Bot merged commit 9cec563 into rust-lang:main Aug 31, 2026
13 checks passed
@rustbot rustbot added this to the 1.100.0 milestone Aug 31, 2026
rust-bors Bot pushed a commit that referenced this pull request Aug 31, 2026
Rollup merge of #162049 - ivmarkov:fix-espidf-thread-stack-size, r=joboet

Fix a compile error on the ESP-IDF target

PR #160848 accidentally broke the Tier-3 ESP-IDF target by leaving out one reference to `attr.as_ptr()` which should've been changed to `attr.get()`.

This PR does the absolute minimum to restore correct behavior for the ESP-IDF target (and Nuttx).

CC
@joboet
@nia-e
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.

4 participants