| Summary: | Global and local properties and property files | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Darin Swanson <Darin_Swanson> |
| Component: | Ant | Assignee: | 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
Deferred to post 2.1 Reopening... 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. 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. 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. Changes to AntPropertiesBlock, AntPropertiesTab and InternalAntRunner. Currently looks correct but need to ensure that global properties do not take precedence over local properties. Work is progressing on bug 34297..which is the model for all of these. Should be there early in M5 The bug with the most changes to milestone... Cleaned up some of the last problems here. Global properties were always applied for remote ant builds. Fixed in AntLaunchDelegate. Please verify DarinW. 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. Changes in AntLaunchDelegate Please verify (that I understand the code :-) Good catch..you understand the code :-) Verified. |