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

Bug 315039

Summary: [patch] Eclipse launch configuration should inherit properties from target platform
Product: [Eclipse Project] PDE Reporter: Gunnar Wagenknecht <gunnar>
Component: UIAssignee: Curtis Windatt <curtis.windatt.public>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: curtis.windatt.public, jeffmcaffer
Version: 3.6Keywords: contributed
Target Milestone: 3.8 M6   
Hardware: All   
OS: All   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=175437
Whiteboard:
Attachments:
Description Flags
patch to inherit properties when launching default app
none
patch none

Description Gunnar Wagenknecht CLA 2010-05-31 04:30:48 EDT 
When creating a new Eclipse launch configuration from a target platform not containing products but an application PDE correctly recognizes the default application. However, due to the fix for bug 175437 all config.ini properties inherited from the target platform are cleared during launching.

My observation is that this fix is too drastic. The requirement in bug 175437 only talks about the splash screen. In this case the products indeed don't match.

However, I have a case where I create a new Eclipse launch config based on an
Eclipse application from my target platform. PDE correctly recognizes the
application in config.ini and sets the proper default application to launch. However, because of this fix other important properties are not copied.
Comment 1 Gunnar Wagenknecht CLA 2010-05-31 04:45:26 EDT 
Created attachment 170495 [details]
patch to inherit properties when launching default app

The attached patch adds an additional check to not clear properties when launching the default application but only when no product is launched.
Comment 2 Gunnar Wagenknecht CLA 2010-05-31 04:46:20 EDT 
This should be a 3.6.1 candidate. PDE team please remove target milestone if you think it shouldn't.
Comment 3 Curtis Windatt CLA 2010-08-10 11:04:44 EDT 
Moving to 3.7, this is not a regression (or at least not a recent one).
Comment 4 Gunnar Wagenknecht CLA 2010-09-23 11:24:38 EDT 
Created attachment 179467 [details]
patch

Revised patch while working on bug 326059.
Comment 5 Jeff McAffer CLA 2010-12-13 10:03:49 EST 
This seems pretty reasonable.  Basically the original changes assumed that product extensions were being used.  If just applications are being used we should have some similar behaviour.
Comment 6 Curtis Windatt CLA 2012-02-29 16:41:37 EST 
Yes, this change is quite reasonable.  Sorry that it's been left for so long.  Never enough committer time to go around :) Pushed the fix to master.