From f794418ba9278123c23d6935bee9ff6dd22dac31 Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 7 Mar 2011 19:03:27 -0800 Subject: [PATCH] Arcanist: parse SVN "replaced" status Summary: SVN has a special 'replaced' status which was not being parsed correctly. Parse it correctly. Test Plan: Replaced a file, ran arc diff, got a sensible diff. Reviewed By: aran Reviewers: aran CC: aran Differential Revision: 60 --- src/repository/api/subversion/ArcanistSubversionAPI.php | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/repository/api/subversion/ArcanistSubversionAPI.php b/src/repository/api/subversion/ArcanistSubversionAPI.php index 4dd7fa7e..5edfbc19 100644 --- a/src/repository/api/subversion/ArcanistSubversionAPI.php +++ b/src/repository/api/subversion/ArcanistSubversionAPI.php @@ -110,6 +110,12 @@ class ArcanistSubversionAPI extends ArcanistRepositoryAPI { case 'added': $mask |= self::FLAG_ADDED; break; + case 'replaced': + // This is the result of "svn rm"-ing a file, putting another one + // in place of it, and then "svn add"-ing the new file. Just treat + // this as equivalent to "modified". + $mask |= self::FLAG_MODIFIED; + break; case 'modified': $mask |= self::FLAG_MODIFIED; break;