Re: [PATCH] rebase --preserve-merges: ignore merge.log config

2013-08-21 Thread Eric Sunshine
On Wed, Aug 21, 2013 at 2:48 PM, Ralf Thielow ralf.thie...@gmail.com wrote:
 When merge.log config is set, rebase --preserve-merges
 will add the log lines to the message of the rebased merge
 commit.

 A rebase should not modify a commit message automatically.

 Teach git-rebase to ignore that configuration by passing --no-log
 to the git-merge call.

 Signed-off-by: Ralf Thielow ralf.thie...@gmail.com
 ---
 diff --git a/t/t3409-rebase-preserve-merges.sh 
 b/t/t3409-rebase-preserve-merges.sh
 index 2e0c364..2454811 100755
 --- a/t/t3409-rebase-preserve-merges.sh
 +++ b/t/t3409-rebase-preserve-merges.sh
 @@ -96,4 +108,17 @@ test_expect_success 'rebase -p preserves no-ff merges' '
 )
  '

 +test_expect_success 'rebase -p ignores merge.log config' '
 +   (
 +   cd clone4 
 +   git fetch 
 +   git -c merge.log=1 rebase -p origin/topic 
 +   cat expected -\EOF 
 +
 +   EOF

This might be clearer with a simple 'echo' instead of 'cat' with heredoc:

  echo expected 

 +   git log --format=%b -1 current
 +   test_cmp expected current
 +   )
 +'
 +
  test_done
 --
 1.8.4.rc4.dirty
--
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


Re: [PATCH] rebase --preserve-merges: ignore merge.log config

2013-08-21 Thread Junio C Hamano
Ralf Thielow ralf.thie...@gmail.com writes:

 When merge.log config is set, rebase --preserve-merges
 will add the log lines to the message of the rebased merge
 commit.

 A rebase should not modify a commit message automatically.

 Teach git-rebase to ignore that configuration by passing --no-log
 to the git-merge call.

 Signed-off-by: Ralf Thielow ralf.thie...@gmail.com
 ---

Thanks; will queue with the following squashed-in.

 t/t3409-rebase-preserve-merges.sh | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/t/t3409-rebase-preserve-merges.sh 
b/t/t3409-rebase-preserve-merges.sh
index 2454811..8c251c5 100755
--- a/t/t3409-rebase-preserve-merges.sh
+++ b/t/t3409-rebase-preserve-merges.sh
@@ -113,10 +113,8 @@ test_expect_success 'rebase -p ignores merge.log config' '
cd clone4 
git fetch 
git -c merge.log=1 rebase -p origin/topic 
-   cat expected -\EOF 
-
-   EOF
-   git log --format=%b -1 current
+   echo expected 
+   git log --format=%b -1 current 
test_cmp expected current
)
 '
--
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