threads / discuss / 29721

Submodule status inside nested submodule fails

Subject: Submodule status inside nested submodule fails

## tl;dr

5 messages between Feb 24, 2012 and May 10, 2012.

replies: 4people: 3as markdown or json

Charles Brossollet· Feb 24, 2012, 09:53 UTC · lore
Hello there,

Using MSysGit 1.7.9 on Win7 (64 bit), I have a repo with the following structure:

main/
  src/
  ext/
    submodule/
       modules/module1
       modules/module2
submodule is... a submodule, having itself submodules.

When I query submodule status --recursive in main/, no problem. But when I query submodule status in ext/submodule, I get error "You need to run this command from the toplevel of the working tree.", which is a wrong error message.

Searching for message in git source I found the test triggering the message to be an empty result for "git rev-parse --show-cdup". Running the command in ext/submodule strangely returns the *absolute* path of ext/submodule, which should not happen because this command returns path of current dir relative to the containing working tree.

Is there something wrong with my repo? Or have I found a bug? Any help appreciated

Charles
Charles Brossollet· Feb 24, 2012, 13:41 UTC · re: Charles Brossollet · lore

Re: Submodule status inside nested submodule fails

Charles Brossollet <chbrosso <at> lltech.fr> writes:
 
Show 5 quoted lines
> Searching for message in git source I found the test triggering the message to
> be an empty result for "git rev-parse --show-cdup". Running the command in
> ext/submodule strangely returns the *absolute* path of ext/submodule, which
> should not happen because this command returns path of current dir relative to
> the containing working tree.

Things are OK if I run the commands from Windows' CMD, but it fails when run in MinGW shell, so I believe the problem is related to MsysGit, discussion follows up on their list.

rollinsdr@gmail.com· May 8, 2012, 18:11 UTC · re: Charles Brossollet · lore

Re: Submodule status inside nested submodule fails

I'm having the same issue. Did it ever get resolved, or is there a hack to fix it, or is there some link I can track it on?

Thanks in advance for the help!
Charles Brossollet wrote
Show 32 quoted lines
> 
> Charles Brossollet <chbrosso <at> lltech.fr> writes:
> Using MSysGit 1.7.9 on Win7 (64 bit), I have a repo with the following 
> structure: 
> 
> main/ 
>   src/ 
>   ext/ 
>     submodule/ 
>        modules/module1 
>        modules/module2 
> 
> submodule is... a submodule, having itself submodules. 
> 
> When I query submodule status --recursive in main/, no problem. 
> But when I query submodule status in  ext/submodule, I get error "You need
> to 
> run this command from the toplevel of the working tree.", which is a wrong 
> error message. 
> 
> Searching for message in git source I found the test triggering the
> message to 
> be an empty result for "git rev-parse --show-cdup". Running the command in 
> ext/submodule strangely returns the *absolute* path of ext/submodule,
> which 
> should not happen because this command returns path of current dir
> relative to 
> the containing working tree. 
> 
> Is there something wrong with my repo? Or have I found a bug? Any help 
> appreciated 
> 
Charles Brossollet wrote
Show 25 quoted lines
> 
> Charles Brossollet <chbrosso <at> lltech.fr> writes:
>  
>> Searching for message in git source I found the test triggering the
>> message to
>> be an empty result for "git rev-parse --show-cdup". Running the command
>> in
>> ext/submodule strangely returns the *absolute* path of ext/submodule,
>> which
>> should not happen because this command returns path of current dir
>> relative to
>> the containing working tree.
> 
> Things are OK if I run the commands from Windows' CMD, but it fails when
> run in
> MinGW shell, so I believe the problem is related to MsysGit, discussion
> follows
> up on their list.
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@.kernel
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

-- View this message in context: http://git.661346.n2.nabble.com/Submodule-status-inside-nested-submodule-fails-tp7314413p7540130.html Sent from the git mailing list archive at Nabble.com.

Heiko Voigt· May 9, 2012, 21:34 UTC · re: rollinsdr@gmail.com · lore

Re: Submodule status inside nested submodule fails

Hi,
On Tue, May 08, 2012 at 11:11:15AM -0700, rollinsdr@gmail.com wrote:
Show 21 quoted lines
> I'm having the same issue. Did it ever get resolved, or is there a hack to
> fix it, or is there some link I can track it on?
> 
> 
> Charles Brossollet wrote
> > 
> > Charles Brossollet <chbrosso <at> lltech.fr> writes:
> > Using MSysGit 1.7.9 on Win7 (64 bit), I have a repo with the following 
> > structure: 
> > 
> > main/ 
> >   src/ 
> >   ext/ 
> >     submodule/ 
> >        modules/module1 
> >        modules/module2 
> > 
> > submodule is... a submodule, having itself submodules. 
> > 
> > When I query submodule status --recursive in main/, no problem. 
> > But when I query submodule status in  ext/submodule, I get error "You need

Without having looked at the code itself this smells like an issue with the newly introduced gitlink files and git rev-parse --show-cdup not taking this into account.

I will have a look at this issue.
Cheers Heiko
rollinsdr@gmail.com· May 10, 2012, 01:40 UTC · re: Heiko Voigt · lore

Re: Submodule status inside nested submodule fails

Thanks Heiko,
Heiko Voigt-3 wrote
Show 40 quoted lines
> 
> Hi,
> 
> On Tue, May 08, 2012 at 11:11:15AM -0700, rollinsdr@ wrote:
>> I'm having the same issue. Did it ever get resolved, or is there a hack
>> to
>> fix it, or is there some link I can track it on?
>> 
>> 
>> Charles Brossollet wrote
>> > 
>> > Charles Brossollet <chbrosso <at> lltech.fr> writes:
>> > Using MSysGit 1.7.9 on Win7 (64 bit), I have a repo with the following 
>> > structure: 
>> > 
>> > main/ 
>> >   src/ 
>> >   ext/ 
>> >     submodule/ 
>> >        modules/module1 
>> >        modules/module2 
>> > 
>> > submodule is... a submodule, having itself submodules. 
>> > 
>> > When I query submodule status --recursive in main/, no problem. 
>> > But when I query submodule status in  ext/submodule, I get error "You
>> need
> 
> Without having looked at the code itself this smells like an issue with
> the newly introduced gitlink files and git rev-parse --show-cdup not
> taking this into account.
> 
> I will have a look at this issue.
> 
> Cheers Heiko
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@.kernel
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

Actually the issue went away once I upgraded from 1.7.9 to 1.7.10, and re-cloned my repo. I guess it was fixed in 1.7.10, but some stuff in the config files had to be rewritten during the new clone to fix the issue b/c just switching to 1.7.10 and trying to use the existing repo still had the same issue.

Thanks for all the work you guys do! DAVE

-- View this message in context: http://git.661346.n2.nabble.com/Submodule-status-inside-nested-submodule-fails-tp7314413p7545109.html Sent from the git mailing list archive at Nabble.com.

← back to recent threads