> -----Original Message-----
> From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> Sent: Thursday, 9 August 2001 6:19 AM
> To: [EMAIL PROTECTED]
> Subject: cvs commit: jakarta-ant/src/main/org/apache/tools/ant/types
> Description.java
>
>
> sbailliez 01/08/08 13:18:34
>
> Added: docs/manual/CoreTasks description.html
> src/main/org/apache/tools/ant/types Description.java
> Log:
OK, I have some thoughts on this patch. Firstly, let me say that I think a
description element is a good thing. In fact I thought that was how the
target descriptions should have been done (i.e. elements not attributes). My
concern is the fact that description has been implemented as a data type. It
IS convenient to implement it that way, but conceptually it feels wrong. The
description, is not a datatype. The fact that this datatype does not hold
its own data but redirects it straight into the project object is an
indicator of this mismatch, IMHO.
BTW, You can't currently have two <description> elements although I guess we
could decide what that means and potentially concatenate them.
I would have thought this element should have been handled directly in the
ProjectHelper parsing code. It also creates little oddities such as
<project name="test" default="main">
<description>All your buildfile are belong to us</description>
<target name="main">
<description>The main event</description>
</target>
<target name="never">
<description>No GST, never ever</description>
</target>
</project>
Conor
