You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by Erik Meade <em...@geekfarm.org> on 2000/11/10 04:21:51 UTC

RE: [PATCH] UnknownElement

I'm reposting this as it didn't get commited, folks still ask
about it.

> -----Original Message-----
> From: Erik Meade [mailto:emeade@geekfarm.org]
> Sent: Wednesday, October 25, 2000 6:12 PM
> To: ant-dev@jakarta.apache.org
> Subject: RE: [PATCH] UnknownElement
>
>
> Now reads: if this is an optional task put the optional.jar in the
> lib directory of your ant installation (ANT_HOME)
>
> > -----Original Message-----
> > From: Conor MacNeill [mailto:conor@m64.com]
> > Sent: Wednesday, October 25, 2000 5:42 AM
> > To: ant-dev@jakarta.apache.org; emeade@geekfarm.org
> > Subject: RE: [PATCH] UnknownElement
> >
> >
> > Erik,
> >
> > Perhaps it would be better to say "put optional.jar in the lib directory
> > of your ant installation (ANT_HOME)"
> > What do you think?
> >
> > Conor
> >
> > > -----Original Message-----
> > > From: Erik Meade [mailto:emeade@geekfarm.org]
> > > Sent: Wednesday, 25 October 2000 15:13
> > > To: ant-dev@jakarta.apache.org
> > > Subject: RE: [PATCH] UnknownElement
> > >
> > >
> > > Added in putting option.jar in your classpath if it is an
> > > optional task.
> > >
> > > > -----Original Message-----
> > > > From: Erik Meade [mailto:emeade@geekfarm.org]
> > > > Sent: Thursday, October 12, 2000 7:31 PM
> > > > To: ant-dev@jakarta.apache.org
> > > > Subject: RE: [PATCH] UnknownElement
> > > >
> > > >
> > > > Added in Nico's suggestion.
> > > > Erik
> > > >
> > > > > -----Original Message-----
> > > > > From: Erik Meade
> > > > > Sent: Thursday, October 12, 2000 7:10 AM
> > > > > To: ant-dev@jakarta.apache.org
> > > > > Subject: RE: [PATCH] UnknownElement
> > > > >
> > > > >
> > > > > Probably, at the risk of appearing uninformed... what is
> > > > > involved with using taskdef to declare you task?  I've
> > > > > created new tasks, so my guess is I've done this, but
> > > > > I'm not sure I know what I did...  I'll send another
> > > > > patch tonight or tomorrow and add it in above "add the
> > > > > task to the defaults.properties" as suggested by Stefan.
> > > > >
> > > > > Erik Meade
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Nico Seessle
> > > > > > Sent: Thursday, October 12, 2000 12:35 AM
> > > > > > To: ant-dev@jakarta.apache.org
> > > > > > Subject: Re: [PATCH] UnknownElement
> > > > > >
> > > > > >
> > > > > > ----- Original Message -----
> > > > > > From: "Erik Meade"
> > > > > > To: <an...@jakarta.apache.org>
> > > > > > Sent: Thursday, October 12, 2000 6:59 AM
> > > > > > Subject: [PATCH] UnknownElement
> > > > > >
> > > > > >
> > > > > > > I thought it may be a good idea to list some solutions with
> > > > > > > the maybeConfigure build exception.
> > > > > > >
> > > > > > > Erik Meade
> > > > > >
> > > > > > "1 execute bin/bootstrap.
> > > > > >  2 add the task to the defaults.properties."
> > > > > >
> > > > > > Isn't this missing "3 use taskdef to declare your task?"
> > > > > >
> > > > > > Nico
> > > > > >

RE: [PATCH] UnknownElement

Posted by Erik Meade <em...@geekfarm.org>.
> -----Original Message-----
> From: Conor MacNeill [mailto:conor@ebinteractive.com.au]
> Sent: Thursday, November 09, 2000 7:55 PM
> To: ant-dev@jakarta.apache.org; emeade@geekfarm.org
> Subject: RE: [PATCH] UnknownElement
>
>
> Erik,
>
> Thanks for re-sending that. I did apply this change locally but I haven't
> committed yet for two reasons:
>
> 1. The message is now getting pretty long and I thought perhaps
> it should be
> formatted rather than one huge line

Cool, I wanted to do that, but wasn't sure how it would go over.  Do you
think \n or \n\t is enough?

> 2. I am not sure of the target audience of this message. If we
> are targeting
> users who download a binary distribution, then telling them to
> bootstrap/build is not that cool since those scripts are not even included
> in the binary distribution.

