Page MenuHome

macOS/CMake: install python based on WITH_PYTHON_INSTALL
Changes PlannedPublic

Authored by Ankit Meel (ankitm) on Feb 17 2022, 1:20 PM.

Details

Summary

seems to be a typo

Diff Detail

Event Timeline

Ankit Meel (ankitm) requested review of this revision.Feb 17 2022, 1:20 PM
Ankit Meel (ankitm) created this revision.

As far as I can tell WITH_PYTHON_INSTALL=OFF on macOS is going to create a non-working build, unless WITH_PYTHON_MODULE or WITH_PYTHON_FRAMEWORK are ON.

So not sure what the purpose of this change is by itself. Better handling of linking to a system Python on macOS may be useful, but I'd rather see that done in one diff.

Ankit Meel (ankitm) planned changes to this revision.Feb 18 2022, 2:31 PM