This has been deprecated by jkf in commit 396012, @since 1.7 apparently.
So it can be undeprecated, and Kev's changes can be removed ;-) 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 On 6/15/06, Antoine Levy-Lambert <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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, [EMAIL PROTECTED] <[EMAIL PROTECTED]> 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: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]