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

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

Thread Previous | Thread Next
From:
H.Merijn Brand
Date:
October 7, 2018 16:54
Subject:
Re: [perl #133565] BBC MONGODB/BSON-v1.8.1.tar.gz
Message ID:
20181007185416.3a2d8361@pc09.procura.nl
On Sun, 07 Oct 2018 08:29:23 -0700, "James E Keenan via RT"
<perlbug-followup@perl.org> wrote:

> 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:  
>  [...]  
> > >
> > > 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'

FYI, this is builtin functionality, as shown in my previous example.

$ ./perl -Ilib -V:config_args
config_args='-Dusedevel -Dusethreads -Duseithreads -Duselongdouble -Duse64bitall -des';

or, as it supports full perl regular expressions:

$ ./perl -Ilib -V:config.*
config_arg0='./Configure';
config_arg1='-Dusedevel';
config_arg2='-Dusethreads';
config_arg3='-Duseithreads';
config_arg4='-Duselongdouble';
config_arg5='-Duse64bitall';
config_arg6='-des';
config_argc='6';
config_args='-Dusedevel -Dusethreads -Duseithreads -Duselongdouble -Duse64bitall -des';

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

As in that :)

> > > 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.

-- 
H.Merijn Brand  http://tux.nl   Perl Monger  http://amsterdam.pm.org/
using perl5.00307 .. 5.29   porting perl5 on HP-UX, AIX, and openSUSE
http://mirrors.develooper.com/hpux/        http://www.test-smoke.org/
http://qa.perl.org   http://www.goldmark.org/jeff/stupid-disclaimers/

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