Re: 4.4.0-rc3 tagged

[Don Slutz <dslutz@verizon.com>]

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:
>>>>>>>>> cc1: warnings being treated as errors
>>>>>>>>> xenlight_stubs.c: In function 'Defbool_val':
>>>>>>>>> xenlight_stubs.c:344: warning: implicit declaration of function
>>>>>>>>> 'CAMLreturnT'
>>>>>>>>> xenlight_stubs.c:344: error: expected expression before
>>>>>>>>> 'libxl_defbool'
>>>>>>>>> xenlight_stubs.c: In function 'String_option_val':
>>>>>>>>> xenlight_stubs.c:379: error: expected expression before 'char'
>>>>>>>>> xenlight_stubs.c: In function 'aohow_val':
>>>>>>>>> xenlight_stubs.c:440: error: expected expression before
>>>>>>>>> 'libxl_asyncop_how'
>>>>>> Any idea on what to do about ocaml issue?
>>>>> My guess is that your ocaml is too old and doesn't supply CAMLreturnT.
>>>>> What version do you have?
>>>>>
>>>>> Ian.
>>>>>
>>>> dcs-xen-53:~>ocaml -version
>>>> The Objective Caml toplevel, version 3.09.3
>>>>
>>>>     -Don Slutz
>>>>
>>> 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

and the generated script is bad:


     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

fi


    -Don Slutz



> ---
> commit e49609cc7b93c2633cf5a49206cb29d6bdd612be
> Author: Roger Pau Monne <roger.pau@citrix.com>
> Date:   Mon Feb 10 11:54:13 2014 +0100
>
>      tools: check OCaml version is at least 3.09.3
>      
>      Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>
> diff --git a/tools/configure.ac b/tools/configure.ac
> index 0754f0e..6d1e2ee 100644
> --- a/tools/configure.ac
> +++ b/tools/configure.ac
> @@ -161,6 +161,12 @@ AS_IF([test "x$ocamltools" = "xy"], [
>           AS_IF([test "x$enable_ocamltools" = "xyes"], [
>               AC_MSG_ERROR([Ocaml tools enabled, but unable to find Ocaml])])
>           ocamltools="n"
> +    ], [
> +        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"
> +        ])
>       ])
>   ])
>   AS_IF([test "x$xsmpolicy" = "xy"], [
>
>


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