Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 222125 - App container fails when app is dynamically installed
Summary: App container fails when app is dynamically installed
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: Compendium (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M6   Edit
Assignee: Thomas Watson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 208021
  Show dependency tree
 
Reported: 2008-03-10 17:38 EDT by John Arthorne CLA
Modified: 2008-03-17 16:14 EDT (History)
2 users (show)

See Also:

Attachments
patch (7.41 KB, patch)
2008-03-11 11:53 EDT, Thomas Watson CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description John Arthorne CLA 2008-03-10 17:38:00 EDT 
The automated tests define an application to be run. The test scripts unzip this into the eclipse/dropins folder, and start. However, the p2 reconcile happens after the equinox.app bundle starts. The app fails to load because it hasn't been discovered yet.  We should consider making equinox app more dynamic aware - if the default application is not available, wait until it is available and then run it.
Comment 1 Thomas Watson CLA 2008-03-11 11:53:12 EDT 
Created attachment 92182 [details]
patch

This patch waits to throw the missing application error until the very last possible moment.  This is when EclipseStarter has control back of the main thread after setting the start-level.

This should allow bundles with applications defined to be installed after the app bundle has been started but before the EclipseStarter has control back of the main thread.

John, could you test this patch out in your environment to see if it fixes your problems?
Comment 2 DJ Houghton CLA 2008-03-13 10:50:01 EDT 
John has tested this and we have released it and tagged for a test i-build.
Comment 3 DJ Houghton CLA 2008-03-17 15:58:44 EDT 
Tom, I think we can close this now?
Comment 4 Thomas Watson CLA 2008-03-17 16:14:18 EDT 
Yes we can.  In for M6