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

Bug 338366

Summary: Web package needs to use p2 to publish the root files
Product: z_Archived Reporter: Hugues Malphettes <hmalphettes>
Component: RTPAssignee: Hugues Malphettes <hmalphettes>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Hugues Malphettes CLA 2011-02-28 02:46:02 EST
At the moment, it is using the mvn assembly plugin to simply put together an install.
We need to publish the root files using p2 to supporting correctly installing the Web package with the rtp bootstrapper.
This will also fix the fact that currently start.sh and start.bat don't work: those 2 scripts require the eclipse.ini file and that file is generated by p2.

The tycho build requires us to use either the intalio branch of tycho or tycho-0.11-snapshots at the moment. The tycho bugs related to publishing root files:
TYCHO-465, TYCHO-566.

I'll try with the 0.11-snapshots builds first.
Comment 1 Hugues Malphettes CLA 2011-03-01 03:03:34 EST
Fixed with the commit 9bff6c20ee1190583f568c18fa215baf213d3eb6
Also refactored the rt basic package product to use p2 for the publishing of the root files as agreed with Holger on IRC.
Comment 2 Hugues Malphettes CLA 2011-03-01 04:08:13 EST
(In reply to comment #1)
> Fixed with the commit 9bff6c20ee1190583f568c18fa215baf213d3eb6
> Also refactored the rt basic package product to use p2 for the publishing of
> the root files as agreed with Holger on IRC.
With the current build, we still need to manually change the permission on start.sh for both packages:
   chmod +x start.sh
before we can distribute the tar.gz.
Otherwise as far as I know the build is complete.

Once TYCHO-566 is fixed we won't need this do this manually anymore.
Comment 3 Hugues Malphettes CLA 2011-03-01 05:54:54 EST
Please note that we will need a new build of orbit to fix a broken javax.servlet.jsp-2.1 as reported here: 338515
In the mean time we could point the rtp build an older version of jetty: based on 7.2.2 instead of 7.3.0
Comment 4 Hugues Malphettes CLA 2011-03-07 22:30:38 EST
(In reply to comment #3)
> Please note that we will need a new build of orbit to fix a broken
> javax.servlet.jsp-2.1 as reported here: 338515
> In the mean time we could point the rtp build an older version of jetty: based
> on 7.2.2 instead of 7.3.0

Bug 338515 is fixed.