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

Bug 463653

Summary:	Remove workaround of "copy up" Info.plist in config feature.
Product:	[Eclipse Project] Platform	Reporter:	David Williams <david_williams>
Component:	Releng	Assignee:	David Williams <david_williams>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	4.5
Target Milestone:	4.5 M7
Hardware:	PC
OS:	Linux
Whiteboard:
Bug Depends on:
Bug Blocks:	434596

Description David Williams

2015-04-01 04:36:57 EDT

Long history, but p2 has now fixed so "cfBundleNamed" should be "org.eclipse.Eclipse" instead of the troublesome "org.eclipse.eclipse". 

This should allow us to no longer "copy up" the template file, and just let Tycho generate it, as we do everwhere else. 

This has the advantage of "less dirt" in build tree (even though we hide is, with .gitignore). 

Also avoids "cross repository" dependencies, which are part of the cause of "monolithic builds". (as well as fragile builds). 

If this works, I believe Equinox can delete their copy.

Comment 1 David Williams

2015-04-09 22:16:14 EDT

I'm not sure Equinox can remove it's copy ... I believe it does become part of "executable feature" and is needed there. 

But removing the "copy up", in local test builds, now create the exact same Info.plist as before, with the case correct, and believe the pom clean up alone is worth removing the workaround. 

http://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/commit/?id=e3049554f38433de2748b00a0b7c2b0df01587ac

I think I can remove the ".gitignore" now too, which was put in place to avoid the "direct" of the copy-up. (bug 459584).