-
Notifications
You must be signed in to change notification settings - Fork 4
Description
While reviewing .gitmodules, I found some inconsistencies that can break or confuse git submodule workflows:
- Invalid stanza for
pyBurstAnalysisGUI:
[submodule "src/pyBurstAnalysisGUI/"]
branch = master
This entry has no path or url, so it’s not a valid submodule definition. See .gitmodules:124.
-
Stale/wrapper entries and missing paths:
.gitmodulesdeclares submodules for wrapper or missing directories, e.g.externals/gsl,externals/gsl_module/gsl,externals/cppunit,externals/clhep,resources/clhep/clhep,src/evtUtils. This can causegit submodule update --init --recursiveto try to init submodules that aren’t real gitlinks or don’t exist. See.gitmodules:115,.gitmodules:126,.gitmodules:129,.gitmodules:141,.gitmodules:144,.gitmodules:150. -
git://remotes:
Some entries usegit://(e.g., GSL). These are frequently blocked on locked-down networks; https remotes are more reliable.
Suggested fix:
- Remove or repair the invalid
pyBurstAnalysisGUIstanza. - Remove stale/wrapper entries or convert them to proper nested submodule entries only where needed.
- Replace
git://remotes withhttps://equivalents. - Ensure
.gitmodulesreflects the actual submodule layout present in the repo.