Skip Menu |
Report information
Id: 133565
Status: open
Priority: 0/
Queue: perl5

Owner: Nobody
Requestors: andreas.koenig.7os6VVqR [at] franz.ak.mind.de
Cc:
AdminCc:

Operating System: (no value)
PatchStatus: (no value)
Severity: low
Type: unknown
Perl Version: (no value)
Fixed In: (no value)



To: perlbug [...] perl.org
Date: Sun, 07 Oct 2018 14:11:21 +0200
Subject: BBC MONGODB/BSON-v1.8.1.tar.gz
From: Andreas Koenig <andreas.koenig.7os6VVqR [...] franz.ak.mind.de>
BBC candidate: https://jira.mongodb.org/browse/PERL-1009 -- andreas PS: perl, perl, perl
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 2.1k
On Sun, 07 Oct 2018 12:11:36 GMT, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote: Show quoted text
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 [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 ##### 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)
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 2.6k
On Sun, 07 Oct 2018 13:30:19 GMT, jkeenan wrote: Show quoted text
> 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 > > [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 > ##### > > 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. >
Further investigation indicates that the regression in the BSON test with respect to perl-5.28.0 becomes manifest when '-Duselongdouble' is added to the configuration arguments. I built perl at v5.28.0 and at blead with only 'Duselongdouble' -- no ithreads or DEBUGGING. I used cpanm in each case to try to install BSON. The test failure did not appear at v5.28.0, only at blead. Show quoted text
> 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)
To: perl5-porters [...] perl.org
Date: Sun, 7 Oct 2018 16:30:01 +0200
Subject: Re: [perl #133565] BBC MONGODB/BSON-v1.8.1.tar.gz
From: "H.Merijn Brand" <h.m.brand [...] xs4all.nl>
CC: perlbug-followup [...] perl.org
Download (untitled) / with headers
text/plain 2.9k
On Sun, 07 Oct 2018 06:30:20 -0700, "James E Keenan via RT" <perlbug-followup@perl.org> wrote: Show quoted text
> 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! 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 Show quoted text
> [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'; Show quoted text
> 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/
Download (untitled)
application/pgp-signature 473b

Message body not shown because it is not plain text.

RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 3.2k
On Sun, 07 Oct 2018 14:30:35 GMT, hmbrand wrote: Show quoted text
> 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. Show quoted text
> 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'; Show quoted text
> > 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)
From: "H.Merijn Brand" <h.m.brand [...] xs4all.nl>
Date: Sun, 7 Oct 2018 18:54:16 +0200
Subject: Re: [perl #133565] BBC MONGODB/BSON-v1.8.1.tar.gz
CC: perlbug-followup [...] perl.org
To: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 4.2k
On Sun, 07 Oct 2018 08:29:23 -0700, "James E Keenan via RT" <perlbug-followup@perl.org> wrote: Show quoted text
> 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'; Show quoted text
> $ ./bin/perl -V:'(long)?d(ou)?ble?size' > doublesize='8'; > longdblsize='16';
As in that :) Show quoted text
> > > 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/
Download (untitled)
application/pgp-signature 473b

Message body not shown because it is not plain text.

RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 3.1k
On Sun, 07 Oct 2018 14:21:24 GMT, jkeenan wrote: Show quoted text
> On Sun, 07 Oct 2018 13:30:19 GMT, jkeenan 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 > > > > [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 > > ##### > > > > 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. > >
> > Further investigation indicates that the regression in the BSON test > with respect to perl-5.28.0 becomes manifest when '-Duselongdouble' is > added to the configuration arguments. > > I built perl at v5.28.0 and at blead with only 'Duselongdouble' -- no > ithreads or DEBUGGING. I used cpanm in each case to try to install > BSON. The test failure did not appear at v5.28.0, only at blead. > >
> > 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.
Incidentally, the failing test was only recently added to the BSON test suite. See: https://jira.mongodb.org/browse/PERL-1009?focusedCommentId=2028499&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-2028499 -- James E Keenan (jkeenan@cpan.org)


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

For issues related to this RT instance (aka "perlbug"), please contact perlbug-admin at perl.org