Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 327291 - Add commit hash to history view
Summary: Add commit hash to history view
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 0.9.0   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Mathias Kinzler CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-07 20:22 EDT by Bernard Leach CLA
Modified: 2010-10-26 07:16 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Bernard Leach CLA 2010-10-07 20:22:35 EDT 
Build Identifier: 20100218-1602

It would be helpful to display the commit hash value in the history view.

When you compare two revisions from the history it shows the commit hash next  the file name but from the history view table its not clear which revision is which.  To find out which revision is which you need to select a single revision from the history view and then check the detail for that commit.

Reproducible: Always

Steps to Reproduce:
1. Open history view for a resource
2. Select two entries
3. Right click and select 'Compare with each other'
4. Diff view shows the two revisions but they are differentiated by their commit hash which is not visible in the history table.
Comment 1 Stefan Lay CLA 2010-10-08 10:17:57 EDT 
Code review at http://egit.eclipse.org/r/#change,1714
Comment 2 Matthias Sohn CLA 2010-10-15 05:26:14 EDT 
merged as 5586cfcde8e494f80a1fec31207bc9bdd6d4aad5
Comment 3 Stefan Lay CLA 2010-10-20 10:40:51 EDT 
The search for the commit id had existed before: You had to choose "commit" in the Find-Listbox. The new search for "Id" does the same. 

I would suggest to remove the old entry "commit" because the new one, "Id", does better fit to the column label.
Comment 4 Mathias Kinzler CLA 2010-10-25 08:38:09 EDT 
Code review at http://egit.eclipse.org/r/#change,1789
Comment 5 Stefan Lay CLA 2010-10-26 07:16:54 EDT 
merged as e2af439afd8e7aeaac2ff70c67a21e11bf2beb7a