Re: Restricted types: Re: Location in non-Task tasks

2006-09-13 Thread Stefan Bodewig
On Tue, 12 Sep 2006, Peter Reilly <[EMAIL PROTECTED]> wrote: > Ok, I dug out my old code and after digging out some of the bugs > and misunderstanding, I have modded IH, CH (componenthelper), > and to allow "restricted" types. > > These are like normal types except: > 1) they are only allowed

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Peter Reilly
On 9/13/06, Peter Reilly <[EMAIL PROTECTED]> wrote: Some classes thus may have multiple names associated with them, a unique top-level name and a restricted name that can be s Opps, I hit a emacs keystroke that sent the email... to continue: Some classes thus may have multiple names assoc

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Dominique Devienne
(just kidding). Score:5, Funny ;-) For me: sounds the best. Do not forget that this will probally not be used outside the ant core code as the resultant name may not be used at the top-level, or in targets - or sequential bodies. I don't see why it would be limited to ant core. Any antlib

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Peter Reilly
On 9/13/06, Matt Benson <[EMAIL PROTECTED]> wrote: --- Dominique Devienne <[EMAIL PROTECTED]> wrote: > > I did not see the changes. But in principle I > agree with Dominique > > that a roledef should have 3 attributes : > > name, role, classname > > Just to clarify, I like the way Peter impleme

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Dominique Devienne
Am I correct in assuming we'd have, with whatever name: I think so. Peter can correct me if I'm wrong. What about ? Then what about or ? I don't like them ;-))) --DD - To unsubscribe, e-mail: [EMAIL PROTECTED] For additi

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Peter Reilly
Thanks, this is good summing up. On 9/13/06, Dominique Devienne <[EMAIL PROTECTED]> wrote: > Sorry I did not read all the svn commit emails. Which classes implement these role definitions ? None, at least none that "define" a role. Peter has extended IH and co. to lookup nested-element in tas

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Matt Benson
--- Dominique Devienne <[EMAIL PROTECTED]> wrote: > > I did not see the changes. But in principle I > agree with Dominique > > that a roledef should have 3 attributes : > > name, role, classname > > Just to clarify, I like the way Peter implemented it > (binding a class > to a name, without speci

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Dominique Devienne
Sorry I did not read all the svn commit emails. Which classes implement these role definitions ? None, at least none that "define" a role. Peter has extended IH and co. to lookup nested-element in tasks/types that have add(X)-type methods, in a new mapping he called restricted types (because t

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Antoine Levy-Lambert
Original-Nachricht Datum: Wed, 13 Sep 2006 12:14:07 +0100 Von: "Peter Reilly" <[EMAIL PROTECTED]> An: "Ant Developers List" Betreff: Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks] > I do not like the idea of a role. > >

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Dominique Devienne
I did not see the changes. But in principle I agree with Dominique that a roledef should have 3 attributes : name, role, classname Just to clarify, I like the way Peter implemented it (binding a class to a name, without specifying what "role"s this class should play). But the term "role" also im

Re: Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Peter Reilly
On 9/13/06, Antoine Levy-Lambert <[EMAIL PROTECTED]> wrote: Hi, I did not see the changes. But in principle I agree with Dominique that a roledef should have 3 attributes : name, role, classname or can we make the role attribute optional by finding out by introspection whether a class is a cond

Roles [WAS:Re: Restricted types: Re: Location in non-Task tasks]

2006-09-13 Thread Antoine Levy-Lambert
Hi, I did not see the changes. But in principle I agree with Dominique that a roledef should have 3 attributes : name, role, classname or can we make the role attribute optional by finding out by introspection whether a class is a condition, or a selector, or a ... In any case we should have

Re: Restricted types: Re: Location in non-Task tasks

2006-09-12 Thread Dominique Devienne
> > however we can add the @ant.type tags now for documentation?, > > Sure, provided we settle on a doc tag name, which ideally would match > the <*def> we choose. --DD @ant.type --> @ant.task --> @ant.role --> Role implies (to me at least) that the name is bound to a given role, like bein

Re: Restricted types: Re: Location in non-Task tasks

2006-09-12 Thread Peter Reilly
On 9/12/06, Dominique Devienne <[EMAIL PROTECTED]> wrote: > > > > > 2) Introduce a or for the purpose > > > > > of locating extension points as nested elements. > Ok, I dug out my old code and after digging out some of the bugs > and misunderstanding, I have modded IH, CH (componenthelper), >

Re: Restricted types: Re: Location in non-Task tasks

2006-09-12 Thread Dominique Devienne
> > > > 2) Introduce a or for the purpose > > > > of locating extension points as nested elements. Ok, I dug out my old code and after digging out some of the bugs and misunderstanding, I have modded IH, CH (componenthelper), and to allow "restricted" types. Cool!!! I am not too sure th

Restricted types: Re: Location in non-Task tasks

2006-09-12 Thread Peter Reilly
On 9/11/06, Dominique Devienne <[EMAIL PROTECTED]> wrote: > > > 2) Introduce a or for the purpose > > > of locating extension points as nested elements. > but I looked at the code and realized that the type > handling in ant would become too complicated - there I see your point. Increased co