AcademySoftwareFoundation / AcademySoftwareFoundation/OpenShadingLanguage

Bitcode serialization fails with unset Python_EXECUTABLE variable

Aperta
#1,503 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
C++
Stelle
2.3k
Fork
414
Merge medio
3g 1h
PR unite (30g)
10

Descrizione

### Problem

The custom command to serialize the bitcode for liboslexec fails if the `Python_EXECUTABLE` CMake variable is not set. This is due to a slight change in the invocation of the serialization script:

```diff
- COMMAND python "${CMAKE_SOURCE_DIR}/src/build-scripts/serialize-bc.py" ${llvm_bc} ${llvm_bc_cpp} ${prefix}
+ COMMAND ${Python_EXECUTABLE} "${CMAKE_SOURCE_DIR}/src/build-scripts/serialize-bc.py" ${llvm_bc} ${llvm_bc_cpp} ${prefix}
```

This can be worked around by setting `Python_EXECUTABLE` or setting `USE_PYTHON=ON` at configure time.

Perhaps for the sake of compatibility it would be better to either set a default value so that the build will work as before, or to issue a warning or error if the variable is not set?

### Versions

* OSL branch/version: main (033dae37)
* OS: Ubuntu 20.04
* C++ compiler: clang 12.0.0
* LLVM version: 12.0.0
* OIIO version: N/A

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Individua il comando personalizzato che invoca src/build-scripts/serialize-bc.py ed esamina come viene impostato Python_EXECUTABLE durante la configurazione. Riproduci la compilazione della serializzazione del bitcode con Python_EXECUTABLE non impostato, quindi fai in modo che il comportamento funzioni oppure fallisca con una diagnostica chiara e verifica il percorso di compilazione interessato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cmake, python
Ambito
build-system
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.