You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by bu...@apache.org on 2010/06/03 01:50:33 UTC

DO NOT REPLY [Bug 49373] New: Property expansion inaccurate for prefixed file-based property loads.

https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

           Summary: Property expansion inaccurate for prefixed file-based
                    property loads.
           Product: Ant
           Version: 1.8.1
          Platform: PC
        OS/Version: Windows NT
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Core tasks
        AssignedTo: notifications@ant.apache.org
        ReportedBy: apache-bt@g3i.de


Unexpected behaviour change between Ant 1.8.0 (OK) and Ant 1.8.1 (wrong).

 <property file="foo.properties" prefix="prfx" />

Loading properties from a file should apply the prefix only to the newly loaded
properties. The prefix should not be used for property expansions on the value
side.

Below build.xml works with Ant 1.8.0 and fails with Ant 1.8.1:

---8<---
<?xml version="1.0" encoding="UTF-8"?>
<project name="Test1" default="test1">

    <target name="test1">
        <echo file="foo.properties" message="abc=$${def}" />
        <property name="def" value="ghi" />
        <property name="prfx.def" value="jkl" />
        <property file="foo.properties" prefix="prfx" />
        <echo message="Value of $${prfx.abc} is ${prfx.abc}, should be ghi,
should not be jkl." />
    </target>

</project>
---8<---

1.8.0:
[echo] Value of ${prfx.abc} is ghi, should be ghi, should not be jkl.

1.8.1 (wrong):
[echo] Value of ${prfx.abc} is jkl, should be ghi, should not be jkl.

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

DO NOT REPLY [Bug 49373] Property expansion inaccurate for prefixed file-based property loads.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sbaramov@gmail.com

--- Comment #6 from Stefan Bodewig <bo...@apache.org> 2010-09-08 23:45:59 EDT ---
*** Bug 49901 has been marked as a duplicate of this bug. ***

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

DO NOT REPLY [Bug 49373] Property expansion inaccurate for prefixed file-based property loads.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

--- Comment #3 from Stefan Bodewig <bo...@apache.org> 2010-07-13 11:50:17 EDT ---
I've locally confirmed that ant 1.8.0 works fine for Jelly's tests but 1.8.1
breaks it, so the error in Jelly seems unrelated to this issue or its fix.

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

DO NOT REPLY [Bug 49373] Property expansion inaccurate for prefixed file-based property loads.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |owennnn@hotmail.com

--- Comment #5 from Stefan Bodewig <bo...@apache.org> 2010-08-21 03:55:32 EDT ---
*** Bug 49790 has been marked as a duplicate of this bug. ***

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

DO NOT REPLY [Bug 49373] Property expansion inaccurate for prefixed file-based property loads.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

--- Comment #2 from Stefan Bodewig <bo...@apache.org> 2010-06-29 23:42:26 EDT ---
svn revision 959177 looks cleaner

It may not be the final solution, though.  It looks as if we still have a
backwards compatibility issue that manifests itself in the
jelly-tags-ant build in Gump.

http://vmgump.apache.org/gump/public/commons-jelly/commons-jelly-tags-ant/gump_work/build_commons-jelly_commons-jelly-tags-ant.html

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

DO NOT REPLY [Bug 49373] Property expansion inaccurate for prefixed file-based property loads.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dhumeniuk@gmail.com

--- Comment #7 from Stefan Bodewig <bo...@apache.org> 2010-09-27 11:01:29 EDT ---
*** Bug 49989 has been marked as a duplicate of this bug. ***

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

DO NOT REPLY [Bug 49373] Property expansion inaccurate for prefixed file-based property loads.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |1.8.2

--- Comment #1 from Stefan Bodewig <bo...@apache.org> 2010-06-29 12:17:51 EDT ---
fixed with svn revision 959019

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

DO NOT REPLY [Bug 49373] Property expansion inaccurate for prefixed file-based property loads.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49373

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ngrigoriev@gmail.com

--- Comment #4 from Stefan Bodewig <bo...@apache.org> 2010-08-09 16:00:05 EDT ---
*** Bug 49729 has been marked as a duplicate of this bug. ***

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