Mailing-List: contact dev-help@ant.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Ant Developers List" <dev@ant.apache.org>
Received-SPF: pass (asf.osuosl.org: domain of antoine@gmx.de designates
 213.165.64.21 as permitted sender)
Date: Fri, 16 Jun 2006 02:00:04 +0200
Message-ID: <20060616000004.39980@gmx.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
From: "Antoine Levy-Lambert" <antoine@gmx.de>
In-Reply-To: <255d8d690606151314g691a961axbd1d1f157e303aff@mail.gmail.com>
References: <20060615024736.D25AB1A983A@eris.apache.org>	
 <255d8d690606151126j2fd70d40la9c0fdf3a074bbe1@mail.gmail.com>	
 <20060615193714.39980@gmx.net>
 <255d8d690606151314g691a961axbd1d1f157e303aff@mail.gmail.com>
Subject: Re: Re: Re: svn commit: r414442 - in
 /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs:
 AbstractJarSignerTask.java ExecTask.java Javac.java Javadoc.java
 TempFile.java Touch.java XSLTProcess.java
To: Ant Developers List <dev@ant.apache.org>
Content-Transfer-Encoding: 7bit


-------- Original-Nachricht --------
Datum: Thu, 15 Jun 2006 15:14:57 -0500
Von: Dominique Devienne <ddevienne@gmail.com>
An: Ant Developers List <dev@ant.apache.org>
Betreff: Re: Re: svn commit: r414442 - in /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs: AbstractJarSignerTask.java ExecTask.java Javac.java Javadoc.java TempFile.java Touch.java XSLTProcess.java

> This has been deprecated by jkf in commit 396012, @since 1.7 apparently.
>
this is true
 
> So it can be undeprecated, and Kev's changes can be removed ;-)
> 
yes


> I appreciate Kev's efforts to remove the deprecations, but IMHO
> resolveFile(String) shouldn't have been deprecated.
> 
> Am I the only one to think that? --DD

I thought that Martin only added "since Ant 1.x.y" mentions next to already deprecated function calls.

I am enclined to follow your opinion.

Regards,

Antoine

> 
> On 6/15/06, Antoine Levy-Lambert <antoine@gmx.de> wrote:
> > Hello Dominique,
> >
> > Project.resolveFile(String fileName) is
> > @deprecated since 1.7
> >
> > so Kevin is removing our own calls to deprecated methods.
> >
> > The real question is why did we deprecate Project.resolveFile(String) in
> the first place ? Second, was there a consensus to do it ?
> >
> > Regards,
> >
> > Antoine
> > -------- Original-Nachricht --------
> > Datum: Thu, 15 Jun 2006 13:26:05 -0500
> > Von: Dominique Devienne <ddevienne@gmail.com>
> > An: Ant Developers List <dev@ant.apache.org>
> > Betreff: Re: svn commit: r414442 - in
> /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs: AbstractJarSignerTask.java ExecTask.java Javac.java
> Javadoc.java TempFile.java Touch.java XSLTProcess.java
> >
> > > On 6/14/06, kevj@apache.org <kevj@apache.org> wrote:
> > > > Author: kevj
> > > > Log: use fileutils.resolveFile
> > > > + private static final FileUtils FILE_UTILS =
> FileUtils.getFileUtils();
> > > > -  File keystoreFile = getProject().resolveFile(keystore);
> > > > + File keystoreFile =
> FILE_UTILS.resolveFile(getProject().getBaseDir(),
> > > keystore);
> > >
> > > Hmmm, I'm sorry, but I don't like these changes...
> > >
> > > Sprinkling static FILE_UTILS references in many classes that already
> > > have references to project, which has always been the "proper" mean to
> > > resolve a file, is not a desirable change IMHO.
> > >
> > > Just use a FILE_UTILS in Project.java once and for all, and be done
> > > with it, no? getProject().resolveFile(string) reads better, and has
> > > always been the "canonical" way to resolve a file to me.
> > >
> > > If other are in favor of your changes, i'd be happy to read the
> > > arguments for it, but otherwise I'm kinda -1 on this ;-) Thanks, --DD
> > >

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org