Was I understanding correctly from an earlier e-mail that:
<delete>
<fileset>
<includes name="${the.file.I.want.to.delete}" />
<includes name="${or.the.whole.tree.I.want.to.delete}"
/>
<includes name="${some.file.that.does.not.exist}" />
</fileset>
</delete>
Would accomplish the same thing as this proposed patch?
My reasons for being -1 on this:
1) A user who previously intended his builds to fail because of the
absence of a file or directory when trying to delete it would now have
lost the ability to accomplish that, with the only workarounds being
not-yet-existing features.
2) A user that wants to accomplish what Gert suggests can do like the
above.
3) In spite of previous comments, when I turn my English-to-Geek
translator loose on this thread, it still keeps popping up "Valid; adj.:
Just like Ant" and "Invalid; adj.: Different than Ant"
4) Metaphors about honey, vinegar, and catching flies.
In short, I'm liking the thoughts on where this feature is going, but
I'm not wild about the "hurry up and commit this!" vibe when there seems
to be a decided lack of consensus. Personally, I'd rather see one of
the proposed "available" "exists" or the embedded function stuff added
-prior-, so that we don't end up with an interim net loss on
functionality.
Of course, if I'm speaking ex-rectum regarding the fileset and delete
interaction, then my argument is reduced to: "D00d! Chill!"
:)
Brian
-----Original Message-----
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED]] On Behalf Of
[EMAIL PROTECTED]
Sent: Thursday, February 20, 2003 6:59 AM
To: Ian MacLean; [EMAIL PROTECTED]
Cc: Simon Steele; Nant-Developers
Subject: Re: Re: [nant-dev] Delete task behaviour
Hi Ian,
When should I log a diagnotic message if the file/Directory does not
exist ? Ony when the task runs in verbose mode, if the project runs in
verbose mode or always ?
I would prefer not to log it always, but I'll do whatever you decide ...
What do we do with the path length check that's in the DeleteTask class
? I'd say, remove it, but I guess my ability to judge on these kind of
things is way off today :-)
Gert
By the way: are you sure you won't reconsider the decision to remove the
quiet attribute ? :-)
------------------------
Ian MacLean <[EMAIL PROTECTED]> wrote:
------------------------
>[EMAIL PROTECTED] wrote:
>
>>
>> If I understand you correctly, you want me to remove the quiet
attribute and always output a message in the log when you try to delete
a file or directory that doesn't exist, right ?
>
>yeah for now. I don't have a problem with the concept but I think we
>should think about having it in the base class. We can always whack it
>back in later.
>
>
>> I don't really agree with your remarks, but we live in a democracy ..
>>
>> So if I update the patch, you'll commit it, right ?
>>
>
>Yep. Although Gerry may still have issues with me :)
>
>Ian
>
>>>Simon,
>>>
>>>>>I assume you are talking about the quiet attribute. I'm wondering
how
>
>
>
>
>
>-------------------------------------------------------
>This SF.net email is sponsored by: SlickEdit Inc. Develop an edge.
>The most comprehensive and flexible code editor you can use.
>Code faster. C/C++, C#, Java, HTML, XML, many more. FREE 30-Day Trial.
>www.slickedit.com/sourceforge
>_______________________________________________
>Nant-developers mailing list
>[EMAIL PROTECTED]
>https://lists.sourceforge.net/lists/listinfo/nant-developers
>
-------------------------------------------------------
This SF.net email is sponsored by: SlickEdit Inc. Develop an edge.
The most comprehensive and flexible code editor you can use.
Code faster. C/C++, C#, Java, HTML, XML, many more. FREE 30-Day Trial.
www.slickedit.com/sourceforge
_______________________________________________
Nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers
-------------------------------------------------------
This SF.net email is sponsored by: SlickEdit Inc. Develop an edge.
The most comprehensive and flexible code editor you can use.
Code faster. C/C++, C#, Java, HTML, XML, many more. FREE 30-Day Trial.
www.slickedit.com/sourceforge
_______________________________________________
Nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers
