Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 422675

Summary: Show dialog for editing merge commit message when doing a merge
Product: [Technology] EGit Reporter: Jon Schewe <jpschewe>
Component: UIAssignee: Project Inbox <egit.ui-inbox>
Status: RESOLVED WORKSFORME QA Contact:
Severity: enhancement    
Priority: P3 CC: matthias.sohn, robin
Version: unspecifiedKeywords: helpwanted
Target Milestone: ---   
Hardware: PC   
OS: Mac OS X   
Whiteboard:

Description Jon Schewe CLA 2013-11-27 09:58:38 EST
It would be nice to be able to edit the commit message for a merge rather than always using the default message.
Comment 1 Robin Stocker CLA 2013-11-27 16:50:27 EST
You can edit the message by amending the commit after merging. Is that not enough? I'm not sure showing a dialog for each merge is a good idea, or what would you propose?
Comment 2 Jon Schewe CLA 2013-11-27 17:21:41 EST
Current versions of git have you edit the merge messages by default now, so I don't see why bringing up a dialog in the same fashion would be a bad idea.

Using the ammend previous commit is doable. Another option would be to have another checkbox on the merge dialog to say do the merge, but don't commit.
Comment 3 Robin Stocker CLA 2013-11-27 17:28:45 EST
(In reply to Jon Schewe from comment #2)
> Current versions of git have you edit the merge messages by default now, so
> I don't see why bringing up a dialog in the same fashion would be a bad idea.

Hm, does it also do that when a merge happens as part of a pull?

> Using the ammend previous commit is doable. Another option would be to have
> another checkbox on the merge dialog to say do the merge, but don't commit.

That's also already available in EGit 3.1, see the screenshot of the merge dialog here:

https://wiki.eclipse.org/EGit/New_and_Noteworthy/3.1#Features
Comment 4 Jon Schewe CLA 2013-11-27 17:33:51 EST
(In reply to Robin Stocker from comment #3)
> (In reply to Jon Schewe from comment #2)
> > Current versions of git have you edit the merge messages by default now, so
> > I don't see why bringing up a dialog in the same fashion would be a bad idea.
> 
> Hm, does it also do that when a merge happens as part of a pull?
> 
I don't know, I don't use pull, I always use fetch and merge. 

> > Using the ammend previous commit is doable. Another option would be to have
> > another checkbox on the merge dialog to say do the merge, but don't commit.
> 
> That's also already available in EGit 3.1, see the screenshot of the merge
> dialog here:
> 
> https://wiki.eclipse.org/EGit/New_and_Noteworthy/3.1#Features

OK, I'll just need to wait for 3.1 then as I'm on 3.0. When should I expect 3.1 to show up in the update channels?
Comment 5 Robin Stocker CLA 2013-11-27 17:38:57 EST
(In reply to Jon Schewe from comment #4)
> OK, I'll just need to wait for 3.1 then as I'm on 3.0. When should I expect
> 3.1 to show up in the update channels?

Kepler SR2 (4.3.2) should come with the upcoming EGit 3.2. If you want it now, you can also add the stable EGit update site and update, works nicely:

http://download.eclipse.org/egit/updates
Comment 6 Jon Schewe CLA 2013-11-27 23:31:57 EST
Got it, then I don't have an issue with this.
Comment 7 Matthias Sohn CLA 2013-11-28 05:24:56 EST
(In reply to Jon Schewe from comment #4)
> OK, I'll just need to wait for 3.1 then as I'm on 3.0. When should I expect
> 3.1 to show up in the update channels?

3.1 is available here
http://www.eclipse.org/egit/download/
you may update your workbench using the egit main update site: http://download.eclipse.org/egit/updates
Comment 8 Robin Stocker CLA 2013-11-28 07:50:29 EST
Changing from FIXED to WORKSFORME as there are multiple ways to do this already, but showing a dialog was not implemented with this bug (so it's not FIXED).