Bitcode serialization fails with unset Python_EXECUTABLE variable about openshadinglanguage HOT 4 OPEN

Ah apologies, I thought python was already being loaded in since it worked in my setup and in CI.

I'm guessing we just need an appropriately placed find_package(Python REQUIRED)?

Alternatively -- is there a more lightweight way of serializing that bytecode into a cpp?

from openshadinglanguage.

@tgrant-nv were you building with USE_PYTHON=OFF?

Python is needed for the python bindings to the oslquery library, so there is a find_package(Python) already happening at some point. And also python must be available in order to run the testsuite (which are all driving by python scripts).

from openshadinglanguage.

@fpsunflower, no problem at all. Yes, adding a find_package does work as you would expect. Perhaps the find_python macro in pythonutils.cmake can be made aware of the Python requirement when the tests or bitcode generation are enabled.

My previous minimal configuration was setting USE_PYTHON=OFF, though of course I have Python (2.7 and 3.8) installed. The testsuite runs fine without enabling Python in the build, probably because runtest.py is executable and has the proper shebang.

from openshadinglanguage.

Makes sense. For windows, it seems that using Python_EXECUTABLE is preferable, at least in my setup the one in the path that run when you just use python is the wrong one, while cmake finds the right one via the pybind dependency.

I have yet to get the testuite fully working on windows - but I'm wondering if we should use the same strategy there ... (since I don't believe the #! will be picked up).

from openshadinglanguage.