Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316608 - [Compatibility] no build ID in 4.0 SDK
Summary: [Compatibility] no build ID in 4.0 SDK
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 1.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 1.0 RC0   Edit
Assignee: John Arthorne CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-11 09:47 EDT by Paul Webster CLA
Modified: 2010-06-14 16:39 EDT (History)
2 users (show)

See Also:

Attachments
Fix (2.12 KB, text/plain)
2010-06-11 13:42 EDT, John Arthorne CLA 		no flags Details
View All 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 Paul Webster CLA 2010-06-11 09:47:37 EDT 
When I execute Copy Build Id I get this:

org.eclipse.core.commands.ExecutionException: No build ID in this instance.
	at org.eclipse.ui.internal.ide.commands.CopyBuildIdToClipboardHandler.execute(CopyBuildIdToClipboardHandler.java:40)
	at org.eclipse.ui.internal.handlers.HandlerProxy.execute(HandlerProxy.java:293)
	at org.eclipse.ui.internal.handlers.E4HandlerProxy.execute(E4HandlerProxy.java:56)

Maybe the recent changes we made are preventing the build ID from being generated?

PW
Comment 1 Paul Webster CLA 2010-06-11 09:52:57 EDT 
It appears to have worked in I20100607-0856 but failed in I20100610-2220

PW
Comment 2 John Arthorne CLA 2010-06-11 10:02:40 EDT 
I'll look into this... I was playing around with some related files the other day.
Comment 3 John Arthorne CLA 2010-06-11 13:42:01 EDT 
Created attachment 171748 [details]
Fix

Apparently this is a known problem in the .product file editor, that it eats arbitrary properties that it doesn't know about. These files have to be edited by hand with a text editor. See bug 260328 for details.
Comment 4 John Arthorne CLA 2010-06-11 13:42:21 EDT 
Kim can you release this fix to the R4_HEAD stream?
Comment 5 Kim Moir CLA 2010-06-11 14:06:43 EDT 
I've released the patch. I have to run a test build now for a signing issue in bug 316056 which will also include this fix.
Comment 6 John Arthorne CLA 2010-06-11 16:32:51 EDT 
Assigning to myself to confirm fix.
Comment 7 Paul Webster CLA 2010-06-14 15:39:24 EDT 
I got a build ID out of I20100613-2136

PW
Comment 8 John Arthorne CLA 2010-06-14 16:39:42 EDT 
Me too. Marking fixed.