Hey Luke, On Wed, Jan 4, 2017 at 2:41 PM, Luke Diamand <[email protected]> wrote: > On 3 January 2017 at 19:57, Pranit Bauva <[email protected]> wrote: >> The exit code of the upstream in a pipe is ignored thus we should avoid >> using it. By writing out the output of the git command to a file, we can >> test the exit codes of both the commands. > > Do we also need to fix t9814-git-p4-rename.sh ?
I don't think so. As Johannes[1] and Stefan[2] pointed out, we should avoid upstream pipes for git. p4 can be treated as an "external command" just like grep/sed. [1]: http://public-inbox.org/git/[email protected]/ [2]: http://public-inbox.org/git/CAGZ79kZRFLzD7wcAnFvke9vBxxTAgE7=Ud7F_O95EfkWqz=l...@mail.gmail.com/
