On Tue, 04 Mar 2008, <[EMAIL PROTECTED]> wrote:
> add "testcase" for bug 44323
The quotes are here because the build file
,----
| <project>
| <echo>Before import: ${foo}</echo>
| <property name="foo" value="bar"/>
| <import file="./recursive-selfimport.xml"/>
| </project>
`----
demonstrates the problem:
,----
| [EMAIL PROTECTED]:~/ASF/ant/ant-HEAD> ./build.sh -f
src/etc/testcases/taskdefs/import/recursive-selfimport.xml
| Buildfile: src/etc/testcases/taskdefs/import/recursive-selfimport.xml
| Before import: ${foo}
| Before import: bar
|
| BUILD SUCCESSFUL
| Total time: 0 seconds
`----
The line "Before import: bar" must not be there at all.
But, the bug only surfaces when the build file name has been given as
a relative path. Contrast with
,----
| [EMAIL PROTECTED]:~/ASF/ant/ant-HEAD> ./build.sh -f
`pwd`/src/etc/testcases/taskdefs/import/recursive-selfimport.xml
| Buildfile:
/home/stefan/ASF/ant/ant-HEAD/src/etc/testcases/taskdefs/import/recursive-selfimport.xml
| Before import: ${foo}
|
| BUILD SUCCESSFUL
| Total time: 1 second
`----
Our AntUnit as well as our JUnit tests (at least when run inside of
BuildFileTest) will always pass the absolute file name into the test,
so the bug is hidden.
Stefan
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
