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

Bug 313348

Summary:	RCP description wrong and too much content
Product:	[Technology] EPP	Reporter:	Jeff McAffer <jeffmcaffer>
Component:	rcp-package	Assignee:	Project Inbox <epp.packager-inbox>
Status:	CLOSED DUPLICATE	QA Contact:
Severity:	normal
Priority:	P3	CC:	mknauer, wayne.beaton
Version:	unspecified
Target Milestone:	1.3.0
Hardware:	PC
OS:	Mac OS X - Carbon (unsup.)
Whiteboard:
Bug Depends on:
Bug Blocks:	230357

Description Jeff McAffer

2010-05-18 09:19:43 EDT

Looking at the current RCP package the following things seem off:

- The description talks abotu ECF being in the package. The list of features does not mention ECF.  It is not clear what it would mean to RCP developers to have ECF in the IDE anyway.

- The package includes source for things like PDE, CVS and JDT. This should not be needed.  Source for the Platform itself is perhaps understandable as it enables the convenience of targetting the IDE itself when writing RCP apps.

Comment 1 Markus Knauer

2010-05-18 09:43:59 EDT

(In reply to comment #0)
> - The description talks abotu ECF being in the package. The list of features
> does not mention ECF.  It is not clear what it would mean to RCP developers to
> have ECF in the IDE anyway.

Good point... that was a leftover from earlier times. I've removed it in the description (visible with RC1).

> - The package includes source for things like PDE, CVS and JDT. This should not
> be needed.  Source for the Platform itself is perhaps understandable as it
> enables the convenience of targetting the IDE itself when writing RCP apps.

The sources are there for historical reasons. When we created the package, we had the intention to make it similar to the "SDK" (or "Eclipse Classic") which contains all those sources. But I fully agree that we should consider removing them in Helios.

If there is no objection, I will remove the source features in Helios RC1.

In addition to that, there are some other changes in bug 230357 waiting for feedback...

Comment 2 Wayne Beaton

2010-05-18 13:22:33 EDT

I'll raise a vote to keep the source in.

I'd rather keep the source in for things like PDE, CVS, JDT, and Mylyn. I use this particular package myself and have found the inclusion of this code convenient.

Comment 3 Markus Knauer

2010-05-21 07:44:22 EDT

Closing as "duplicate" of bug 230357

* The description has been adjusted in the soon to be released Helios RC1. If you have better ideas how to describe the package content in just a few words, please add them to the other bug.

* I haven't removed the sources yes, but I will reconsider it for RC2. Again, this is covered in the other bug. ;-)

*** This bug has been marked as a duplicate of bug 230357 ***