Mailing-List: contact notifications-help@ant.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ant.apache.org
From: bugzilla@apache.org
To: notifications@ant.apache.org
Subject: DO NOT REPLY [Bug 42046] properties get double-expanded in macrodefs
Date: Tue, 22 Nov 2011 12:51:55 +0000
Message-ID: <bug-42046-48744-8F9ZPaalwj@https.issues.apache.org/bugzilla/>
In-Reply-To: <bug-42046-48744@https.issues.apache.org/bugzilla/>
References: <bug-42046-48744@https.issues.apache.org/bugzilla/>
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0

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.