revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-21 22:32:41 +01:00
Code Issues Releases Wiki Activity

Use binary safe diff in arc diff

Browse source 
Summary: Otherwise svn diff fails when the file is binary but the "svn:mime-type = application/x-shellscript" is missing in it.

Test Plan: arc diff succeeded against deleting a binary file which doesn't have svn:mime-type = application/x-shellscript.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D2915
This commit is contained in:
Jason Ge 2012-07-03 13:25:49 -07:00
parent 088091ca40
commit f9415e37d0
2 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
scripts/repository/binary_safe_diff.sh Executable file
View file

@ -0,0 +1,6 @@
#!/bin/sh
diff "$@"
if [ "$?" = "2" ]; then
exit 1
fi

5
src/repository/api/ArcanistSubversionAPI.php
View file

@ -268,9 +268,12 @@ final class ArcanistSubversionAPI extends ArcanistRepositoryAPI {
// itself (such as property changes) and also give us changes to any
// files within the directory (basically, implicit recursion). We don't
// want that, so prevent recursive diffing.
$root = phutil_get_library_root('arcanist');
return new ExecFuture(
'(cd %s; svn diff --depth empty --diff-cmd diff -x -U%d %s)',
'(cd %s; svn diff --depth empty --diff-cmd %s -x -U%d %s)',
$this->getPath(),
$root.'/../scripts/repository/binary_safe_diff.sh',
$this->getDiffLinesOfContext(),
$path);
}