Thanks for the patch. Unfortunately, it doesn't help in my case as it
invokes the is_submodule_modified() routine which you didn't modify.
Here's my call trace....
#0 is_submodule_modified (path=path@entry=0x17c2f08 "groc", ignore_untracked=0)
at submodule.c:939
#1 0x00000000004aa4dc in match_stat_with_submodule (
diffopt=diffopt@entry=0x7fffffffde18, ce=ce@entry=0x17c2eb0,
st=st@entry=0x7fffffffd840, ce_option=ce_option@entry=0,
dirty_submodule=dirty_submodule@entry=0x7fffffffd83c) at diff-lib.c:81
#2 0x00000000004ab4f5 in run_diff_files (revs=revs@entry=0x7fffffffd920,
option=option@entry=0) at diff-lib.c:217
#3 0x000000000054c0d4 in wt_status_collect_changes_worktree
(s=s@entry=0x7de280 <s>)
at wt-status.c:559
#4 0x000000000054ecf6 in wt_status_collect (s=s@entry=0x7de280 <s>)
at wt-status.c:678
#5 0x0000000000422171 in cmd_status (argc=<optimized out>,
argv=<optimized out>,
prefix=0x0) at builtin/commit.c:1390
#6 0x0000000000405abe in run_builtin (argv=<optimized out>,
argc=<optimized out>,
p=<optimized out>) at git.c:352
#7 handle_builtin (argc=1, argv=0x7fffffffe570) at git.c:551
#8 0x0000000000405dd8 in run_argv (argv=0x7fffffffe320, argcp=0x7fffffffe32c)
at git.c:606
#9 cmd_main (argc=1, argc@entry=2, argv=0x7fffffffe570,
argv@entry=0x7fffffffe568)
at git.c:678
#10 0x0000000000405060 in main (argc=2, argv=0x7fffffffe568) at common-main.c:40
I think the following fix is still needed to is_submodule_modified():
strbuf_addf(&buf, "%s/.git", path);
git_dir = read_gitfile(buf.buf);
if (!git_dir) {
git_dir = buf.buf;
==> if (!is_git_directory(git_dir)) {
==> die("Corrupted .git dir in submodule %s", path);
==> }
}
Thanks,
Uma
On Mon, Aug 29, 2016 at 11:23 PM, Jacob Keller <jacob.keller@xxxxxxxxx> wrote:
> On Mon, Aug 29, 2016 at 11:09 PM, Jacob Keller <jacob.keller@xxxxxxxxx> wrote:
>> On Mon, Aug 29, 2016 at 5:12 PM, Uma Srinivasan <usrinivasan@xxxxxxxxxxx> wrote:
>>> This is great! Thanks Jake. If you happen to have the patch ID it
>>> would be helpful.
>>>
>>> Uma
>>>
>>
>> http://public-inbox.org/git/1472236108.28343.5.camel@xxxxxxxxx/
>
>
> Actually correct patch is
> http://public-inbox.org/git/20160825233243.30700-6-jacob.e.keller@xxxxxxxxx/
>
> Thanks,
> Jake