Skip to content

Make.inc: fix xcrun invocation#61040

Merged
fingolfin merged 1 commit intomasterfrom
mh/xcrun
Feb 15, 2026
Merged

Make.inc: fix xcrun invocation#61040
fingolfin merged 1 commit intomasterfrom
mh/xcrun

Conversation

@fingolfin
Copy link
Member

@fingolfin fingolfin commented Feb 15, 2026

The officially supported argument is --sdk, not -sdk, though it seems the later also works in practice. But with BinaryBuilder, we use a 'fake' xcrun script which only supports the officially documented argument name.

I've also submitted a PR to let BinaryBuilderBase.jl accept this; but I think it's wise for us to stick to the officially documented options.

The officially supported argument is --sdk, not -sdk, though it
seems the later also works in practice. But with BinaryBuilder, we
use a 'fake' xcrun script which only supports the officially
documented argument name.
@fingolfin fingolfin requested a review from giordano February 15, 2026 14:31
@fingolfin fingolfin added building Build system, or building Julia or its dependencies system:mac Affects only macOS backport 1.11 Change should be backported to release-1.11 backport 1.12 Change should be backported to release-1.12 backport 1.13 Change should be backported to release-1.13 labels Feb 15, 2026
@fingolfin fingolfin merged commit 4820747 into master Feb 15, 2026
8 of 11 checks passed
@fingolfin fingolfin deleted the mh/xcrun branch February 15, 2026 16:02
KristofferC pushed a commit that referenced this pull request Feb 25, 2026
The officially supported argument is --sdk, not -sdk, though it seems
the later also works in practice. But with BinaryBuilder, we use a
'fake' xcrun script which only supports the officially documented
argument name.

I've also submitted a [PR to let BinaryBuilderBase.jl accept
this](JuliaPackaging/BinaryBuilderBase.jl#464);
but I think it's wise for us to stick to the officially documented
options.

(cherry picked from commit 4820747)
@KristofferC KristofferC mentioned this pull request Feb 25, 2026
36 tasks
KristofferC pushed a commit that referenced this pull request Mar 3, 2026
The officially supported argument is --sdk, not -sdk, though it seems
the later also works in practice. But with BinaryBuilder, we use a
'fake' xcrun script which only supports the officially documented
argument name.

I've also submitted a [PR to let BinaryBuilderBase.jl accept
this](JuliaPackaging/BinaryBuilderBase.jl#464);
but I think it's wise for us to stick to the officially documented
options.

(cherry picked from commit 4820747)
@KristofferC KristofferC mentioned this pull request Mar 3, 2026
56 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 1.11 Change should be backported to release-1.11 backport 1.12 Change should be backported to release-1.12 backport 1.13 Change should be backported to release-1.13 building Build system, or building Julia or its dependencies system:mac Affects only macOS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants