Skip to content

Conversation

@mgorny
Copy link
Contributor

@mgorny mgorny commented Dec 4, 2025

PR Checklist:

  • note any issues closed by this PR with closing keywords
  • if you are adding a new page under docs/ or community/, you have added it to the sidebar in the corresponding _sidebar.json file
  • put any other relevant information below

See conda-forge/conda-smithy#2430.

@netlify
Copy link

netlify bot commented Dec 4, 2025

Deploy Preview for conda-forge-previews ready!

Name Link
🔨 Latest commit 7897a81
🔍 Latest deploy log https://app.netlify.com/projects/conda-forge-previews/deploys/6932f8b43d7a6b00081b5b54
😎 Deploy Preview https://deploy-preview-2684--conda-forge-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 72
Accessibility: 96
Best Practices: 100
SEO: 89
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.


Starting with conda-smithy 3.53.4, the build scripts will no longer use the system macOS SDK directory by default. The CI systems will use a dedicated `/opt/conda-sdks` directory, while users performing local builds will need to choose a writable directory, and provide the path to it via `OSX_SDK_DIR` environment variable.

This change may result in some build systems, particularly CMake, storing paths to this temporary build directory in installed metadata. Feedstocks will need to substitute the stored paths with path-agnostic solutions (for example, see [substitutions in openpmd-api-feedstock](https://github.com/conda-forge/openpmd-api-feedstock/blob/15f9b3648f087d3e06331d6ec9ddff0710300593/recipe/build.sh#L100-L107)) or the correct sysroot paths (for example, see [substitutions in cartographer-feedstock](https://github.com/conda-forge/cartographer-feedstock/blame/1812f8c13bccbad20daf6ba079f7722cace93a15/recipe/conda_build_config.yaml#L19-L23).
Copy link
Member

Choose a reason for hiding this comment

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

Based on the discussion in conda-forge/conda-smithy#2430, this should only be necessary for people compiling against our artefacts outside conda-forge, not in our CI. I'd like to reflect that here, otherwise it sounds like a massive deterioration of the feedstock maintenance situation (which is why I had written this).

Also, please wrap your lines at some finite length (something <130 characters due to the way GH's interface works)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My understanding is that it would also affect a transition period when some feedstocks are rendered with this change, and others still haven't been and therefore may embed wrong paths. And either way, I think fixing paths is the technically correct thing to do, even if it wasn't strictly necessary when the OSX SDK directory paths happened to match.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As for wrapping, I've followed 2025-10-15-conda-forge-discourse-read-only.md but perhaps that just happened to have the wrapping length beyond the paragraph length :-). I'll reflow.

Signed-off-by: Michał Górny <[email protected]>
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.

2 participants