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

Bug 335793

Summary: Manifest versions can be corrupted
Product: [Modeling] TMF Reporter: Ed Willink <ed>
Component: XtextAssignee: Project Inbox <tmf.xtext-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: moritz.eysholdt, sebastian.zarnekow
Version: 2.0.0Flags: moritz.eysholdt: indigo+
Target Milestone: RC1   
Hardware: PC   
OS: Windows Vista   
Whiteboard:

Description Ed Willink CLA 2011-01-30 14:23:41 EST
After regenerating an editor the manifest can be corrupted as below:

Require-Bundle: org.eclipse.core.runtime,
 org.eclipse.ocl.examples.xtext.base;bundle-version="[3.1.0,
 4.0.0)";visibility:=reexport,
 org.eclipse.xtext.util;bundle-version="[2.0.0,
 3.0.0)";visibility:=reexport,
 org.antlr.runtime;bundle-version="[3.2.0,
 org.eclipse.emf.common;visibility:=reexport,
 org.eclipse.xtext.common.types;bundle-version="[2.0.0,
 org.eclipse.xtext,
 org.eclipse.emf.ecore

[This was a problem occasionally in Xtext 1.0 as well as now 2.0M4 and a half]
Comment 1 Ed Willink CLA 2011-01-30 15:15:12 EST
I think the original was:

Require-Bundle: org.eclipse.core.runtime,
 org.eclipse.ocl.examples.xtext.base;bundle-version="[3.1.0,4.0.0)";visibility:=reexport,
 org.eclipse.xtext.util;bundle-version="[2.0.0,3.0.0)";visibility:=reexport,
 org.antlr.runtime;bundle-version="[3.2.0,4.0.0)";visibility:=reexport,
 org.eclipse.emf.common;visibility:=reexport,
 org.eclipse.xtext.common.types;bundle-version="[2.0.0,3.0.0)";visibility:=reexport
Comment 2 Ed Willink CLA 2011-05-05 01:26:11 EDT
Ping. Any news on this? Still a problem on M7.

I'm getting a little frustrated at having to do a Replace with Latest from HEAD on 8 manifests after each editor regeneration.
Comment 3 Sebastian Zarnekow CLA 2011-05-05 11:23:52 EDT
A patch is most welcome.
Comment 4 Moritz Eysholdt CLA 2011-05-16 11:45:49 EDT
fixed by splitting the comma-separated-list of exported packages quote-aware.

pushed to master.
Comment 5 Ed Willink CLA 2011-05-16 12:25:23 EDT
Thanks Moritz. I was beginning to think I might need to take up Sebastian's patch request.
Comment 6 Karsten Thoms CLA 2017-09-19 18:04:01 EDT
Closing all bugs that were set to RESOLVED before Neon.0
Comment 7 Karsten Thoms CLA 2017-09-19 18:14:05 EDT
Closing all bugs that were set to RESOLVED before Neon.0