Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 222260 - [publisher] Incorrect showsplash argument
Summary: [publisher] Incorrect showsplash argument
Status: RESOLVED DUPLICATE of bug 274127
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: p2 (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Andrew Niefer CLA
QA Contact: Pascal Rapicault CLA
URL:
Whiteboard:
Keywords:
: 217740 (view as bug list)
Depends on:
Blocks: 264746
  Show dependency tree
 
Reported: 2008-03-11 12:26 EDT by Andrew Niefer CLA
Modified: 2009-04-28 22:20 EDT (History)
3 users (show)

See Also:


Attachments
Adds default splash screen support to the publisher (1.36 KB, patch)
2009-02-16 19:20 EST, Ian Bull CLA
no flags Details | Diff
mylyn/context/zip (8.81 KB, application/octet-stream)
2009-02-16 19:20 EST, Ian Bull CLA
no flags Details
Updated patch (2.56 KB, patch)
2009-02-16 22:25 EST, Ian Bull CLA
jeffmcaffer: iplog+
Details | Diff
mylyn/context/zip (4.75 KB, application/octet-stream)
2009-02-16 22:27 EST, Ian Bull CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Niefer CLA 2008-03-11 12:26:16 EDT
Framework admin is mangling the showplash argument and in general is getting it completely wrong.  It should not resolve the showsplash path in all cases.

Some effort is required during metadata generation to ensure a correct value.
1) If the splash bitmap comes from a feature/plugin, a CU fragment for the splash should probably be generated to get the correct path in the case of bundle pools.

2) If the splash bitmap is coming from the root files then a relative path would be required (which framework admin can relativize).  

3) I'm not sure how a absolute path for the bitmap should be handled.

See http://wiki.eclipse.org/Equinox_Launcher#Command_line_arguments for an outline of the semantics of the -showplash argument
Comment 1 Andrew Niefer CLA 2008-03-19 15:47:12 EDT
I have changed framework admin to not do anything to the -showsplash argument.  The normal value for eclipse "org.eclipse.platform" will work when the bundle pool in colocated with the install.

Still need to generate CUs to contribute the splash screen to the ini properly.
Comment 2 John Arthorne CLA 2008-05-06 17:31:40 EDT
Anything else to do here? Can you update the target milestone if there is more work?
Comment 3 Andrew Niefer CLA 2008-05-12 11:15:29 EDT
There is more work to do here, but it is too much to make it in 3.4
Comment 4 Andrew Niefer CLA 2008-07-15 10:55:35 EDT
*** Bug 217740 has been marked as a duplicate of this bug. ***
Comment 5 Ian Bull CLA 2009-02-12 13:16:49 EST
This bug is back :-).  Actually, the publish product work that we did doesn't manage this very well.
Comment 6 Ian Bull CLA 2009-02-13 13:27:40 EST
Andrew, where did you create the CU for the splash screen in Eclipse 3.4?  I can't seem to find anything in the generator for this.
Comment 7 Andrew Niefer CLA 2009-02-13 15:05:19 EST
The generator did not do anything for splash screen CUs, it just ignored the whole issue and -showsplash and osgi.splashPath just passed through with any other eclipse.ini or config.ini property.
Comment 8 Ian Bull CLA 2009-02-13 17:28:32 EST
did this work when the splash screen was in a bundle pool? 
Comment 9 Pascal Rapicault CLA 2009-02-14 16:02:05 EST
Be careful here as, like the initial bug mentioned, there could be something wrong going on in framework admin.
Comment 10 Ian Bull CLA 2009-02-16 18:57:06 EST
Ok, I see, splash screens don't work with the bundle pool.

I just installed Eclipse with the following director command:
-console -consolelog -application org.eclipse.equinox.p2.director.app.application
-metadataRepository http://download.eclipse.org/eclipse/updates/3.4 -artifactRepository http://download.eclipse.org/eclipse/updates/3.4 -installIU org.eclipse.sdk.ide -destination c:/es/testInstall/ -profile SDKProfile

Because no bundle pool is specified, it puts it in the p2 area.  When I launched, no splash screen is shown. 

Should I just mimic this behaviour in 3.5?
Comment 11 Ian Bull CLA 2009-02-16 19:20:25 EST
Created attachment 125837 [details]
Adds default splash screen support to the publisher

This patch sets up the splash screen the same way the generator did.  This patch should be reviewed and applied, but the bug report should remain open (there is still more work to do). This simply gets us to the same state with the publisher as we were with the generator.
Comment 12 Ian Bull CLA 2009-02-16 19:20:33 EST
Created attachment 125838 [details]
mylyn/context/zip
Comment 13 Ian Bull CLA 2009-02-16 22:25:35 EST
Created attachment 125847 [details]
Updated patch

This patch fixes a test case that was failing in the previous patch.
Comment 14 Ian Bull CLA 2009-02-16 22:27:05 EST
Created attachment 125848 [details]
mylyn/context/zip
Comment 15 Jeff McAffer CLA 2009-02-18 22:33:29 EST
Comment on attachment 125847 [details]
Updated patch

patch committed. Thanks. Leaving bug open to deal with the shared install splash problem.
Comment 16 Pascal Rapicault CLA 2009-04-28 22:20:00 EDT
Similar issues exist around the splash when the bundles are not located in the typical area. Closing as dup

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