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:
.gitmodules declares 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 cause git submodule update --init --recursive to 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 use git:// (e.g., GSL). These are frequently blocked on locked-down networks; https remotes are more reliable.
Suggested fix:
- Remove or repair the invalid
pyBurstAnalysisGUI stanza.
- Remove stale/wrapper entries or convert them to proper nested submodule entries only where needed.
- Replace
git:// remotes with https:// equivalents.
- Ensure
.gitmodules reflects the actual submodule layout present in the repo.
While reviewing
.gitmodules, I found some inconsistencies that can break or confusegit submoduleworkflows:pyBurstAnalysisGUI:[submodule "src/pyBurstAnalysisGUI/"]
branch = master
This entry has no
pathorurl, 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 use
git://(e.g., GSL). These are frequently blocked on locked-down networks; https remotes are more reliable.Suggested fix:
pyBurstAnalysisGUIstanza.git://remotes withhttps://equivalents..gitmodulesreflects the actual submodule layout present in the repo.