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 > [EMAIL PROTECTED] 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.
