Open jgoosens opened 1 week ago
Thanks for reporting!
The submodules implementation here is very problematic as it's non-standard, trying to be 'smarter' while ending up working correctly only in a small set of cases.
Any improvements are welcome if they are accompanied by a test, even though the whole implementation is probably inherently broken beyond repair. Maybe I am also too dismissive towards my work of the past.
This issue expands on #1058
My current setup is the following: a superrepo, whose default branch is
main
, and 4 submodules, 2 of which have default branchmain
(just like the superrepo), and 2 of which havemaster
as their default branch.My .gitmodules file looks like this:
When I clone without initializing the submodules, and subsequently run
I get the equivalent behaviour of
git submodule update --init --recursive
, but it still prints theFailed to checkout tracking branch refs/heads/master
warning. So far, so good.Now, in theory, running
should behave equivalently to
git submodule update --init --recursive --remote
, but it ends up in a state like described in #1058.As I commented on the issue above, I could get around this issue by "correctly" initializing my .gitmodules file first by running
For issue #1058, this is still a valid comment (hence my posting it there), as it's unlikely that the default branch is
develop
and it's likely that the superrepo branch differs from the submodule's.Note, however, that in my use-case the superrepo's and the "incorrectly configured" submodules' branches match (
main
, which is very common), yet it still spits out the warning above at best and leaves you in an inconsistent state at worst.Proposed enhancement: If the submodule config does not provide a branch explicitly, the ref to check out should be the default branch of the submodule remote, or, if that is unknown information, the default branch of the superrepo. Thus, there's a priority of refs to try out: