Hi Junio, On Fri, Feb 11, 2022 at 1:23 AM Junio C Hamano <gitster@xxxxxxxxx> wrote: > FWIW, I have the posted patch plus the following SQUASH??? fix-up I'm not so sure what does "SQUASH???" mean especially the three question marks, i.e. is it just an incidental text or a commit message convention? Is it for the convenience of grepping through the "git log" outputs (cause I found the commit 50d631c71c right away by grepping through the "git log" output)? > parked in the 'seen' branch. As the script is quiescent right now, > I do not mind merging it down, now we spent more time on it ;-) > > t/t0001-init.sh | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/t/t0001-init.sh b/t/t0001-init.sh > index c72a28d3a5..d479303efa 100755 > --- a/t/t0001-init.sh > +++ b/t/t0001-init.sh > @@ -6,7 +6,8 @@ TEST_PASSES_SANITIZE_LEAK=true > . ./test-lib.sh > > check_config () { > - if test_path_is_dir "$1" && test_path_is_file "$1/config" && test_path_is_dir "$1/refs" > + if test_path_is_dir "$1" && > + test_path_is_file "$1/config" && test_path_is_dir "$1/refs" > then > : happy > else Yeah, I think wrapping it around is a good idea :-) > -- > 2.35.1-102-g2b9c120970 > -- Thanks & Regards, Shaoxuan