Skip to content

Use full flag name in strip command for Darwin - #144829

Merged
bors merged 1 commit into
rust-lang:masterfrom
Kivooeo:strip-flag
Aug 4, 2025
Merged

Use full flag name in strip command for Darwin#144829
bors merged 1 commit into
rust-lang:masterfrom
Kivooeo:strip-flag

Conversation

@Kivooeo

@KivooeoKivooeo commented Aug 2, 2025

Copy link
Copy Markdown
Member

Darwin always uses rust-objcopy which supports long-form flags

Solaris unchanged due to not having support for --discard-all and only -x

fixes#135038

r? @WaffleLapkin (since bot will ping you anyway, feel free to reroll)

@rustbot

Copy link
Copy Markdown
Collaborator

WaffleLapkin is not on the review rotation at the moment.
They may take a while to respond.

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in compiler/rustc_codegen_ssa

cc @WaffleLapkin

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Aug 2, 2025
@Kivooeo
Kivooeoforce-pushed the strip-flag branch 2 times, most recently from 9854eb1 to bf409adCompareAugust 2, 2025 16:07
@KivooeoKivooeo changed the title Use full flag names in strip commandsUse full flag name in strip command for DarwinAug 2, 2025
@WaffleLapkin

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Aug 2, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit f87e829 has been approved by WaffleLapkin

It is now in the queue for this repository.

@borsbors 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 2, 2025
@WaffleLapkin

Copy link
Copy Markdown
Member

@bors rollup

Zalathar added a commit to Zalathar/rust that referenced this pull request Aug 3, 2025
Use full flag name in strip command for Darwin
Darwin always uses `rust-objcopy` which supports long-form flags
Solaris unchanged due to not having support for `--discard-all` and only `-x`
fixesrust-lang#135038
r? `@WaffleLapkin` (since bot will ping you anyway, feel free to reroll)
@ZalatharZalathar mentioned this pull request Aug 3, 2025
bors added a commit that referenced this pull request Aug 3, 2025
Rollup of 14 pull requests
Successful merges:
- #143857 (Port #[macro_export] to the new attribute parsing infrastructure)
- #143900 ([rustdoc] Correctly handle `should_panic` doctest attribute and fix `--no-run` test flag on the 2024 edition)
- #144070 (Implement `hash_map` macro )
- #144322 (Add lint against dangling pointers from local variables)
- #144443 (Make target pointer width in target json an integer)
- #144667 (`AlignmentEnum` should just be `repr(usize)` now)
- #144779 (Implement debugging output of the bootstrap Step graph into a DOT file)
- #144790 (Multiple bounds checking elision failures)
- #144794 (Port `#[coroutine]` to the new attribute system)
- #144808 (`Interner` arg to `EarlyBinder` does not affect auto traits)
- #144816 (Update E0562 to account for the new impl trait positions)
- #144822 (Return a struct with named fields from `hash_owner_nodes`)
- #144824 (Updated test links in compiler)
- #144829 (Use full flag name in strip command for Darwin)
r? `@ghost`
`@rustbot` modify labels: rollup
Zalathar added a commit to Zalathar/rust that referenced this pull request Aug 3, 2025
Use full flag name in strip command for Darwin
Darwin always uses `rust-objcopy` which supports long-form flags
Solaris unchanged due to not having support for `--discard-all` and only `-x`
fixesrust-lang#135038
r? ``@WaffleLapkin`` (since bot will ping you anyway, feel free to reroll)
@ZalatharZalathar mentioned this pull request Aug 3, 2025
bors added a commit that referenced this pull request Aug 3, 2025
Rollup of 13 pull requests
Successful merges:
- #143857 (Port #[macro_export] to the new attribute parsing infrastructure)
- #143900 ([rustdoc] Correctly handle `should_panic` doctest attribute and fix `--no-run` test flag on the 2024 edition)
- #144070 (Implement `hash_map` macro )
- #144322 (Add lint against dangling pointers from local variables)
- #144443 (Make target pointer width in target json an integer)
- #144667 (`AlignmentEnum` should just be `repr(usize)` now)
- #144790 (Multiple bounds checking elision failures)
- #144794 (Port `#[coroutine]` to the new attribute system)
- #144808 (`Interner` arg to `EarlyBinder` does not affect auto traits)
- #144816 (Update E0562 to account for the new impl trait positions)
- #144822 (Return a struct with named fields from `hash_owner_nodes`)
- #144824 (Updated test links in compiler)
- #144829 (Use full flag name in strip command for Darwin)
r? `@ghost`
`@rustbot` modify labels: rollup
Zalathar added a commit to Zalathar/rust that referenced this pull request Aug 3, 2025
Use full flag name in strip command for Darwin
Darwin always uses `rust-objcopy` which supports long-form flags
Solaris unchanged due to not having support for `--discard-all` and only `-x`
fixesrust-lang#135038
r? ```@WaffleLapkin``` (since bot will ping you anyway, feel free to reroll)
@ZalatharZalathar mentioned this pull request Aug 3, 2025
bors added a commit that referenced this pull request Aug 3, 2025
Rollup of 13 pull requests
Successful merges:
- #143857 (Port #[macro_export] to the new attribute parsing infrastructure)
- #144070 (Implement `hash_map` macro )
- #144322 (Add lint against dangling pointers from local variables)
- #144667 (`AlignmentEnum` should just be `repr(usize)` now)
- #144678 (Make no_mangle on foreign items explicit instead of implicit)
- #144790 (Multiple bounds checking elision failures)
- #144794 (Port `#[coroutine]` to the new attribute system)
- #144805 (compiletest: Preliminary cleanup of `ProcRes` printing/unwinding)
- #144808 (`Interner` arg to `EarlyBinder` does not affect auto traits)
- #144816 (Update E0562 to account for the new impl trait positions)
- #144822 (Return a struct with named fields from `hash_owner_nodes`)
- #144824 (Updated test links in compiler)
- #144829 (Use full flag name in strip command for Darwin)
r? `@ghost`
`@rustbot` modify labels: rollup
@Zalathar

