Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 315045 - Resulting zips should contain proper rootfiles
Summary: Resulting zips should contain proper rootfiles
Status: RESOLVED WONTFIX
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Dash Athena (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 critical (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 315057
  Show dependency tree
 
Reported: 2010-05-31 06:02 EDT by Anna Dushistova CLA
Modified: 2012-01-30 11:32 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anna Dushistova CLA 2010-05-31 06:02:17 EDT
It seems like right now the Eclipse SDK rootfiles are getting overwritten with the rootfiles that come with dependencies.
Zips should either allow to include rootfiles from the project or make Eclipse SDK ones the default ones.
Comment 1 Martin Oberhuber CLA 2010-05-31 07:24:55 EDT
The main problem here is the Eclipse SUA ("notice.html") which has been revised for Helios. We must not ship and ZIP's with Helios where the notice.html is dated before 2010.

I'm marking this as critical because this does have legal implications for the Helios release.
Comment 2 Nick Boldt CLA 2010-06-01 16:35:59 EDT
How are you gentlemen!

Not sure I understand the problem. Per bug 307016, you can now copy the rootfiles from your generated SDK into other zips using this task in your buildExtra (if they're not appearing there already):

<antcall target="addRootfilesToZip">
  <param name="targetZip" value="${buildDirectory}/${buildLabel}/${allZip}" />
</antcall>

Did someone set you up the bomb?

You really should take off every zip, for great justice! Instead, use p2 repos only, for they will properly install everything (including rootfiles) following p2's proper rules.

AYBABTU memes aside, what exactly is the issue here? Your zips contain incorrect rootfiles? How are they getting there? AFAIK the source of rootfiles is your features and their build.properties files.

http://help.eclipse.org/galileo/index.jsp?topic=/org.eclipse.pde.doc.user/tasks/pde_rootfiles.htm
Comment 3 Martin Oberhuber CLA 2010-06-01 16:48:19 EDT
They get there by this in buildExtra.xml (for example):

<!-- build aTargetZip -->
<zip destfile="${buildDirectory}/${buildLabel}/aTarget.zip" update="true">
	<zipfileset src="${buildDirectory}/${buildLabel}/${allZip}" >
		<include name="**/org.eclipse.*.aTarget*/**" />
	</zipfileset>
	<zipfileset refid="rootfiles" />
</zip>


The request is giving guidance where the refid="rootfiles" comes from, and how to replace it with something proper.

With respect to rootfiles defined in the features, I'm afraid this doesn't help us here since Athena builds a single master feature ... or should we be holding our rootfiles as siblings of the master feature?
Comment 4 Nick Boldt CLA 2010-06-01 17:41:57 EDT
(In reply to comment #3)
> They get there by this in buildExtra.xml (for example):
> 
> <!-- build aTargetZip -->
> <zip destfile="${buildDirectory}/${buildLabel}/aTarget.zip" update="true">
>     <zipfileset src="${buildDirectory}/${buildLabel}/${allZip}" >
>         <include name="**/org.eclipse.*.aTarget*/**" />
>     </zipfileset>
>     <zipfileset refid="rootfiles" />
> </zip>
> 
> 
> The request is giving guidance where the refid="rootfiles" comes from, and how
> to replace it with something proper.

Assuming these files are correctly located in your SDK zip, then you can copy them over into the aTarget.zip like this:

<zip destfile="${buildDirectory}/${buildLabel}/aTarget.zip" update="true">
    <zipfileset src="${buildDirectory}/${buildLabel}/${allZip}" >
        <include name="**/org.eclipse.*.aTarget*/**" />
    </zipfileset>
</zip>
<zip destfile="${buildDirectory}/${buildLabel}/aTarget.zip" update="true">
  <zipfileset src="${buildDirectory}/${buildLabel}/${SDKZip}"
    includes="eclipse/epl-v10.html, eclipse/notice.html"/>
</zip>

Or, like this should work too:

<zip destfile="${buildDirectory}/${buildLabel}/aTarget.zip" update="true">
    <zipfileset src="${buildDirectory}/${buildLabel}/${allZip}" >
        <include name="**/org.eclipse.*.aTarget*/**" />
    </zipfileset>
</zip>
<ant target="addRootfilesToZip" antfile="${helper}">
  <param name="targetZip" value="${buildDirectory}/${buildLabel}/aTarget.zip" />
</antcall>
 
> With respect to rootfiles defined in the features, I'm afraid this doesn't help
> us here since Athena builds a single master feature ... or should we be holding
> our rootfiles as siblings of the master feature?

Why not just copy the rootfiles into the master feature, so that they're available in the Master zip, and therefore can be added to the SDK, runtime, examples, and any other non-p2-repo-zips you feel you need to create. :)
Comment 5 Nick Boldt CLA 2010-06-01 17:42:44 EDT
> <ant target="addRootfilesToZip" antfile="${helper}">
>   <param name="targetZip" value="${buildDirectory}/${buildLabel}/aTarget.zip"
> />
> </antcall>

Oops. Make that <ant>...</ant>, not <ant>...</antcall>.
Comment 6 Martin Oberhuber CLA 2010-06-01 17:55:38 EDT
(In reply to comment #4)

> Why not just copy the rootfiles into the master feature, so that they're
> available in the Master zip, and therefore can be added to the SDK, runtime,
> examples, and any other non-p2-repo-zips you feel you need to create. :)

I like this approach since it makes us the owner of the rootfiles (rather than copying from the target Eclipse). Testing this now.
Comment 7 Anna Dushistova CLA 2011-12-19 16:27:53 EST
Guess it's not valid anymore.