Skip to content

Fix test failure on macOS with link_section#2246

Merged
traviscross merged 1 commit intorust-lang:masterfrom
ehuss:fix-macos-link_section-test
May 5, 2026
Merged

Fix test failure on macOS with link_section#2246
traviscross merged 1 commit intorust-lang:masterfrom
ehuss:fix-macos-link_section-test

Conversation

@ehuss
Copy link
Copy Markdown
Contributor

@ehuss ehuss commented May 1, 2026

rust-lang/rust#155065 recently added validation for link_section on macOS. I'm working around it here by just limiting this test to linux.

Alternatives:

  • Show different syntaxes for different targets (but I would prefer to keep the example as plain as possible).
  • Mark it as ignore (want to avoid this if at all possible).

Fixes #2245

rust-lang/rust#155065 recently added validation
for link_section on macOS. I'm working around it here by just limiting
this test to linux.

Alternatives:
- Show different syntaxes for different targets (but I would prefer to
  keep the example as plain as possible).
- Mark it as `ignore` (want to avoid this if at all possible).

Fixes rust-lang#2245
@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label May 1, 2026
@ehuss ehuss mentioned this pull request May 1, 2026
@tlyu
Copy link
Copy Markdown
Contributor

tlyu commented May 2, 2026

Thanks! This fix seems to work for me.

@traviscross traviscross added this pull request to the merge queue May 5, 2026
Merged via the queue into rust-lang:master with commit 6dd9c81 May 5, 2026
6 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label May 5, 2026
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.

tests fail on macOS

4 participants