[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makef
From: |
Werner LEMBERG |
Subject: |
Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil... |
Date: |
Tue, 16 Apr 2013 07:14:24 +0200 (CEST) |
Bernd,
this is still not OK.
> RCS file: /cvsroot/groff/groff/INSTALL.gen,v
> retrieving revision 1.7
> retrieving revision 1.8
> diff -u -b -r1.7 -r1.8
*Never* change this file manually! It is a generic one provided from
GNU (normally called `INSTALL'). Please restore it (or get the newest
version from the `gnulib' or `automake' git repository).
> RCS file: /cvsroot/groff/groff/aclocal.m4,v
> [...]
> +# ===========================================================================
> +# http://www.gnu.org/software/autoconf-archive/ax_prog_perl_version.html
> +# ===========================================================================
> [...]
As told earlier, such stuff shouldn't be in this file; there should
only be lines of the form
m4_include([m4/foo.m4])
The reason for this problem is that you are using the macro
AX_PROG_PERL_VERSION (in `groff.m4') without adding the file
`ax_prog_perl_version.m4' to the repository.
> RCS file: INSTALL.autotools
I think this is the wrong file name; I would rather call it
`INSTALL.CVS'. And it makes only sense to have it in case the
`configure' script and all other generated files are removed from the
CVS.
Two other minor issues:
. file `groff.m4':
@@ -134,6 +148,7 @@
AC_MSG_WARN([missing program$plural:
The program$plural
+
$missing
cannot be found in the PATH.
Consequently, groff's HTML backend (grohtml) will not work properly;
Why the additional newline? If you do so, I would rather expect to
have another one after the `$missing' line also.
. file `Makefile.in':
All variables which are defined globally should be added to the
`MDEFINES' definition. You haven't done so for `PERLVERSION'.
Werner
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil...,
Werner LEMBERG <=
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Bernd Warken, 2013/04/16
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Werner LEMBERG, 2013/04/17
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Ralph Corderoy, 2013/04/18
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Werner LEMBERG, 2013/04/18
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Keith Marshall, 2013/04/18
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Ralph Corderoy, 2013/04/18
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Heinz-Jürgen Oertel, 2013/04/18
- Re: [Groff] [Groff-commit] groff ChangeLog INSTALL.gen Makefile.in Makefil..., Ralph Corderoy, 2013/04/18