Yup, this was privately reported and I just squashed a fix in right now ;-) Thanks. "cd t && make test-lint" would have caught it. On Tue, Sep 22, 2015 at 12:11 PM, Michael Blume <blume.mike@xxxxxxxxx> wrote: > I'm seeing test failures > > non-executable tests: t9825-git-p4-handle-utf16-without-bom.sh > > ls -l shows that all the other tests are executable but t9825 isn't. > > On Tue, Sep 22, 2015 at 9:02 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >> Lars Schneider <larsxschneider@xxxxxxxxx> writes: >> >>> This works. >> >> OK, and thanks; as I don't do perforce, the squash was without any >> testing. >> >>> Do we need the “-e” option? >> >> In syntactic sense, no, but our codebase tends to prefer to have >> one, because it is easier to spot which ones are the instructions if >> you consistently have "-e" even when you give only one. >> -- >> To unsubscribe from this list: send the line "unsubscribe git" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html