Fix broken FetchContent: point utilities/parallelzone at master - #64
Merged
Conversation
NWChemEx/Utilities and NWChemEx/ParallelZone have both completed their
CI/dependency migration and merged build_overhaul into master -- and
this org has delete_branch_on_merge enabled, so both build_overhaul
branches were deleted the moment those PRs merged.
cmake/dependencies/{utilities,parallelzone}.cmake still pinned
GIT_TAG build_overhaul (intentionally held there until every ecosystem
repo finished migrating, per the plan), which means FetchContent now
fails outright for anyone configuring a repo that depends on either:
"Failed to checkout tag: 'build_overhaul'" -- confirmed breaking
TensorWrapper and PluginPlay's PR builds right now.
The original plan to batch this flip until the last repo merges didn't
account for branches disappearing incrementally as each repo merges.
Revised approach: flip each dependency's GIT_TAG to master as soon as
that specific repo's own migration PR merges, rather than waiting.
Only utilities and parallelzone have merged so far, so only those two
move here -- the remaining entries (chemist, simde, chemcache,
integrals, nux, nwchemex, pluginplay, tensorwrapper) still correctly
point at build_overhaul, since master doesn't have their migrated
content yet.
|
🚀 [bumpr] Bumped! |
This was referenced Aug 5, 2026
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
1 participant
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
Urgent -- currently breaking CI on every repo that transitively depends on Utilities or ParallelZone (confirmed on NWChemEx/TensorWrapper#252 and NWChemEx/PluginPlay#388:
CMake Error ... Failed to checkout tag: 'build_overhaul').NWChemEx/UtilitiesandNWChemEx/ParallelZonehave both completed their CI/dependency migration and merged intomaster. This org hasdelete_branch_on_mergeenabled, so bothbuild_overhaulbranches were deleted the instant those PRs merged.cmake/dependencies/{utilities,parallelzone}.cmakestill pinnedGIT_TAG build_overhaul, per the original plan to hold that pin until every ecosystem repo finished migrating -- that plan didn't account for branches disappearing incrementally as each repo merges, one at a time, well before the last one does.masternow, since their content is actually there. The remaining entries (chemist, simde, chemcache, integrals, nux, nwchemex, pluginplay, tensorwrapper) correctly stay onbuild_overhaul--masterdoesn't have their migrated content yet, so flipping them now would fetch stale content instead.Going forward, each repo's
cmake/dependencies/<repo>.cmakeentry will flip tomasterin the same wave that repo's own migration PR merges, rather than batching all of them into one cleanup at the end.Test plan
test_cmake_build/test_pip_buildjobs once this merges, confirm the FetchContent failure is gone