xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Don Slutz <dslutz@verizon.com>
To: "Roger Pau Monné" <roger.pau@citrix.com>,
	"Don Slutz" <dslutz@verizon.com>,
	"Ian Campbell" <Ian.Campbell@citrix.com>,
	"Andrew Cooper" <andrew.cooper3@citrix.com>
Cc: xen-devel@lists.xenproject.org,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	Rob Hoes <Rob.Hoes@citrix.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: 4.4.0-rc3 tagged
Date: Mon, 10 Feb 2014 14:23:47 -0500	[thread overview]
Message-ID: <52F92743.8090901@terremark.com> (raw)
In-Reply-To: <52F92181.8010907@citrix.com>

On 02/10/14 13:59, Roger Pau Monné wrote:
> On 10/02/14 19:48, Don Slutz wrote:
>> On 02/10/14 05:59, Roger Pau Monné wrote:
>>> On 10/02/14 10:55, Ian Campbell wrote:
>>>> On Fri, 2014-02-07 at 16:29 +0000, Andrew Cooper wrote:
>>>>> On 07/02/14 16:22, Don Slutz wrote:
>>>>>> On 02/07/14 05:05, Ian Campbell wrote:
>>>>>>> On Thu, 2014-02-06 at 19:15 -0500, Don Slutz wrote:
>>>>>>
[snip]
>>>>> Which, according to google, was introduced in 3.09.4
>>>>>
>>>>> I think the ./configure script needs a min version check.
>>>> Yes, I think so too. Rob, could you advise on a suitable minimum and
>>>> perhaps patch tools/configure.ac and/or m4/ocaml.m4 as necessary.
>>>>
>>>> Also CCing Roger who added the ocaml autoconf stuff.
>>> The Ocaml autoconf stuff was picked from http://forge.ocamlcore.org/.
>>> Here is an untested patch for our configure script to check for the
>>> minimum required OCaml version (3.09.3):
>>>
>>> (remember to re-generate the configure script after applying)
>> Not sure if the older Autoconf (2.68) is at fault, but I get:
>>
>> dcs-xen-54:~/xen/tools>autoconf configure.ac >configure
>> configure.ac:14: error: possibly undefined macro: AS_IF
>>        If this token and others are legitimate, please use m4_pattern_allow.
>>        See the Autoconf documentation.
>> configure.ac:162: error: possibly undefined macro: AC_MSG_ERROR
> On Debian systems you need the autoconf-archive package which provides
> the AX_COMPARE_VERSION macro, and then you also need to run aclocal
> before autoconf.
>
> Roger.
>

The CentOS 5.10 system has a too old autoconf to do the rebuild, So I went to a Fedora 17 system.

It does have autoconf-archive package, nut that make no difference:


Installed:
   autoconf-archive.noarch 0:2012.09.08-1.fc17

Complete!
dcs-xen-54:~/xen/tools>autoconf configure.ac >configure
configure.ac:14: error: possibly undefined macro: AS_IF
       If this token and others are legitimate, please use m4_pattern_allow.
       See the Autoconf documentation.
configure.ac:162: error: possibly undefined macro: AC_MSG_ERROR


And the same bad code is still generated:


     if test "x$OCAMLC" = "xno" || test "x$OCAMLFIND" = "xno"; then :

         if test "x$enable_ocamltools" = "xyes"; then :

             as_fn_error $? "Ocaml tools enabled, but unable to find Ocaml" "$LINENO" 5
fi
         ocamltools="n"

else

         AX_COMPARE_VERSION($OCAMLVERSION, lt, 3.09.4,
             AS_IF([test "x$enable_ocamltools" = "xyes"], [
                 AC_MSG_ERROR([Your version of OCaml: $OCAMLVERSION is not supported])])
             ocamltools="n"
         )

fi



    -Don Slutz




_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2014-02-10 19:23 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-31 12:14 4.4.0-rc3 tagged Ian Jackson
2014-01-31 23:07 ` Don Slutz
2014-02-03 11:48   ` Ian Jackson
2014-02-03 12:01     ` Stefano Stabellini
2014-02-07  0:15       ` Don Slutz
2014-02-07 10:05         ` Ian Campbell
2014-02-07 16:22           ` Don Slutz
2014-02-07 16:29             ` Andrew Cooper
2014-02-10  9:55               ` Ian Campbell
2014-02-10 10:59                 ` Roger Pau Monné
2014-02-10 18:48                   ` Don Slutz
2014-02-10 18:59                     ` Roger Pau Monné
2014-02-10 19:23                       ` Don Slutz [this message]
2014-02-10 19:33                         ` Roger Pau Monné
2014-02-10 23:53                           ` Don Slutz
2014-02-11  9:41                             ` Ian Campbell
2014-02-11  9:56                               ` Roger Pau Monné
2014-02-11 10:00                                 ` Ian Campbell
2014-02-11 10:12                                   ` Roger Pau Monné
2014-02-11 10:14                                     ` Ian Campbell
2014-02-11 16:31                                       ` Ian Jackson
2014-03-12 13:54                   ` Ian Campbell
2014-03-12 13:56                     ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52F92743.8090901@terremark.com \
    --to=dslutz@verizon.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=Rob.Hoes@citrix.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=roger.pau@citrix.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).