Repository navigation
Verify green after ecosystem rollout - #74
Merged
Merged
Conversation
SCF#73's test_cmake_build previously failed on all four legs (Linux+macOS, both compilers) with "link interface of nwx::simde contains nwx::chemist but the target was not found" -- not the config-dependency bug NWXCMake#87/#88 already fixed, but a self- collision: the integration-testing step's `pip install nwchemex` transitively floor-pins nwchemex-scf>=0.0.57, pulling in SCF's own stale, previously-published wheel (0.0.60, predating the rollout). That stale wheel bundles the old broken two-line simdeConfig.cmake, and since it installs into the same site-packages/lib/cmake/simde/ path as the freshly-resolved nwchemex-simde 0.0.81, it clobbered the correct config. #73 merged anyway: merge.yaml doesn't run test_cmake_build, so the new release (nwchemex-scf 0.0.61, published 2026-09-15T15:18 UTC) was built clean from git master, unaffected by the collision, and its own bundled dependency configs are now correct. This empty commit re-runs test_cmake_build to confirm that installing the new self-consistent nwchemex-scf no longer clobbers anything.
This was referenced Sep 15, 2026
Contributor
|
🚀 [bumpr] Bumped! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
test_cmake_buildpreviously failed on all four legs withlink interface of nwx::simde contains nwx::chemist but the target was not found-- a different bug from the NWXCMake#87/#88 config-dependencyissue already fixed this rollout.
pip install nwchemextransitively pulls in
nwchemex-scf(self), and the previouslypublished SCF wheel (0.0.60, predating this rollout) bundles the old
broken two-line
simdeConfig.cmake. Since it installs into the samesite-packages/lib/cmake/simde/path as the freshly-resolvednwchemex-simde 0.0.81, it clobbered the correct config at pip-installtime -- before CMake ever ran.
merge.yamldoesn't runtest_cmake_build; the resulting release (nwchemex-scf 0.0.61) wasbuilt clean from git master and has correct bundled configs.
test_cmake_buildand confirm theself-collision no longer reproduces now that the self-referential
wheel is internally consistent.
Test plan
test_cmake_buildgreen on all four legs (Linux+macOS x clang-18+gcc-14)🤖 Generated with Claude Code