automake-patches
[Top][All Lists]
Advanced

[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: Tue, 11 Mar 2008 22:29:46 +0100
User-agent: Mutt/1.5.17+20080114 (2008-01-14)

Hello Peter,

* Peter Breitenlohner wrote on Mon, Mar 10, 2008 at 01:26:08PM CET:
>
> [...] 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.

Ah yes, sure.

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

Yes, thank you, also for the detailed comments.  I've applied your
changes in 2 patches (the reindent, plus the actual change set), see
<http://git.savannah.gnu.org/gitweb/?p=automake.git;a=commit;h=f2eadff64fb4b3512b9ede4055cb9defb4b67dbe>
and its parent, with a minor change to your patch as below.  I also
added a NEWS entry.

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

Having one test is fine, and it looks good, too.

I suppose we might want to change the MANS line and the install-manX
rules to be flattened, they already look pretty ugly in the Makefile.
But that can happen in a followup patch.

Cheers, and thanks again!
Ralf

> --- automake-1.10.1.orig/doc/automake.texi    2008-01-21 23:41:02.000000000 
> +0100
> +++ automake-1.10.1/doc/automake.texi 2008-03-08 01:15:41.000000000 +0100

> +Executables and manpages may be renamed upon installation
> +(@pxref{Renaming}).  For manpages this can be avoided by use of the
> address@hidden prefix.  For instance, suppose an executable @samp{foo}
> +allowing to access a library function @samp{foo()} from the command line.

The GCS specify to use function names without parentheses, so I removed
them after foo.




reply via email to

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