Use --no-decorate in the call to git log that tries to read the commit
message to produce patch names. Otherwise, if the user has set
log.decorate to short or full, the patch name will be less useful.
Modify the t-034.sh test case to demonstrate that this is needed.
Signed-off-by: Per Cederqvist <ced...@opera.com>
Signed-off-by: Josef 'Jeff' Sipek <jef...@josefsipek.net>
---
guilt-import-commit | 2 +-
regression/t-034.out | 2 ++
regression/t-034.sh | 2 ++
3 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/guilt-import-commit b/guilt-import-commit
index 45f2404..1da7c8e 100755
--- a/guilt-import-commit
+++ b/guilt-import-commit
@@ -26,7 +26,7 @@ disp "About to begin conversion..." >&2
disp "Current head: `git rev-parse \`git_branch\``" >&2
for rev in `git rev-list $rhash`; do
- s=`git log --pretty=oneline -1 $rev | cut -c 42-`
+ s=`git log --no-decorate --pretty=oneline -1 $rev | cut -c 42-`
# Try to convert the first line of the commit message to a
# valid patch name.
diff --git a/regression/t-034.out b/regression/t-034.out
index bda4399..5d81bd4 100644
--- a/regression/t-034.out
+++ b/regression/t-034.out
@@ -232,6 +232,7 @@ Date: Mon Jan 1 00:00:00 2007 +0000
Signed-off-by: Commiter Name <commiter@email>
% guilt init
+% git config log.decorate short
% guilt import-commit base..HEAD
About to begin conversion...
Current head: 2a8b1889aa5066193bac978e6bf5073ffcfa6541
@@ -259,6 +260,7 @@ Converting 45e81b51 as the_sequence_.lock-_is_forbidden
Converting eebb76e9 as the_sequence_-._is_forbidden
Done.
Current head: d4850419ccc1146c7169f500725ce504b9774ed0
+% git config log.decorate no
% guilt push -a
Applying patch..the_sequence_-._is_forbidden.patch
Patch applied.
diff --git a/regression/t-034.sh b/regression/t-034.sh
index f41f958..648d009 100755
--- a/regression/t-034.sh
+++ b/regression/t-034.sh
@@ -57,7 +57,9 @@ cmd git log
# Import all the commits to guilt.
cmd guilt init
+cmd git config log.decorate short
cmd guilt import-commit base..HEAD
+cmd git config log.decorate no
for patch in .git/patches/master/*.patch; do
touch -a -m -t "$TOUCH_DATE" "$patch"
--
1.8.3.1
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
