ant-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 42046] properties get double-expanded in macrodefs
Date Tue, 22 Nov 2011 12:51:55 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=42046

Stefan Bodewig <bodewig@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |1.8.3

--- Comment #7 from Stefan Bodewig <bodewig@apache.org> 2011-11-22 12:51:55 UTC ---
The resons for the propertyhelper-test failure turned out to be more subtle. 
after applying your patch the test was comparing the o.toString() for arg2 to o
and of course they were no the same.

I've modified the test to use a property who's value is a string and it passes.

What is happening here is that the loadproperties task doesn't work as expected
b the test when expanding properties with non-string values.  I.e. this is
another case where the double-expansion of properties cause a false positive
unit test.

svn revision 1204961 contains the patch and disabled the failing tests that I'm
going to address separately.

Thanks for your patience.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Mime
View raw message