From: Till Varoquaux <tvaroquaux@janestcapital.com>
To: xclerc <xavier.clerc@inria.fr>
Cc: "caml users" <caml-list@inria.fr>,
"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 10:40:54 -0400 [thread overview]
Message-ID: <BANLkTi=Gz0EQkffNTHf0W4X_bsNoA+U5BQ@mail.gmail.com> (raw)
In-Reply-To: <4AEC4555-7685-4156-AEAE-062B64803556@inria.fr>
On Mon, Jun 27, 2011 at 8:49 AM, xclerc <xavier.clerc@inria.fr> wrote:
>
> 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
>
>
>
Thank you for every one who looked into it.
We (Janestreet) are not using 3.12.1 yet. We'll try to have a test
setup ASAP and issue a bug-fix release.
Xavier: your work will be very helpful in squashing that bug; thank
you very much.
Till
> --
> Caml-list mailing list. Subscription management and archives:
> https://sympa-roc.inria.fr/wws/info/caml-list
> Beginner's list: http://groups.yahoo.com/group/ocaml_beginners
> Bug reports: http://caml.inria.fr/bin/caml-bugs
>
>
next prev parent reply other threads:[~2011-06-27 14:40 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
2011-06-27 14:40 ` Till Varoquaux [this message]
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='BANLkTi=Gz0EQkffNTHf0W4X_bsNoA+U5BQ@mail.gmail.com' \
--to=tvaroquaux@janestcapital.com \
--cc=caml-list@inria.fr \
--cc=damien.doligez@inria.fr \
--cc=ftrvxmtrx@gmail.com \
--cc=steph@glondu.net \
--cc=till.varoquaux@gmail.com \
--cc=xavier.clerc@inria.fr \
/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).