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

Bug 361484

Summary:	git status hiccupped, let me commit twice
Product:	[ECD] Orion	Reporter:	Susan McCourt <susan>
Component:	Git	Assignee:	libing wang <libingw>
Status:	RESOLVED WORKSFORME	QA Contact:
Severity:	normal
Priority:	P3	CC:	libingw, tomasz.zarna
Version:	0.3
Target Milestone:	---
Hardware:	PC
OS:	Windows 7
Whiteboard:
Bug Depends on:	339848
Bug Blocks:

Description Susan McCourt

2011-10-19 18:56:16 EDT

I've seen this once before, I think, but definitely saw it today.
Latest code on orion.eclipse.org

I was committing a change to a local topic branch.  I pasted the bug number in the commit text and pushed "commit."  I got a progress message and then no update at all, so I wasn't sure it worked.  So I tried hitting commit again.  Same thing, progress message, no update on the page.  So I opened the log and saw two commits, a minute apart.  The first commit has the change, the second commit has no change at all.  

You can see the commits involved in case this matters, as I pushed them to a remote branch.

branch is origin/bug361481

This commit is the real one:
5595c5765292d098d2cf6a598872a73bef56f787

this is the empty one
d057e35606ce3d6947f24c3b4baced4bf073eba8

Comment 1 Tomasz Zarna

2011-10-20 09:50:41 EDT

This could have been spotted if we had bug 339848 fixed. For now, we could disable the Commit button until result of a previous commit has returned. A warning about an empty commit would also help.

Comment 2 libing wang

2011-10-20 10:11:34 EDT

(In reply to comment #1)
> This could have been spotted if we had bug 339848 fixed. For now, we could
> disable the Commit button until result of a previous commit has returned. A
> warning about an empty commit would also help.

Yes. The current logic to disable commit button is complicated (e.g. comment 4 at Bug 355735). We can disable commit button right away after it is clicked but I there is no way to bring it back unless you refresh.
Without solving the issue at the first place, everything else is just temporary. The best workaround I would recommend is to refresh the browser.

Comment 3 Susan McCourt

2011-10-20 10:50:56 EDT

(In reply to comment #1)
> This could have been spotted if we had bug 339848 fixed. For now, we could
> disable the Commit button until result of a previous commit has returned. A
> warning about an empty commit would also help.

if it's straightforward to detect an empty commit, a warning would reduce the noise in the repo, but this is not a big deal (post 0.3)

Mainly wanted to ensure we knew about it.  Sounds like a more substantive solution is planned (bug 339848.

Comment 4 John Arthorne

2015-04-27 11:00:08 EDT

Hasn't been seen in a long time, much of this has been rewritten since. We also have warning for empty commits now that will help if user clicks commit twice.