[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Implementing notrans_man_MANS
From: |
Ralf Wildenhues |
Subject: |
Re: Implementing notrans_man_MANS |
Date: |
Thu, 6 Mar 2008 12:19:22 +0100 |
User-agent: |
Mutt/1.5.17 (2008-02-27) |
Hi Peter,
* Peter Breitenlohner wrote on Thu, Mar 06, 2008 at 10:43:20AM CET:
> On Wed, 5 Mar 2008, Ralf Wildenhues wrote:
>
>> This triple loop is already at 72 iterations. Not good for performance.
>> If this grows further, we may have to access variables in a different
>> manner here.
>
> I assume you are worried by the total number of 72 iterations, not by the
> fact that they come from three nested loops. Given the Automake
> infrastructure (to the extent I understand it) I see no alternative.
Your assumption is correct, and yes, I don't see a simple alternative
either.
>>> +?NOTRY? l2='%NOTRZ%'; \
>>> +?NOTRY? for i in $$l2; do \
>>> .......
>>> +?NOTR? $(INSTALL_DATA) "$$file"
>>> "$(DESTDIR)$(man%SECTION%dir)/$$inst"; \
>>> +?NOTR? done
>>
>> Note to self: should rewrite the above when applying the multi-file
>> install.
>
> What is multi-file install? Maybe I don't need to know.
You don't need to know, but I meant this:
<http://thread.gmane.org/gmane.comp.sysutils.automake.patches/2945/focus=3088>
>> Also, the whole thing would probably be a lot more readable if the
>> ?NOTR?/?DOTR? prefixes vanished and were replaced by full rules, wrapped
>> in `if %?NOTRANS_MANS%'. That would be a better name for NOTR, and
>> since it would then only appear a couple of times, it could be longer
>> without pain, too.
>
> I have tried to wrap the make rule fragments by something like
> `if %?XXX%' but failed, maybe I got the syntax wrong. However, from
> some comments elsewhere I concluded that at present this is not possible.
It is possible but only if you wrap whole rules. You cannot wrap parts.
See depend2.am for examples.
> Do you mean something like:
> install-man: install-man1 install-notrans-man1
> install-man1: trans_DEPENDENCIES
> trans_RULES
> install-notrans-man1: notrans_DEPENDENCIES
> notrans_RULES
No.
>>> address@hidden must be specified first when used in conjunction with
>>> +either @samp{dist_} or @samp{nodist_} (@pxref{Dist}). For instance:
>>
>> Can this limitation be lifted (without making the code much slower)?
>
> I wouldn't know how, and there is an analogous limitation elsewhere:
> nobase_dist_pkgdata_DATA
Ah ok, I wasn't aware of that.
>>> +./configure --program-prefix=gnu- --prefix `pwd`/inst --mandir
>>> `pwd`/inst/man
>>
>> Please quote instances of `pwd' for the master testsuite.
>
> Like --prefix "`pwd`"/inst for Windows paths containing spaces? I just
> copied this from transform.test. A quick grep shows there are still plenty
> of unquoted pwd's around.
Not if you look in git master of Automake.
> BTW: Which one is better: `--prefix PREFIX' or `--prefix=PREFIX'?
Both are accepted.
> In the meantime I have some ideas how to handle manpage translations. Mainly
> an Autoconf/Automake macro and Makefile.am stuff, plus a little bit of help
> from Automake. To whom should I send this, when finished to write it up?
Respective bits to auto{conf,address@hidden, please.
Thanks,
Ralf
- Re: Implementing notrans_man_MANS, Ralf Wildenhues, 2008/03/05
- Re: Implementing notrans_man_MANS, Peter Breitenlohner, 2008/03/06
- Re: Implementing notrans_man_MANS,
Ralf Wildenhues <=
- Re: Implementing notrans_man_MANS, Peter Breitenlohner, 2008/03/06
- Re: Implementing notrans_man_MANS, Ralf Wildenhues, 2008/03/06
- Re: Implementing notrans_man_MANS, Peter Breitenlohner, 2008/03/10
- Re: Implementing notrans_man_MANS, Ralf Wildenhues, 2008/03/11
- Re: Implementing notrans_man_MANS, Peter Breitenlohner, 2008/03/12