Ok, got it now. To reproduce the issue:
- Run git submodule update --recursive to make sure their SHA is changed. Then `git add /path/to/changed submodule` or just `git add .` - Modify any file from the parent repository - Neither of `git status`, `git diff` and `git diff-files --name-only` will show changes to a submodule, only changes to that file which was changed in parent repo. - Make a git commit. It will not list changes to submodule as wll. - `git show HEAD` will show changes to both file from and parent repository (which is expected) and will also show changes to the submodule hash (which is unexpected i'd say). On Fri, Nov 22, 2013 at 11:01 PM, Sergey Sharybin <sergey....@gmail.com> wrote: > Ah, didn't notice you're the author of that pull-request Ramkumar :) > > So guess issue with arc can be considered solved now. But i'm still > collecting more details about how to manage to commit change addons > hash without arc command even (it happens to Campbell Barton really > often). > > Will report back when we'll know something. > > On Fri, Nov 22, 2013 at 10:35 PM, Ramkumar Ramachandra > <artag...@gmail.com> wrote: >> Sergey Sharybin wrote: >>> Ramkumar, not actually sure what you mean? >>> >>> For me `git diff --name-only HEAD --` ignores changes to submodules >>> hash changes. >> >> `git diff --name-only HEAD --` compares the worktree to HEAD (listing >> both staged and unstaged changes); we want `git diff --name-only --` >> to compare the worktree to the index (listing only unstaged changes), >> as Peff notes. >> >>> Also apparently it became a known TODO for phabricator >>> developers [1]. >> >> That was me :) >> >>> So, after all is it expected behavior of ls-files or not and if not >>> shall i report it as a separate thread? :) >> >> Actually, I doubt it's worth fixing ls-files. Your problem should be >> fixed when this is merged (hopefully in a few hours): >> >> https://github.com/facebook/arcanist/pull/121 >> >> Cheers. > > > > -- > With best regards, Sergey Sharybin -- With best regards, Sergey Sharybin -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
