On Tue, Oct 2, 2018 at 1:31 PM Jeff King <peff@xxxxxxxx> wrote:
> Actually, I guess we do not need to save $? at all, since we have only a
> single process to care about. So even simpler:
>
> spatch ... 2>$@+ 2>$@.log ||
> {
> cat $@.log
> exit 1
> }
> # if we get here, we were successful
> mv $@+ $@ ;# etc
>
> would work. That's missing all the Makefile=required backslashes and
> semicolons, of course. ;)
>
I opted to drop to just save the return, immediately after calling.
It's a bit less code change, and I think the result is as clear as the
above would be. This way we do drop the subshell, not that it matters
much in the end...
Thanks,
Jake
> -Peff
