diff mbox

configure.ac: check for org.junit.rules.ExternalResource

Message ID 973BE144-8E8E-4076-97C3-ACB74D84F63A@inktank.com (mailing list archive)
State New, archived
Headers show

Commit Message

Gary Lowell Jan. 14, 2013, 8:41 p.m. UTC
On Jan 12, 2013, at 1:59 PM, Danny Al-Gaaf wrote:

> Am 11.01.2013 06:13, schrieb Gary Lowell:
> [...]
>> Thanks Danny.  Installing sharutils solved that minor issue.  We now
>> get though the build just fine on opensuse 12, but sles 11sp2 gives
>> more warnings (pasted below).  Should we be using a newer version of
>> autoconf  on sles?  I've tried moving AC_CANONICAL_TARGET earlier in
>> the file, but that causes some other issues with the new java
>> macros.
>> 
>> Thanks, Gary
> 
> I'll take a look at it, I guess it's a problem in configure.ac.
> 
> I see the same warnings in our build system at SUSE for SLES (see: e.g.
> logs at
> https://build.opensuse.org/project/monitor?project=home%3Adalgaaf%3Abranches%3Afilesystems),
> but the package builds just fine for openSUSE, SLES, Fedora, RHEL and
> CentOS there

Looks like it is the usage of AC_CANONICAL_SYSTEM in ac_prog_javah.m4 that cause the warning on centos and sles.  Using AC_CANONICAL_HOST fixed the warning.  I'm not sure if that's the real solution, or if something should be changed in the configure.ac file.
The diff of the change I tried is below.

Thanks,
Gary



--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/m4/ac_prog_javah.m4 b/m4/ac_prog_javah.m4
index 0cc93f9..118e02f 100644
--- a/m4/ac_prog_javah.m4
+++ b/m4/ac_prog_javah.m4
@@ -10,7 +10,7 @@  dnl @version 2002-03-25
 dnl @license AllPermissive
 
 AC_DEFUN([AC_PROG_JAVAH],[
-AC_REQUIRE([AC_CANONICAL_SYSTEM])dnl
+AC_REQUIRE([AC_CANONICAL_HOST])dnl
 AC_REQUIRE([AC_PROG_CPP])dnl
 AC_PATH_PROG(JAVAH,javah)
 if test x"`eval 'echo $ac_cv_path_JAVAH'`" != x ; then