Re: Dealing with build attributes of targets

Adriaan de Groot <adridg-FlD2LfDziEhmR6Xm/[email protected]>
Newsgroups gmane.comp.tools.aap.devel
Message-ID <[email protected]>
On Thursday 23 October 2003 14:54, you wrote:
> Adriaan de Groot wrote:
> > :ltlib { attributes for ltlib, like installvar } target { attributes for
> > : the
> > target, like installdir } : { attributes XXX } sources { attributes for
> > each source individually }
>
> I agree that the attributes that come right after the ":" should apply
> to all the sources that come after it.  It's logical.

Yay. It _is_, however, a nasty change if there's 3rd parties using AAP and 
relying on the current behavior.


> I think we should check that this is consistent with other commands, and
> also for dependencies.  (problem with dependencies is that you can't put
> an attribute at the start: "{attr} target : source", thus it uses
> "target : {attr} source")

Well, there's also target { attr } : source available, which makes the most 
sense to me for attaching attr to target. If an attr is first in a list of 
things, it applies to all of them. Otherwise, it applies to the thing just 
before it.

> Besides the missing documentation there is another problem: The

<hunchback>Yesss, masster. I will go fetcchh the braaaiins now.</hunckback>


> attributes should apply to the sources for this build command, but when
> the source is also used elswhere, the attribute do not necessarily
> apply.  Thus adding the attributes to the node is not a good idea.
> The attributes should be passed to add_buildrule() and added to the
> generated dependencies there.

OK. That is more advanced AAP hackery than I can muster quickly, I think. 
Perhaps you can give it a nudge?



-------------------------------------------------------
This SF.net email is sponsored by: The SF.net Donation Program.
Do you like what SourceForge.net is doing for the Open
Source Community?  Make a contribution, and help us add new
features and functionality. Click here: http://sourceforge.net/donate/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.