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

Bug 371722

Summary: [Tooling] RWT Launcher does not work anymore
Product: [RT] RAP Reporter: Ivan Furnadjiev <ivan>
Component: RelengAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P2    
Version: 1.5   
Target Milestone: 1.5 M6   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Sketch
none
Proposed patch none

Description Ivan Furnadjiev CLA 2012-02-16 06:03:34 EST 
In the platform Juno M4 the Jetty was switched from version 6 to 8. This rendered our RWT launcher broken. It is broken because:
1. Jetty bundles have been changed.
2. org.mortbay.jetty.Main class does not exists anymore - there is no replacement in the platform Jetty bundles.
3. Jetty WebApp bundle is not part of the platform - we can't use WebAppContext anymore.
The first two problems are easily solved - we can create our own Jetty launcher class based on the old org.mortbay.jetty.Main class. The missing WebApp bundles cause more trouble. We have two possibilities:
1. Add Jetty WebApp bundle (with required bundles if any) to the tooling.
2. Do not use WebAppContext (web.xml) for registering the application in Jetty, but create a context manually in this new Jetty launcher class - create servlet with mapping, adding the listener, adding init parameters...
Comment 1 Ivan Furnadjiev CLA 2012-03-15 07:28:22 EDT 
Created attachment 212716 [details]
Sketch
Comment 2 Ivan Furnadjiev CLA 2012-03-16 08:30:43 EDT 
Created attachment 212782 [details]
Proposed patch
Comment 3 Ivan Furnadjiev CLA 2012-03-16 08:37:38 EDT 
This patch introduces our own Jetty lancher class (JettyLauncher, currently located in the org.eclipse.rap.ui.launch.rwt bundle). It is a stripped version of the old org.mortbay.jetty.Main class. Two additional bundles are needed:
org.eclipse.jetty.webapp
org.eclipse.jetty.xml
which are missing from the platform.
Comment 4 Ivan Furnadjiev CLA 2012-03-16 11:14:47 EDT 
Applied the patch to CVS HEAD.