search

cctbx / cctbx_project

Computational Crystallography Toolbox
https://cci.lbl.gov/docs/cctbx
Other
214 stars 111 forks source link

FYI: macOS ld: warning: -undefined dynamic_lookup may not work with chained fixups #815

Open rwgk opened 1 year ago

rwgk commented 1 year ago

To hopefully save you some time digging this up:

That's deprecated in Xcode 14.

Solution: https://github.com/python/cpython/issues/97524

bkpoon commented 1 year ago

Thanks for the heads up. Does the warning only show up when -mmacosx-version-min=12.0? I'm not seeing the warning in our Xcode 14 builds on Monterey, but we set the minimum to 10.9 for Intel and 11.0 for Apple Silicon.

bkpoon commented 1 year ago

Also, is this just for Xcode or also generally for clang? The conda packages (and new Phenix releases) are built with clang from the conda-forge channel. Developers are free to use Xcode, though.

rwgk commented 1 year ago

Thanks for the heads up. Does the warning only show up when -mmacosx-version-min=12.0? I'm not seeing the warning in our Xcode 14 builds on Monterey, but we set the minimum to 10.9 for Intel and 11.0 for Apple Silicon.

Sorry I don't know. There is some more background here: https://github.com/pybind/pybind11/pull/4301

My recommendation: just wait until you actually see the warning, then use the links here to see what others did.

bkpoon commented 1 year ago

Yeah, that's what I was thinking too. I'll update to Ventura when the .1 release is available and see if anything changes as newer versions of Xcode come out.

I also pinged the conda-forge folks to see if they know anything more.