Bugs item #1405061, was opened at 2006-01-13 19:23
Message generated for change (Comment added) made by drieseng
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=402868&aid=1405061&group_id=31650

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Tasks
Group: 0.85
Status: Open
Resolution: None
Priority: 5
Submitted By: Jordan Samuels (jordansamuels)
Assigned to: Nobody/Anonymous (nobody)
Summary: FileSet and DirSet  / BuildElement initialization broken

Initial Comment:
=======
The bug
=======

FileSet and DirSet initialization appears to depend on
the path to the nant file itself, and not the current
working directory.  In particular, if the nant file is
in a different directory, and FileSet and DirSet
elements are specified in the nant script using
relative paths, then they are incorrectly initialized
based on the relative path of the nant file, not the
current working directory.

Due to the (bug/feature?) that FileSet and DirSet
creation silently masks file-not-found errors, it's
virtually impossible to determine the cause of empty
file/dir sets without intimate knowledge of this
initialization flaw.

============
The platform
============

Windows XP Professional, NAnt 0.85 (Build 0.85.1932.0;
rc3; 4/16/2005).

Reproduction

The attached file "FileAndDirTask.cs" defines a
subclass of Nant.Core.Task task in C#.  This task
contains BuildElement's FileSet and DirSet, and it
display the size of each set based on the initialized
performed automatically by nant.

The nant file used for testing is:

<?xml version='1.0'?>
<project name="Bug">
  <target name='test'>
    <loadtasks assembly='C:\FileAndDir.dll' /> 
    <FileAndDir>
      <Files>
        <include name='file1.txt' />  
      </Files>
      <Dirs>
        <include name='dir1' />  
      </Dirs>
    </FileAndDir>
  </target>
</project>

If file1.txt and dir1 are in the current directory, and
I run nant like this:

  nant -buildfile:FileAndDir.nant test

The task outputs "I have 1 files and 1 dirs ."

If, however, I move the nant file up one directory, and
run nant like this:

  nant -buildfile:..\FileAndDir.nant test

The task outputs "I have 0 files and 0 dirs ."









----------------------------------------------------------------------

>Comment By: Gert Driesen (drieseng)
Date: 2006-01-17 15:12

Message:
Logged In: YES 
user_id=707851

Jordan:

I don't think it's likely that we'll change the way 
<fileset> works. We always resolve relative path against 
the project directory, not the current directory.

Modifying this behaviour would be a major breaking change.

Matthew:

I agree that a mechanism to explictly specify a set of 
files could be very useful.

However, I don't think modifying <fileset> is the right 
thing to do. We might be better off introducing a 
<filelist> type that does not do pattern matching. 
The "mustexist" behaviour of a <filelist> would then be 
determined by the task in which it's used. I think think 
we need an attribute for this purpose.

----------------------------------------------------------------------

Comment By: Jordan Samuels (jordansamuels)
Date: 2006-01-17 14:47

Message:
Logged In: YES 
user_id=1426721

I agree re: the "asis" / "mustexist" issue.  However, this
is relatively minor compared to the violation of the
"Principle of Least Astonishment" when the path to the nant
file pollutes these file patterns.

----------------------------------------------------------------------

Comment By: Matthew Mastracci (mmastrac)
Date: 2006-01-16 18:27

Message:
Logged In: YES 
user_id=85368

NAnt currently treats all entries without the "asis" flag as
patterns.  Since a pattern can match zero or more items, it
always succeeds, even if the file doesn't exist.

I just got bit with this using the <touch> task.  Perhaps we
should add a "mustexist" flag to the entries.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=402868&aid=1405061&group_id=31650


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
nant-developers mailing list
nant-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nant-developers

Reply via email to