revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-08 07:52:39 +01:00
Code Issues Releases Wiki Activity

Simplify diff exit code handling.

Browse source 
Summary:
This fixes T9970 in an alternate manner, with the same effect: the
binary_safe_diff.sh script returns 0 if the diff succeeds, 1 in all
other cases.

Test Plan:
Tested locally with a fixed binary_safe_diff.sh, resulting in this
correct review:

https://reviews.freebsd.org/D4542

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: eadler, Korvin, stevenh

Maniphest Tasks: T9970

Differential Revision: https://secure.phabricator.com/D14759
This commit is contained in:
Dimitry Andric 2015-12-13 02:35:07 -08:00 committed by epriestley
parent 97056a3b85
commit 7d25fcdbdb
1 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
scripts/repository/binary_safe_diff.sh
View file

@ -1,8 +1,3 @@
#!/bin/sh
diff "$@"
RES="$?"
if [ "$RES" = "2" ]; then
exit 1
fi
exit "$RES"
diff "$@" || exit 1