Re: Building GNU Classpath on Fedora 17

Pekka Enberg <[email protected]> Tue, 12 Mar 2013 15:47:17 +0200
Newsgroups gmane.comp.java.classpath.devel
Message-ID <CAOJsxLHrnR8hL43yVhvaueeG=wzv_pbOsXfHZ8hFX9GOyxurRg@mail.gmail.com>
On Tue, Mar 12, 2013 at 12:47 PM, Andrew Hughes <[email protected]> wrote:
> Looking at rpm -ql gettext-devel on my RHEL system, it seems to add a slew
> of m4 files to /usr/share/aclocal.  Maybe if -e /usr/share/aclocal/gettext.m4
> in autogen.sh would be sufficient?

The attached patch works for me. Does it look OK to commit?
0001-Check-for-gettext-in-autogen.sh.patch (application/octet-stream, 1.1 KB)
From 1d3e3929e46fc37e01d87f9b4c1e49c32019d19b Mon Sep 17 00:00:00 2001
From: Pekka Enberg <[email protected]>
Date: Tue, 12 Mar 2013 15:41:54 +0200
Subject: [RFC/PATCH] Check for gettext in autogen.sh

Cc: Andrew John Hughes  <[email protected]>
Cc: Brian Jones <[email protected]>
Signed-off-by: Pekka Enberg <[email protected]>
---
 ChangeLog  |  5 +++++
 autogen.sh | 10 ++++++++++
 2 files changed, 15 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index 289a979..68fd4ca 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2013-03-12  Pekka Enberg <[email protected]>
+
+	* autogen.sh:
+	Check for gettext.
+
 2013-03-09  Pekka Enberg <[email protected]>
 
 	* .gitignore: Exclude autogen-generated files.
diff --git a/autogen.sh b/autogen.sh
index adb8f0c..6c39cc9 100755
--- a/autogen.sh
+++ b/autogen.sh
@@ -34,6 +34,16 @@ if $have_libtool ; then : ; else
 	DIE=1
 fi
 
+have_gettext=false
+if [ -e /usr/share/aclocal/gettext.m4 ] ; then
+	have_gettext=true
+fi
+if $have_gettext ; then : ; else
+	echo
+	echo "You must have gettext installed to compile $PROJECT."
+	DIE=1
+fi
+
 if test "$DIE" -eq 1; then
 	exit 1
 fi
-- 
1.7.11.7