ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mariusz Nowostawski <mari...@marni.otago.ac.nz>
Subject Re: [PATCH] make Execute take advantage of 1.3's CWD support
Date Wed, 09 Aug 2000 22:26:45 GMT
On 9 Aug 2000, Stefan Bodewig wrote:
> >>>>> "JG" == Jesse Glick <Jesse.Glick@netbeans.com> writes:
>  JG> It should also correct a bug in ExecTask that antRun was used
>  JG> even when no 'dir' attribute was given (the working directory did
>  JG> not need to be set at all).
> Here I disagree.
> Whenever we specify a file or directory, we assume them to be relative
> to the project's basedir. Following this line of thought I interpret a
> missing dir attribute as "use the basedir" - which may very well be
> something other than user.dir. If it happens to be the same, we don't
> change the directory anyway.

I agree with Stefan here.

>  JG> Two other thoughts about using antRun (not antRun.bat, only the
>  JG> POSIX version):
> 
> I hadn't looked into the script before - I'd like to add another
> point:
> If I happen to have a script /tmp/ls.sh, antRun will invoke this
> script instead of /usr/bin/ls for
> <exec dir="/tmp" executable="ls" />
> which is not what I'd expect - quite the opposite, there is a reason I
> don't have . in my PATH.
> Any objections against dropping this behavior?

No objections. 
I will have a look at the current "exec"-related stuff and will test it
today or tommorrow, I promise ;o)


Mime
View raw message