Skip to content

Revert "Fast rebuilds of C/C++ code when needed. (#106)"#108

Merged
StephenOman merged 1 commit intoNetHack-LE:mainfrom
jbcoe:jbcoe/revert-fast-c++-build
Mar 21, 2026
Merged

Revert "Fast rebuilds of C/C++ code when needed. (#106)"#108
StephenOman merged 1 commit intoNetHack-LE:mainfrom
jbcoe:jbcoe/revert-fast-c++-build

Conversation

@jbcoe
Copy link
Collaborator

@jbcoe jbcoe commented Mar 20, 2026

This reverts commit 0b96483.

Fast rebuilds with uv and scikit-build and use of Python tooling within a CMake build are incompatible. The isolated environment used for the build means that the Python executable cached in the build directory is not valid when the build directory is used for a rebuild.

https://scikit-build-core.readthedocs.io/en/stable/configuration/#editable-installs suggests disabling build isolation which significantly complicates building with uv.

This reverts commit 0b96483.

Fast rebuilds with uv and scikit-build and use of Python tooling
within a CMake build are incompatible. The isolated environment
used for the build means that the Python executable cached in the
build directory is not valid when the build directory is used for
a rebuild.
@StephenOman StephenOman merged commit ce7283a into NetHack-LE:main Mar 21, 2026
19 checks passed
@StephenOman StephenOman added the bug Something isn't working label Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants