Mailing list for all users of the OCaml language and system.
 help / color / mirror / Atom feed
From: xclerc <xavier.clerc@inria.fr>
To: caml users <caml-list@inria.fr>
Cc: "Serge Ziryukin" <ftrvxmtrx@gmail.com>,
	"Damien Doligez" <damien.doligez@inria.fr>,
	"Stéphane Glondu" <steph@glondu.net>
Subject: Re: [Caml-list] OCaml 3.12.1 compatibility report
Date: Mon, 27 Jun 2011 14:49:09 +0200	[thread overview]
Message-ID: <4AEC4555-7685-4156-AEAE-062B64803556@inria.fr> (raw)
In-Reply-To: <4E085B9B.1020502@glondu.net>


Le 27 juin 2011 à 12:29, Stéphane Glondu a écrit :

> Le 16/06/2011 19:59, Serge Ziryukin a écrit :
>> [...]
>>> bin_prot-1.3.1.tar.gz
>> [...]
>> 
>> Did you also run 'make test' on bin_prot?
>> It fails for me:
>> 
>> I: Running command '/Users/ftrvxmtrx/temp/sandbox/bin_prot-1.3.1/_build/lib_test/test_runner'
>> ........................F................................F.........
>> ==============================================================================
>> Failure: 1:Bin_prot_c:24:bigstring
>> 
>> OUnit: read_bigstring [| ... |]: returned wrong result
>> ------------------------------------------------------------------------------
>> ==============================================================================
>> Failure: 0:Bin_prot_ml:24:bigstring
>> 
>> OUnit: read_bigstring [| ... |]: returned wrong result
>> ------------------------------------------------------------------------------
>> Ran: 67 tests in: 0.04 seconds.
>> FAILED: Cases: 67 Tried: 67 Errors: 0 Failures: 2 Skip:0 Todo:0
> 
> I can reproduce this error, and it appears with this new OCaml version (and not with OCaml 3.12.0).

Had quite a hard time to pinpoint the problem, but finally got it.
The tests are failing because a try is made to compare two big arrays with different layouts.
It used to be accepted by the big array compare function, but now only big arrays with the
same kind and layout can be compared [1].


Regards,

Xavier Clerc


[1] cf. http://caml.inria.fr/cgi-bin/viewcvs.cgi/ocaml/version/3.12/otherlibs/bigarray/bigarray_stubs.c?rev=11037&r1=10496&r2=11037



  reply	other threads:[~2011-06-27 12:49 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-16 15:01 Damien Doligez
2011-06-16 15:11 ` [Caml-list] " Sylvain Le Gall
2011-06-16 15:18 ` [Caml-list] " Stéphane Glondu
2011-06-16 15:33 ` Florent Ouchet
2011-06-16 17:46   ` Gerd Stolpmann
2011-06-27  9:31     ` Damien Doligez
2011-06-28  8:07       ` Florent Ouchet
2011-06-16 17:59 ` Serge Ziryukin
2011-06-27 10:29   ` Stéphane Glondu
2011-06-27 12:49     ` xclerc [this message]
2011-06-27 14:40       ` Till Varoquaux
2011-06-27 15:22       ` Xavier Leroy
2011-06-27 15:57         ` Markus Mottl
2011-06-28 16:52           ` Damien Doligez
2011-06-30 21:37             ` Markus Mottl
2011-07-04 11:45               ` xclerc

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=4AEC4555-7685-4156-AEAE-062B64803556@inria.fr \
    --to=xavier.clerc@inria.fr \
    --cc=caml-list@inria.fr \
    --cc=damien.doligez@inria.fr \
    --cc=ftrvxmtrx@gmail.com \
    --cc=steph@glondu.net \
    /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