Skip to content

BBC: Blead Breaks Mojo::JSON::MaybeXS #23086

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

Closed
cjg-cguevara opened this issue Mar 5, 2025 · 3 comments
Closed

BBC: Blead Breaks Mojo::JSON::MaybeXS #23086

cjg-cguevara opened this issue Mar 5, 2025 · 3 comments
Labels
BBC Blead Breaks CPAN - changes in blead broke a cpan module(s)

Comments

@cjg-cguevara
Copy link

This is a bug report for perl from "Carlos Guevara" [email protected],
generated with the help of perlbug 1.43 running under perl 5.41.10.


BBC: Blead Breaks Mojo::JSON::MaybeXS

Please see http://fast-matrix.cpantesters.org/?dist=Mojo::JSON::MaybeXS


Flags

  • category=core
  • severity=low

Perl configuration

Site configuration information for perl 5.41.10:

Configured by cpan at Wed Mar  5 17:48:50 EST 2025.

Summary of my perl5 (revision 5 version 41 subversion 10) configuration:
  Commit id: c2ae512726a79a146740ae764a0049d40eac1a17
  Platform:
    osname=linux
    osvers=5.14.0-503.26.1.el9_5.x86_64
    archname=x86_64-linux
    uname='linux cjg-rhel9 5.14.0-503.26.1.el9_5.x86_64 #1 smp preempt_dynamic mon feb 10 05:23:56 est 2025 x86_64 x86_64 x86_64 gnulinux '
    config_args='-des -Dprefix=/home/cpan/bin/perl -Dscriptdir=/home/cpan/bin/perl/bin -Dusedevel -Duse64bitall'
    hint=recommended
    useposix=true
    d_sigaction=define
    useithreads=undef
    usemultiplicity=undef
    use64bitint=define
    use64bitall=define
    uselongdouble=undef
    usemymalloc=n
    default_inc_excludes_dot=define
  Compiler:
    cc='cc'
    ccflags ='-fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2'
    optimize='-O2'
    cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include'
    ccversion=''
    gccversion='11.5.0 20240719 (Red Hat 11.5.0-5)'
    gccosandvers=''
    intsize=4
    longsize=8
    ptrsize=8
    doublesize=8
    byteorder=12345678
    doublekind=3
    d_longlong=define
    longlongsize=8
    d_longdbl=define
    longdblsize=16
    longdblkind=3
    ivtype='long'
    ivsize=8
    nvtype='double'
    nvsize=8
    Off_t='off_t'
    lseeksize=8
    alignbytes=8
    prototype=define
  Linker and Libraries:
    ld='cc'
    ldflags =' -fstack-protector-strong -L/usr/local/lib'
    libpth=/usr/local/lib /usr/lib /usr/lib64 /usr/local/lib64
    libs=-lpthread -ldb -ldl -lm -lcrypt -lutil -lc
    perllibs=-lpthread -ldl -lm -lcrypt -lutil -lc
    libc=/lib/../lib64/libc.so.6
    so=so
    useshrplib=false
    libperl=libperl.a
    gnulibc_version='2.34'
  Dynamic Linking:
    dlsrc=dl_dlopen.xs
    dlext=so
    d_dlsymun=undef
    ccdlflags='-Wl,-E'
    cccdlflags='-fPIC'
    lddlflags='-shared -O2 -L/usr/local/lib -fstack-protector-strong'


---
@INC for perl 5.41.10:
    /home/cpan/bin/perl/lib/site_perl/5.41.10/x86_64-linux
    /home/cpan/bin/perl/lib/site_perl/5.41.10
    /home/cpan/bin/perl/lib/5.41.10/x86_64-linux
    /home/cpan/bin/perl/lib/5.41.10

---
Environment for perl 5.41.10:
    HOME=/home/cpan
    LANG=en_US.UTF-8
    LANGUAGE (unset)
    LC_ALL=C
    LD_LIBRARY_PATH (unset)
    LOGDIR (unset)
    PATH=/home/cpan/bin/perl/bin:/home/cpan/bin:/usr/share/Modules/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin
    PERL_BADLANG (unset)
    SHELL=/bin/bash
@cjg-cguevara cjg-cguevara changed the title Blead Breaks Mojo::JSON::MaybeXS BBC: Blead Breaks Mojo::JSON::MaybeXS Mar 5, 2025
@jkeenan jkeenan added the BBC Blead Breaks CPAN - changes in blead broke a cpan module(s) label Mar 6, 2025
@jkeenan
Copy link
Contributor

jkeenan commented Mar 6, 2025

CPANtesters is having significant trouble accepting reports from testers and displaying reports to observers. Until that changes, I'm not going to begin attempting a bisection.

@jkeenan
Copy link
Contributor

jkeenan commented Mar 6, 2025

Mojo::JSON::MaybeXS has a dependency on JSON::MaybeXS. Breakage of the latter has been reported in #23053.

khwilliamson added a commit to khwilliamson/perl5 that referenced this issue Mar 9, 2025
This reverts commit d62b9fa.
That commit turns out to be ill advised.  I forgot that the function
call it replaced also calls utf8_to_uv() and then munges its return value.
This commit to succeed would have needed the same munging.  But rather
than repeat that logic, just call the original function, by simply
reverting this commit.

This fixes Perl#23053.  I haven't tested them, but it likely also takes care
of Perl#22820, Perl#23086, and Perl#22977
khwilliamson added a commit that referenced this issue Mar 12, 2025
This reverts commit d62b9fa.
That commit turns out to be ill advised.  I forgot that the function
call it replaced also calls utf8_to_uv() and then munges its return value.
This commit to succeed would have needed the same munging.  But rather
than repeat that logic, just call the original function, by simply
reverting this commit.

This fixes #23053.  I haven't tested them, but it likely also takes care
of #22820, #23086, and #22977
@jkeenan
Copy link
Contributor

jkeenan commented Mar 13, 2025

Successfully installed against v5.41.9-77-g28aeb4ecd4 (Mar 12 2025) on Linux and FreeBSD. Closing ticket.

@jkeenan jkeenan closed this as completed Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BBC Blead Breaks CPAN - changes in blead broke a cpan module(s)
Projects
None yet
Development

No branches or pull requests

2 participants