New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BBC: ce6f496d72 breaks MONGODB/BSON-v1.8.1.tar.gz #16712
Comments
From @andkBBC candidate: https://jira.mongodb.org/browse/PERL-1009 |
From @jkeenanOn 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: ##### I could not reproduce your result. At blead: ##### [BSON-v1.8.1] 520 $ It was only by adding additional configuration arguments that I was able to reproduce the regression you reported from perl-5.28.0. ##### The failure in your report (http://www.cpantesters.org/cpan/report/8822a532-c7a3-11e8-82ba-24c27b903acc) looks like this: ##### # Failed test 'case: positive ms' 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: ##### Thank you very much. |
The RT System itself - Status changed from 'new' to 'open' |
From @jkeenanOn Sun, 07 Oct 2018 13:30:19 GMT, jkeenan wrote:
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.
-- |
From @TuxOn Sun, 07 Oct 2018 06:30:20 -0700, "James E Keenan via RT"
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
Can you verify that the perl that was built was actually a longdouble $ perl -V:'(long)?d(ou)?ble?size'
-- |
From @jkeenanOn Sun, 07 Oct 2018 14:30:35 GMT, hmbrand wrote:
I know.
Yes, the last build was longdouble: $ ./bin/perl -Ilib -V | grep config_args $ ./bin/perl -V:'(long)?d(ou)?ble?size'
-- |
From @TuxOn Sun, 07 Oct 2018 08:29:23 -0700, "James E Keenan via RT"
FYI, this is builtin functionality, as shown in my previous example. $ ./perl -Ilib -V:config_args or, as it supports full perl regular expressions: $ ./perl -Ilib -V:config.*
As in that :)
-- |
From @jkeenanOn Sun, 07 Oct 2018 14:21:24 GMT, jkeenan wrote:
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 -- |
From @jkeenanOn Tue, 09 Oct 2018 22:07:44 GMT, jkeenan wrote:
It appears that the problem in mongodb has been fixed upstream; see https://jira.mongodb.org/browse/PERL-1009. Building perl from current blead with longdouble and testing the BSON module PASSes for me. ##### $ bleadperl -V:config_args $ bleadprove -l t/corpus/datetime.t But we'll need a new CPAN release of BSON to resolve this ticket once and for all. Thank you very much. -- |
From @jkeenanOn Tue, 20 Nov 2018 19:16:27 GMT, jkeenan wrote:
BSON has received a new CPAN release, which builds successfully on Perl 5 blead. Resolving ticket. ##### -- |
@jkeenan - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#133565 (status was 'resolved')
Searchable as RT133565$
The text was updated successfully, but these errors were encountered: