develooper Front page | perl.perl5.porters | Postings from October 2018

[perl #133565] BBC MONGODB/BSON-v1.8.1.tar.gz

Thread Previous | Thread Next
From:
James E Keenan via RT
Date:
October 7, 2018 15:29
Subject:
[perl #133565] BBC MONGODB/BSON-v1.8.1.tar.gz
Message ID:
rt-4.0.24-20432-1538926163-1157.133565-15-0@perl.org
On Sun, 07 Oct 2018 14:30:35 GMT, hmbrand wrote:
> On Sun, 07 Oct 2018 06:30:20 -0700, "James E Keenan via RT"
> <perlbug-followup@perl.org> wrote:
> 
> > On Sun, 07 Oct 2018 12:11:36 GMT,
> > andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
> > > BBC candidate: https://jira.mongodb.org/browse/PERL-1009
> >
> > I tested BSON-v1.8.1 with both perl-5.28.0 and blead, each with
> > relatively simple configuration arguments:
> >
> > #####
> > sh ./Configure -des -Dusedevel
> > #####
> >
> > I could not reproduce your result.
> >
> > At blead:
> >
> > #####
> > [BSON-v1.8.1] 517 $ ~/testing/blead/bin/perl -I~/testing/blead/lib -v
> > | head -2 | tail -1
> > This is perl 5, version 29, subversion 4 (v5.29.4 (v5.29.3-42-
> > g631637d8aa)) built for x86_64-linux
> 
> This is not a longdouble build!
> 

I know.

> This is perl 5, version 29, subversion 4 (v5.29.4 (v5.29.3-42-
> g631637d8aa)) built for x86_64-linux-thread-multi-ld
> 
> Note the -ld at the end. The bug report specifically stated a failed
> with -Duselongdouble
> 
> > [BSON-v1.8.1] 520 $ ~/testing/blead/bin/prove -I~/testing/blead/lib
> > -b t/corpus/datetime.t
> >   t/corpus/datetime.t .. ok
> > All tests successful.
> > Files=1, Tests=6,  0 wallclock secs ( 0.01 usr  0.00 sys +  0.13 cusr
> > 0.01 csys =  0.15 CPU)
> > Result: PASS
> > #####
> >
> > It was only by adding additional configuration arguments that I was
> > able to reproduce the regression you reported from perl-5.28.0.
> >
> > #####
> > -des -Dusedevel -Uuseithreads -Duselongdouble -DEBUGGING=-g
> > #####
> 
> Can you verify that the perl that was built was actually a longdouble
> build?
> 
> $ perl -V:'(long)?d(ou)?ble?size'
> doublesize='8';
> longdblsize='16';
> 

Yes, the last build was longdouble:

$ ./bin/perl -Ilib -V | grep config_args
    config_args='-des -Dusedevel -Duselongdouble -Dprefix=/home/jkeenan/testing/v5.28.0 -Uversiononly -Dman1dir=none -Dman3dir=none'

$ ./bin/perl -V:'(long)?d(ou)?ble?size'
doublesize='8';
longdblsize='16';

> > The failure in your report
> > (http://www.cpantesters.org/cpan/report/8822a532-c7a3-11e8-82ba-
> > 24c27b903acc) looks like this:
> >
> > #####
> > #   Failed test 'native_to_relaxed_extended_json(json_to_native(rEJ))
> > = rEJ'
> > #   at t/lib/CorpusTest.pm line 255.
> > #          got: '{"a":{"$date":"2012-12-24T12:15:30.500Z"}}'
> > #     expected: '{"a":{"$date":"2012-12-24T12:15:30.501Z"}}'
> > # Looks like you failed 1 test of 6.
> >
> > #   Failed test 'case: positive ms'
> > #   at t/corpus/datetime.t line 13.
> > # Looks like you failed 1 test of 6.
> >  t/corpus/datetime.t ..............
> > Dubious, test returned 1 (wstat 256, 0x100)
> >  Failed 1/6 subtests
> > #####
> >
> > So we will need to investigate where in the additional config_args
> > the regression appears.
> >
> > For reference, the .json data in the BSON test suite looks like this:
> >
> > #####
> > "valid": [
> > ...
> >     {
> >         "description": "positive ms",
> >         "canonical_bson": "10000000096100C5D8D6CC3B01000000",
> >         "relaxed_extjson": "{\"a\" : {\"$date\" : \"2012-12-
> > 24T12:15:30.501Z\"}}",
> >         "canonical_extjson": "{\"a\" : {\"$date\" : {\"$numberLong\"
> > : \"1356351330501\"}}}"
> >     },
> > #####
> >
> > Thank you very much.


-- 
James E Keenan (jkeenan@cpan.org)

---
via perlbug:  queue: perl5 status: open
https://rt.perl.org/Ticket/Display.html?id=133565

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About