Thread (1 message) 1 message, 1 author, 2017-03-30

Re: [PATCH 2/2] submodule.c: correctly handle nested submodules in is_submodule_modified

From: Junio C Hamano <hidden>
Date: 2017-03-30 18:28:58

Jonathan Nieder [off-list ref] writes:
Stefan Beller wrote:
quoted
This bug fix also affects the default output (non-short, non-porcelain)
of git-status, which is not tested here.
Do you have an example?  (In just the commit message would be fine, in
tests would be even better.)
quoted
Signed-off-by: Stefan Beller <redacted>
---
 Documentation/git-status.txt |  2 ++
 submodule.c                  | 21 +++++++++++++++++++--
 t/t3600-rm.sh                |  2 +-
 t/t7506-status-submodule.sh  |  4 ++--
 4 files changed, 24 insertions(+), 5 deletions(-)
Reviewed-by: Jonathan Nieder <redacted>

but I would be a lot more comfortable after looking at the change to
"git status" output.  (E.g. a test demonstrating it can happen in a
followup change if that's simpler.)

Thanks for your patient work on this.
Thank you both.  I re-read the whole thing and it feels to me that
the topic is now ready for 'next'.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help