I'm sorry I'm not a good one to review this. They changed the auto* tools quite a bit since I used to mess with them. Is autogen.sh something auto* creates, just wondering if it is okay to modify...
On Tue, Mar 12, 2013 at 2:47 PM, Pekka Enberg <penb...@kernel.org> wrote: > If gettext-devel package is not installed on Fedora, autogen.sh fails as > follows: > > [penberg@tux classpath]$ sh autogen.sh > configure.ac:505: warning: AC_LIB_PREPARE_PREFIX is m4_require'd but not > m4_defun'd > m4/iconv.m4:11: AM_ICONV_LINKFLAGS_BODY is expanded from... > m4/iconv.m4:22: AM_ICONV_LINK is expanded from... > m4/iconv.m4:77: AM_ICONV is expanded from... > > Make the script more user fiendly by explicitly checking for the > presence of gettext.m4 in the system. > > Cc: Andrew John Hughes <gnu_and...@member.fsf.org> > Cc: Brian Jones <cbjon...@gmail.com> > Signed-off-by: Pekka Enberg <penb...@kernel.org> > --- > ChangeLog | 5 +++++ > autogen.sh | 7 +++++++ > 2 files changed, 12 insertions(+), 0 deletions(-) > > diff --git a/ChangeLog b/ChangeLog > index 289a979..e26f627 100644 > --- a/ChangeLog > +++ b/ChangeLog > @@ -1,3 +1,8 @@ > +2013-03-12 Pekka Enberg <penb...@kernel.org> > + > + * autogen.sh: > + Check that gettext.m4 is installed. > + > 2013-03-09 Pekka Enberg <penb...@kernel.org> > > * .gitignore: Exclude autogen-generated files. > diff --git a/autogen.sh b/autogen.sh > index adb8f0c..df0095f 100755 > --- a/autogen.sh > +++ b/autogen.sh > @@ -34,6 +34,13 @@ if $have_libtool ; then : ; else > DIE=1 > fi > > +if [ ! -e $(aclocal --print-ac-dir)/gettext.m4 ] ; then > + echo > + echo "You must have gettext package and, if applicable to your" > + echo "system, gettext-devel package installed to compile $PROJECT." > + DIE=1 > +fi > + > if test "$DIE" -eq 1; then > exit 1 > fi > -- > 1.7.7.6 >
