[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Implementing notrans_man_MANS

From: Peter Breitenlohner
Subject: Re: Implementing notrans_man_MANS
Date: Mon, 10 Mar 2008 13:26:08 +0100 (CET)

On Thu, 6 Mar 2008, Ralf Wildenhues wrote:

* Peter Breitenlohner wrote on Thu, Mar 06, 2008 at 01:27:05PM CET:
or still something else. If so please indicate what.

Oh sorry, I was in a hurry and not thinking.  What I meant was that
while you can't wrap arbitrary lines ending in backslash-newline in `if
%?NOTRANS_MANS%/else/endif', you can do so at a finer grainage than
whole rules: it's just that it won't work after lines ending in

To some extent this is independent of the question you ask about the
rule target names: whether you interleave trans and notrans in the .am
file and whether you give trans and notrans different target names need
not be dealt with at the same time.  So please for now leave you patch
as is, but only change all those NOTR/DOTR prefixes to use if/else/endif
(which includes a few lines of duplication in the .am file).

Clearer now?  If not, then I guess I can also patch that later.

Hi Ralf,

yes, thanks. However, I can only use if/endif if/endif and not if/else/endif
because a section may have trans_ as well as notrans_ man pages.


Attached are revised versions of the three patches. I hope they take care of
all your earlier comments.


The new variables names are more or less self explanatory (I hope).

I had to use
      @unsorted_deps = (keys %notrans_vars, keys %trans_vars,
                        keys %notrans_this_sect, keys %trans_this_sect);
      my @deps = sort @unsorted_deps;
instead of
      my @deps = sort (keys %notrans_vars, keys %trans_vars,
                       keys %notrans_this_sect, keys %trans_this_sect);
because this may give a perl warning [sort (...) used as function],
resulting in three test (instsh.tes, location.test, warnopts.test) to FAIL.


I have slightly reworded the text preceding the example, and hope this
clarifies the intent.


notrans.test now contains all combinations. I have incorporated the checks
that used to be in man4.test into notrans.test, because otherwise there
would have been much repetition.  Hope this is OK.  If so, remember not to
mention man4.test in the ChangLog.  If not, I could make that two separate

The install-man depedndencies are checked either directly (inspecting or indirectly (letting make generate the files).


Peter Breitenlohner <address@hidden>

Attachment: patch-03-notrans_MANS-impl
Description: Text document

Attachment: patch-04-notrans_MANS-doc
Description: Text document

Attachment: patch-05-notrans_MANS-test
Description: Text document

reply via email to

[Prev in Thread] Current Thread [Next in Thread]