On Wed, Mar 1, 2017 at 12:34 AM, Michael Haggerty <mhagger@xxxxxxxxxxxx> wrote: > On 02/22/2017 03:04 PM, Nguyễn Thái Ngọc Duy wrote: >> Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> >> --- >> t/t1406-submodule-ref-store.sh (new +x) | 95 +++++++++++++++++++++++++++++++++ >> 1 file changed, 95 insertions(+) >> create mode 100755 t/t1406-submodule-ref-store.sh >> >> diff --git a/t/t1406-submodule-ref-store.sh b/t/t1406-submodule-ref-store.sh >> new file mode 100755 >> index 000000000..3b30ba62f >> --- /dev/null >> +++ b/t/t1406-submodule-ref-store.sh >> [...] > > I haven't actually read this far in the patch series, but I noticed that > a test in this file fails: > > > t1406-submodule-ref-store.sh (Wstat: 256 Tests: 15 > Failed: 1) > Failed test: 10 > Non-zero exit status: 1 > > I didn't have time to look into it more; let me know if you can't > reproduce it. Fantastic. No I couldn't reproduce it, even --valgrind did not complain. First step maybe just pushing your branch somewhere so I can try out if you're applying the patches via mail (maybe there's some changes in the base that affect this). .Otherwise /t1406-* -v -i might be enough clue for me to dig in, I hope. -- Duy