[PATCH 1/5 v2] contrib/diffall: comment actual reason for 'cdup'

From

Tim Henigan <tim.henigan@gmail.com>

Date

Mar 14, 2012, 16:38 UTC

Message-ID

<1331743086-32304-2-git-send-email-tim.henigan@gmail.com>

In-Reply-To

<1331743086-32304-1-git-send-email-tim.henigan@gmail.com>

Signed-off-by: Tim Henigan <tim.henigan@gmail.com>
---

v2 did not affect this patch.

 contrib/diffall/git-diffall |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/contrib/diffall/git-diffall b/contrib/diffall/git-diffall
index 9bbd27f..d706a6d 100755
--- a/contrib/diffall/git-diffall
+++ b/contrib/diffall/git-diffall
@@ -36,7 +36,9 @@ fi
 
 start_dir=$(pwd)
 
-# needed to access tar utility
+# All the file paths returned by the diff command are relative to the root
+# of the working copy. So if the script is called from a subdirectory, it
+# must switch to the root of working copy before trying to use those paths.
 cdup=$(git rev-parse --show-cdup) &&
 cd "$cdup" || {
 	echo >&2 "Cannot chdir to $cdup, the toplevel of the working tree"

-- 
1.7.10.rc0