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

Bug 355689

Summary: Can't commit without entering comment
Product: [Technology] EGit Reporter: Dani Megert <daniel_megert>
Component: UIAssignee: Project Inbox <egit.ui-inbox>
Status: CLOSED WONTFIX QA Contact:
Severity: normal    
Priority: P3 CC: matthias.sohn, remy.suen, robin
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Dani Megert CLA 2011-08-24 07:01:40 EDT
org.eclipse.egit.ui_1.1.0.201108230343.jar

For simple/quick commits it should be possible to enter no comment. The same can be done in the CVS client.
Comment 1 Matthias Sohn CLA 2011-08-24 07:36:05 EDT
(In reply to comment #0)
> org.eclipse.egit.ui_1.1.0.201108230343.jar
> 
> For simple/quick commits it should be possible to enter no comment. The same
> can be done in the CVS client.

I think this is a bad practice. Also native git doesn't allow commits with empty commit message. If you do "git commit" using native git it will open an editor to enter a commit message, if you close the editor without providing a commit message it will abort the commit.
It has an option --allow-empty-message but this is intended for use with foreign SCM interface scripts [1].

[1] http://www.kernel.org/pub/software/scm/git/docs/git-commit.html
Comment 2 Dani Megert CLA 2011-08-24 07:42:20 EDT
(In reply to comment #1)
> (In reply to comment #0)
> > org.eclipse.egit.ui_1.1.0.201108230343.jar
> > 
> > For simple/quick commits it should be possible to enter no comment. The same
> > can be done in the CVS client.
> 
> I think this is a bad practice. 
I agree this is in most cases but not always and prohibiting this because one thinks one invented the one fits all cases isn't nice either. The same could be done like in the CVS client: it warns if the comment is empty and inserts "*** empty log message ***" if the user chooses to keep it empty.
Comment 3 Robin Stocker CLA 2013-05-19 19:01:49 EDT
I don't think we will change this -> WONTFIX.
Comment 4 Dani Megert CLA 2013-06-06 05:10:37 EDT
(In reply to comment #3)
> I don't think we will change this -> WONTFIX.

Fair enough.