-
-
Notifications
You must be signed in to change notification settings - Fork 280
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document how to find NumPy with CMake on cross-compiled builds #2321
Conversation
✅ Deploy Preview for conda-forge-previews ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
Co-authored-by: jaimergp <jaimergp@users.noreply.github.com>
Sorry @jaimergp, I just realized I never asked again the review after I address your comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but I don't have the background to assess whether this is the right solution or not. Judging by the discussion in #2260 it looks like the best effort so far, so I'm inclined to merge regardless. I'll leave it open for a couple days before doing so in case someone from @conda-forge/core wants to chime in.
That is correct. My feeling is that probably there should be something that may be fixed in cmake and/or conda-forge cross-compilation setuptools to avoid the need to manually set this include directories, but I was not able to identify this. |
Let’s merge and iterate |
PR Checklist:
docs/
orcommunity/
, you have added it to the sidebar in the corresponding_sidebar.json
fileProposal to fix #2260 .