Good point, one thing I was thinking of doing was checking to see if the
task was optional, but the list where valid tasks are storied doesn't
differenciate between optional and built in.  I was also thinking I could
parse the properties file, but I wasn't sure how that would go over either.

For the bootstrap we could use ANT_HOME and see if bootstrap is in there...
that assumes developers don't move it though, then bin users wouldn't even
see it.

> It is not a big deal, but I thought we should just think about
> who we expect
> to get this message and under what circumstances. What do you think?
>
> --
> Conor MacNeill
> conor@cortexebusiness.com.au
> Cortex eBusiness
> http://www.cortexebusiness.com.au
>
> > -----Original Message-----
> > From: Erik Meade [mailto:emeade@geekfarm.org]
> > Sent: Friday, 10 November 2000 14:22
> > To: ant-dev@jakarta.apache.org
> > Subject: RE: [PATCH] UnknownElement
> >
> >
> > I'm reposting this as it didn't get commited, folks still ask
> > about it.
> >
> > > -----Original Message-----
> > > From: Erik Meade [mailto:emeade@geekfarm.org]
> > > Sent: Wednesday, October 25, 2000 6:12 PM
> > > To: ant-dev@jakarta.apache.org
> > > Subject: RE: [PATCH] UnknownElement
> > >
> > >
> > > Now reads: if this is an optional task put the optional.jar in the
> > > lib directory of your ant installation (ANT_HOME)
> > >
> > > > -----Original Message-----
> > > > From: Conor MacNeill [mailto:conor@m64.com]
> > > > Sent: Wednesday, October 25, 2000 5:42 AM
> > > > To: ant-dev@jakarta.apache.org; emeade@geekfarm.org
> > > > Subject: RE: [PATCH] UnknownElement
> > > >
> > > >
> > > > Erik,
> > > >
> > > > Perhaps it would be better to say "put optional.jar in the
> > lib directory
> > > > of your ant installation (ANT_HOME)"
> > > > What do you think?
> > > >
> > > > Conor
> > > >
> > > > > -----Original Message-----
> > > > > From: Erik Meade [mailto:emeade@geekfarm.org]
> > > > > Sent: Wednesday, 25 October 2000 15:13
> > > > > To: ant-dev@jakarta.apache.org
> > > > > Subject: RE: [PATCH] UnknownElement
> > > > >
> > > > >
> > > > > Added in putting option.jar in your classpath if it is an
> > > > > optional task.
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Erik Meade [mailto:emeade@geekfarm.org]
> > > > > > Sent: Thursday, October 12, 2000 7:31 PM
> > > > > > To: ant-dev@jakarta.apache.org
> > > > > > Subject: RE: [PATCH] UnknownElement
> > > > > >
> > > > > >
> > > > > > Added in Nico's suggestion.
> > > > > > Erik
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: Erik Meade
> > > > > > > Sent: Thursday, October 12, 2000 7:10 AM
> > > > > > > To: ant-dev@jakarta.apache.org
> > > > > > > Subject: RE: [PATCH] UnknownElement
> > > > > > >
> > > > > > >
> > > > > > > Probably, at the risk of appearing uninformed... what is
> > > > > > > involved with using taskdef to declare you task?  I've
> > > > > > > created new tasks, so my guess is I've done this, but
> > > > > > > I'm not sure I know what I did...  I'll send another
> > > > > > > patch tonight or tomorrow and add it in above "add the
> > > > > > > task to the defaults.properties" as suggested by Stefan.
> > > > > > >
> > > > > > > Erik Meade
> > > > > > >
> > > > > > > > -----Original Message-----
> > > > > > > > From: Nico Seessle
> > > > > > > > Sent: Thursday, October 12, 2000 12:35 AM
> > > > > > > > To: ant-dev@jakarta.apache.org
> > > > > > > > Subject: Re: [PATCH] UnknownElement
> > > > > > > >
> > > > > > > >
> > > > > > > > ----- Original Message -----
> > > > > > > > From: "Erik Meade"
> > > > > > > > To: <an...@jakarta.apache.org>
> > > > > > > > Sent: Thursday, October 12, 2000 6:59 AM
> > > > > > > > Subject: [PATCH] UnknownElement
> > > > > > > >
> > > > > > > >
> > > > > > > > > I thought it may be a good idea to list some
> solutions with
> > > > > > > > > the maybeConfigure build exception.
> > > > > > > > >
> > > > > > > > > Erik Meade
> > > > > > > >
> > > > > > > > "1 execute bin/bootstrap.
> > > > > > > >  2 add the task to the defaults.properties."
> > > > > > > >
> > > > > > > > Isn't this missing "3 use taskdef to declare your task?"
> > > > > > > >
> > > > > > > > Nico
> > > > > > > >
> >


