Skip to content

ENH: Enable specifying MacOSX deployment target#94

Merged
dzenanz merged 2 commits intomainfrom
macosxDeployment
Oct 17, 2024
Merged

ENH: Enable specifying MacOSX deployment target#94
dzenanz merged 2 commits intomainfrom
macosxDeployment

Conversation

@dzenanz
Copy link
Copy Markdown
Member

@dzenanz dzenanz commented Oct 15, 2024

std::filesystem requires MACOSX_DEPLOYMENT_TARGET=10.15 or newer.

std::filesystem requires MACOSX_DEPLOYMENT_TARGET=10.15 or newer.
@dzenanz dzenanz marked this pull request as ready for review October 15, 2024 21:06
@dzenanz dzenanz requested a review from thewtex October 15, 2024 21:06
Before this change, "Build ITK" step had full log,
while "Build and test" step had only superficial output, e.g.:

Configure project
   Each . represents 1024 bytes of output
    .. Size of output: 1K
Build project
   Each symbol represents 1024 bytes of output.
   '!' represents an error and '*' a warning.
    .........!! Size of output: 11K
Error(s) when building project
   1 Compiler errors
   1 Compiler warnings
@dzenanz
Copy link
Copy Markdown
Member Author

dzenanz commented Oct 16, 2024

@dzenanz dzenanz requested a review from jhlegarreta October 17, 2024 14:58
Copy link
Copy Markdown
Member

@jhlegarreta jhlegarreta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dzenanz
Copy link
Copy Markdown
Member Author

dzenanz commented Oct 17, 2024

Probably because arm64 variant was introduced with MacOS 11.

@dzenanz dzenanz merged commit 0ada46f into main Oct 17, 2024
@dzenanz dzenanz deleted the macosxDeployment branch October 17, 2024 16:35
@thewtex
Copy link
Copy Markdown
Member

thewtex commented Oct 17, 2024

@dzenanz we have to revert this. Distributable Python packages require an older version of the macOS deployment target.

@dzenanz
Copy link
Copy Markdown
Member Author

dzenanz commented Oct 17, 2024

This only allows changing it. The default stay the same.

@dzenanz
Copy link
Copy Markdown
Member Author

dzenanz commented Oct 17, 2024

So far, this was only changed in ITKNornir:
https://github.com/InsightSoftwareConsortium/ITKNornir/blob/v0.3.1/.github/workflows/build-test-package.yml#L12

Can you point me to an explanation for "Distributable Python packages require an older version of the macOS deployment target." I found this SO answer and this discussion.

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.

3 participants