I think that we will just have to put the old code (1.7) back in. Peter
On Wed, Feb 24, 2010 at 4:10 PM, Stefan Bodewig <bode...@apache.org> wrote: > On 2010-02-18, Matt Benson <gudnabr...@gmail.com> wrote: > >> This problem happens in src/main/org/apache/tools/ant/property/ >> ResolvePropertyMap.java . Peter, you extracted this class some two >> years ago, but at the point in the code at which the offense takes >> place, there is the comment: > >> // Note: the master overrides (even if the name is subsequently >> // prefixed) > > If you trace it back to the older code you'll see it corresponds to a > check whether the property is defined outside of the map as well - and > if it is use the outside value - to enforce property immutability. > > To account for the bug report it may be a good idea to ignore existing > properties only if we are importing with a prefix. This does require > adding a new overload with a prefix argument, but it's the only way we > can re-gain the old behavior for property files AFAICS. > > Stefan > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org > For additional commands, e-mail: dev-h...@ant.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org
