Re: Possible regression in master? (submodules without a "master" branch)
- From
Jens Lehmann <jens.lehmann@web.de>
- Date
- Mar 27, 2014, 17:31 UTC
- Message-ID
- <5334606F.5010109@web.de>
- In-Reply-To
- <xmqqob0ref3v.fsf@gitster.dls.corp.google.com>
Am 27.03.2014 18:16, schrieb Junio C Hamano:
Show 16 quoted lines
> Johan Herland <johan@herland.net> writes: > >> I just found a failure to checkout a project with submodules where >> there is no explicit submodule branch configuration, and the >> submodules happen to not have a "master" branch: >> >> git clone git://gitorious.org/qt/qt5.git qt5 >> cd qt5 >> git submodule init qtbase >> git submodule update >> >> In current master, the last command fails with the following output: > > ... and with a bug-free system, what does it do instead? Just clone > 'qtbase' and make a detached-head checkout at the commit recorded in > the superproject's tree, or something else?
After reverting 23d25e48f5ead73 on current master it clones 'qtbase' nicely with a detached HEAD.
Show 19 quoted lines
>> Cloning into 'qtbase'... >> remote: Counting objects: 267400, done. >> remote: Compressing objects: 100% (61070/61070), done. >> remote: Total 267400 (delta 210431), reused 258876 (delta 202642) >> Receiving objects: 100% (267400/267400), 136.23 MiB | 6.73 MiB/s, done. >> Resolving deltas: 100% (210431/210431), done. >> Checking connectivity... done. >> error: pathspec 'origin/master' did not match any file(s) known to git. >> Unable to setup cloned submodule 'qtbase' >> >> Bisection points to 23d25e48f5ead73c9ce233986f90791abec9f1e8 (W. >> Trevor King: submodule: explicit local branch creation in >> module_clone). Looking at the patch, it seems to introduce an implicit >> assumption on the submodule origin having a "master" branch. Is this >> an intended change in behaviour? > > If an existing set-up that was working in a sensible way is broken > by a change that assumes something that should not be assumed, then > that is a serious regression, I would have to say.
Yes, especially as it promised to not change this use case.