Copy link
Copy Markdown
Member

Trying to diagnose failure in rollup #144846 (comment):

@bors try jobs=test-various

@rust-bors

rust-borsBot commented Aug 3, 2025

Copy link
Copy Markdown
Contributor

⌛ Trying commit f87e829 with merge 418d9bc

To cancel the try build, run the command @bors try cancel.

rust-borsBot added a commit that referenced this pull request Aug 3, 2025
Use full flag name in strip command for Darwin
try-job: test-various
Zalathar added a commit to Zalathar/rust that referenced this pull request Aug 3, 2025
Use full flag name in strip command for Darwin
Darwin always uses `rust-objcopy` which supports long-form flags
Solaris unchanged due to not having support for `--discard-all` and only `-x`
fixesrust-lang#135038
r? `@WaffleLapkin` (since bot will ping you anyway, feel free to reroll)
@ZalatharZalathar mentioned this pull request Aug 3, 2025
@rust-bors

rust-borsBot commented Aug 3, 2025

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 418d9bc (418d9bc759b01db3e16a4d8c2dfc9a0cf98b00fd, parent: da19b9d24c4ed900b91b45ec7f7795ec43613b1e)

bors added a commit that referenced this pull request Aug 3, 2025
Rollup of 13 pull requests
Successful merges:
- #143857 (Port #[macro_export] to the new attribute parsing infrastructure)
- #144070 (Implement `hash_map` macro )
- #144322 (Add lint against dangling pointers from local variables)
- #144667 (`AlignmentEnum` should just be `repr(usize)` now)
- #144706 (Do not give function allocations alignment in consteval and Miri.)
- #144790 (Multiple bounds checking elision failures)
- #144794 (Port `#[coroutine]` to the new attribute system)
- #144805 (compiletest: Preliminary cleanup of `ProcRes` printing/unwinding)
- #144808 (`Interner` arg to `EarlyBinder` does not affect auto traits)
- #144816 (Update E0562 to account for the new impl trait positions)
- #144822 (Return a struct with named fields from `hash_owner_nodes`)
- #144824 (Updated test links in compiler)
- #144829 (Use full flag name in strip command for Darwin)
r? `@ghost`
`@rustbot` modify labels: rollup
samueltardieu added a commit to samueltardieu/rust that referenced this pull request Aug 3, 2025
Use full flag name in strip command for Darwin
Darwin always uses `rust-objcopy` which supports long-form flags
Solaris unchanged due to not having support for `--discard-all` and only `-x`
fixesrust-lang#135038
r? ``@WaffleLapkin`` (since bot will ping you anyway, feel free to reroll)
bors added a commit that referenced this pull request Aug 3, 2025
Rollup of 11 pull requests
Successful merges:
- #142678 (Misc cleanups of `generic_arg_infer` related HIR logic)
- #144650 (Additional tce tests)
- #144738 (Remove the omit_gdb_pretty_printer_section attribute)
- #144790 (Multiple bounds checking elision failures)
- #144805 (compiletest: Preliminary cleanup of `ProcRes` printing/unwinding)
- #144808 (`Interner` arg to `EarlyBinder` does not affect auto traits)
- #144816 (Update E0562 to account for the new impl trait positions)
- #144822 (Return a struct with named fields from `hash_owner_nodes`)
- #144824 (Updated test links in compiler)
- #144829 (Use full flag name in strip command for Darwin)
- #144843 (Weekly `cargo update`)
Failed merges:
- #144794 (Port `#[coroutine]` to the new attribute system)
r? `@ghost`
`@rustbot` modify labels: rollup
samueltardieu added a commit to samueltardieu/rust that referenced this pull request Aug 3, 2025
Use full flag name in strip command for Darwin
Darwin always uses `rust-objcopy` which supports long-form flags
Solaris unchanged due to not having support for `--discard-all` and only `-x`
fixesrust-lang#135038
r? ```@WaffleLapkin``` (since bot will ping you anyway, feel free to reroll)
This was referenced Aug 3, 2025
bors added a commit that referenced this pull request Aug 3, 2025
Rollup of 12 pull requests
Successful merges:
- #142678 (Misc cleanups of `generic_arg_infer` related HIR logic)
- #144070 (Implement `hash_map` macro )
- #144738 (Remove the omit_gdb_pretty_printer_section attribute)
- #144790 (Multiple bounds checking elision failures)
- #144805 (compiletest: Preliminary cleanup of `ProcRes` printing/unwinding)
- #144808 (`Interner` arg to `EarlyBinder` does not affect auto traits)
- #144816 (Update E0562 to account for the new impl trait positions)
- #144822 (Return a struct with named fields from `hash_owner_nodes`)
- #144824 (Updated test links in compiler)
- #144829 (Use full flag name in strip command for Darwin)
- #144843 (Weekly `cargo update`)
- #144851 (Forbid tail calling intrinsics)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 621a832 into rust-lang:masterAug 4, 2025
11 checks passed
@rustbotrustbot added this to the 1.91.0 milestone Aug 4, 2025
rust-timer added a commit that referenced this pull request Aug 4, 2025
Rollup merge of #144829 - Kivooeo:strip-flag, r=WaffleLapkin
Use full flag name in strip command for Darwin
Darwin always uses `rust-objcopy` which supports long-form flags
Solaris unchanged due to not having support for `--discard-all` and only `-x`
fixes#135038
r? ````@WaffleLapkin```` (since bot will ping you anyway, feel free to reroll)
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 4, 2025
Rollup of 12 pull requests
Successful merges:
- rust-lang/rust#142678 (Misc cleanups of `generic_arg_infer` related HIR logic)
- rust-lang/rust#144070 (Implement `hash_map` macro )
- rust-lang/rust#144738 (Remove the omit_gdb_pretty_printer_section attribute)
- rust-lang/rust#144790 (Multiple bounds checking elision failures)
- rust-lang/rust#144805 (compiletest: Preliminary cleanup of `ProcRes` printing/unwinding)
- rust-lang/rust#144808 (`Interner` arg to `EarlyBinder` does not affect auto traits)
- rust-lang/rust#144816 (Update E0562 to account for the new impl trait positions)
- rust-lang/rust#144822 (Return a struct with named fields from `hash_owner_nodes`)
- rust-lang/rust#144824 (Updated test links in compiler)
- rust-lang/rust#144829 (Use full flag name in strip command for Darwin)
- rust-lang/rust#144843 (Weekly `cargo update`)
- rust-lang/rust#144851 (Forbid tail calling intrinsics)
r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cleanup: use full strip cli args in cg_ssa

5 participants

@Kivooeo@rustbot@WaffleLapkin@bors@Zalathar