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

Bug 34971

Summary: Global and local properties and property files
Product: [Eclipse Project] Platform Reporter: Darin Swanson <Darin_Swanson>
Component: AntAssignee: Darin Swanson <Darin_Swanson>
Status: VERIFIED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: darin.eclipse
Version: 2.1   
Target Milestone: 3.0 M9   
Hardware: PC   
OS: Windows NT   
Whiteboard:
Bug Depends on: 45450    
Bug Blocks:    

Description Darin Swanson CLA 2003-03-13 16:41:57 EST
The properties tab for the Ant launch configuration shows nothing about the 
global properties.

It should be made more like the classpath page...option to override the global.
Currently you have to specify a local property to remove or override a global 
property.
Comment 1 Darin Swanson CLA 2003-03-13 16:42:20 EST
Deferred to post 2.1
Comment 2 Darin Swanson CLA 2003-06-03 15:58:42 EDT
Reopening...
Comment 3 Darin Wright CLA 2003-08-13 15:38:52 EDT
Note - since we are considering "appending" behavior in the classpath tab, we 
should do the same for properties. I.e. "default global properties plus my 
specific properties" (like "default classpath plus my specific classpath 
entries"). Also allow for modifying the globals. We need to come up with a 
consistent UI for this.
Comment 4 Darin Swanson CLA 2003-08-13 17:03:02 EDT
I have coded this up yesterday to mimic the current implementation of the 
classpath tab.
We can then adapt this to the changes that come out of the classpath tabs.
Comment 5 Darin Swanson CLA 2003-08-25 19:05:36 EDT
Released first version that mimics current implementation of the classpath tab.
For M4 we can come up with a consistent appending behavior for all of the tabs.
Comment 6 Darin Swanson CLA 2003-08-25 19:10:19 EDT
Changes to AntPropertiesBlock, AntPropertiesTab and InternalAntRunner.
Comment 7 Darin Swanson CLA 2003-08-27 13:53:35 EDT
Currently looks correct but need to ensure that global properties do not take 
precedence over local properties.
Comment 8 Darin Swanson CLA 2003-10-06 13:18:31 EDT
Work is progressing on bug 34297..which is the model for all of these.
Should be there early in M5
Comment 9 Darin Swanson CLA 2004-03-19 18:06:28 EST
The bug with the most changes to milestone...
Comment 10 Darin Swanson CLA 2004-05-05 13:46:42 EDT
Cleaned up some of the last problems here.
Global properties were always applied for remote ant builds. 

Fixed in AntLaunchDelegate.
Comment 11 Darin Swanson CLA 2004-05-05 13:48:20 EDT
Please verify DarinW.
Comment 12 Darin Wright CLA 2004-05-17 17:38:49 EDT
Modified code in AntLaunchDelegate. Looks like the loop to apply global 
properties was not required if there are user properties (i.e. we loop thru 
but ignore each gloabal property). Added code to shortcut the loop.
Comment 13 Darin Wright CLA 2004-05-17 17:39:10 EDT
Changes in AntLaunchDelegate
Comment 14 Darin Wright CLA 2004-05-17 17:39:28 EDT
Please verify (that I understand the code :-)
Comment 15 Darin Swanson CLA 2004-05-17 17:56:44 EDT
Good catch..you understand the code :-)

Verified.