RE: [PATCH] UnknownElement

Posted by Conor MacNeill <co...@ebinteractive.com.au>.
Erik,

Thanks for re-sending that. I did apply this change locally but I haven't
committed yet for two reasons:

1. The message is now getting pretty long and I thought perhaps it should be
formatted rather than one huge line

2. I am not sure of the target audience of this message. If we are targeting
users who download a binary distribution, then telling them to
bootstrap/build is not that cool since those scripts are not even included
in the binary distribution.

It is not a big deal, but I thought we should just think about who we expect
to get this message and under what circumstances. What do you think?

--
Conor MacNeill
conor@cortexebusiness.com.au
Cortex eBusiness
http://www.cortexebusiness.com.au

> -----Original Message-----
> From: Erik Meade [mailto:emeade@geekfarm.org]
> Sent: Friday, 10 November 2000 14:22
> To: ant-dev@jakarta.apache.org
> Subject: RE: [PATCH] UnknownElement
>
>
> I'm reposting this as it didn't get commited, folks still ask
> about it.
>
> > -----Original Message-----
> > From: Erik Meade [mailto:emeade@geekfarm.org]
> > Sent: Wednesday, October 25, 2000 6:12 PM
> > To: ant-dev@jakarta.apache.org
> > Subject: RE: [PATCH] UnknownElement
> >
> >
> > Now reads: if this is an optional task put the optional.jar in the
> > lib directory of your ant installation (ANT_HOME)
> >
> > > -----Original Message-----
> > > From: Conor MacNeill [mailto:conor@m64.com]
> > > Sent: Wednesday, October 25, 2000 5:42 AM
> > > To: ant-dev@jakarta.apache.org; emeade@geekfarm.org
> > > Subject: RE: [PATCH] UnknownElement
> > >
> > >
> > > Erik,
> > >
> > > Perhaps it would be better to say "put optional.jar in the
> lib directory
> > > of your ant installation (ANT_HOME)"
> > > What do you think?
> > >
> > > Conor
> > >
> > > > -----Original Message-----
> > > > From: Erik Meade [mailto:emeade@geekfarm.org]
> > > > Sent: Wednesday, 25 October 2000 15:13
> > > > To: ant-dev@jakarta.apache.org
> > > > Subject: RE: [PATCH] UnknownElement
> > > >
> > > >
> > > > Added in putting option.jar in your classpath if it is an
> > > > optional task.
> > > >
> > > > > -----Original Message-----
> > > > > From: Erik Meade [mailto:emeade@geekfarm.org]
> > > > > Sent: Thursday, October 12, 2000 7:31 PM
> > > > > To: ant-dev@jakarta.apache.org
> > > > > Subject: RE: [PATCH] UnknownElement
> > > > >
> > > > >
> > > > > Added in Nico's suggestion.
> > > > > Erik
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Erik Meade
> > > > > > Sent: Thursday, October 12, 2000 7:10 AM
> > > > > > To: ant-dev@jakarta.apache.org
> > > > > > Subject: RE: [PATCH] UnknownElement
> > > > > >
> > > > > >
> > > > > > Probably, at the risk of appearing uninformed... what is
> > > > > > involved with using taskdef to declare you task?  I've
> > > > > > created new tasks, so my guess is I've done this, but
> > > > > > I'm not sure I know what I did...  I'll send another
> > > > > > patch tonight or tomorrow and add it in above "add the
> > > > > > task to the defaults.properties" as suggested by Stefan.
> > > > > >
> > > > > > Erik Meade
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: Nico Seessle
> > > > > > > Sent: Thursday, October 12, 2000 12:35 AM
> > > > > > > To: ant-dev@jakarta.apache.org
> > > > > > > Subject: Re: [PATCH] UnknownElement
> > > > > > >
> > > > > > >
> > > > > > > ----- Original Message -----
> > > > > > > From: "Erik Meade"
> > > > > > > To: <an...@jakarta.apache.org>
> > > > > > > Sent: Thursday, October 12, 2000 6:59 AM
> > > > > > > Subject: [PATCH] UnknownElement
> > > > > > >
> > > > > > >
> > > > > > > > I thought it may be a good idea to list some solutions with
> > > > > > > > the maybeConfigure build exception.
> > > > > > > >
> > > > > > > > Erik Meade
> > > > > > >
> > > > > > > "1 execute bin/bootstrap.
> > > > > > >  2 add the task to the defaults.properties."
> > > > > > >
> > > > > > > Isn't this missing "3 use taskdef to declare your task?"
> > > > > > >
> > > > > > > Nico
> > > > > > >
>