Re: Implementing notrans_man_MANS

[Peter Breitenlohner]

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
> backslash.
>
> 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.

patch-03-notrans_MANS-impl:

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.

patch-04-notrans_MANS-doc:

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

patch-05-notrans_MANS-test:

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
tests.

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

=========================

regards
Peter Breitenlohner <address@hidden>

patch-03-notrans_MANS-impl
Description: Text document

patch-04-notrans_MANS-doc
Description: Text document

patch-05-notrans_MANS-test
Description: Text document