Re: [PATCH] configure.ac: check for org.junit.rules.ExternalResource

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

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


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


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux