BBC: Blead Breaks Sub::Exporter

[cjg-cguevara]

This is a bug report for perl from "Carlos Guevara" carlos@carlosguevara.com,
generated with the help of perlbug 1.43 running under perl 5.39.1.

---

BBC: Blead Breaks Sub::Exporter

Please see http://fast-matrix.cpantesters.org/?dist=Sub::Exporter

---

Flags

• category=core
• severity=low

---

Perl configuration

Site configuration information for perl 5.39.1:

Configured by cpan at Wed Jul 19 23:35:05 EDT 2023.

Summary of my perl5 (revision 5 version 39 subversion 1) configuration:
  Commit id: 923dde9b8ebe55ba73323ed675156712bc0062c9
  Platform:
    osname=linux
    osvers=5.15.120-0-lts
    archname=x86_64-linux-thread-multi
    uname='linux cjg-alpine3 5.15.120-0-lts #1-alpine smp wed, 05 jul 2023 18:59:57 +0000 x86_64 linux '
    config_args='-des -Dprefix=/home/cpan/bin/perl -Dscriptdir=/home/cpan/bin/perl/bin -Dusedevel -Duse64bitall -Duseithreads'
    hint=recommended
    useposix=true
    d_sigaction=define
    useithreads=define
    usemultiplicity=define
    use64bitint=define
    use64bitall=define
    uselongdouble=undef
    usemymalloc=n
    default_inc_excludes_dot=define
  Compiler:
    cc='cc'
    ccflags ='-D_REENTRANT -D_GNU_SOURCE -D_GNU_SOURCE -fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64'
    optimize='-O2'
    cppflags='-D_REENTRANT -D_GNU_SOURCE -D_GNU_SOURCE -fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong'
    ccversion=''
    gccversion='12.2.1 20220924'
    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/lib /usr/local/lib /lib
    libs=-lpthread -ldb -ldl -lm -lcrypt -lutil -lc
    perllibs=-lpthread -ldl -lm -lcrypt -lutil -lc
    libc=/usr/lib/libc.a
    so=so
    useshrplib=false
    libperl=libperl.a
    gnulibc_version=''
  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.39.1:
    /home/cpan/bin/perl/lib/site_perl/5.39.1/x86_64-linux-thread-multi
    /home/cpan/bin/perl/lib/site_perl/5.39.1
    /home/cpan/bin/perl/lib/5.39.1/x86_64-linux-thread-multi
    /home/cpan/bin/perl/lib/5.39.1

---
Environment for perl 5.39.1:
    HOME=/home/cpan
    LANG=C.UTF-8
    LANGUAGE (unset)
    LC_COLLATE=C
    LD_LIBRARY_PATH (unset)
    LOGDIR (unset)
    PATH=/home/cpan/bin/perl/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
    PERL_BADLANG (unset)
    SHELL=/bin/bash

[demerphq]

This is a bug in Sub::Exporter's test suite, and is fixed by rjbs/Sub-Exporter#17

[demerphq]

Happy Birthday @rjbs.

[jkeenan]

This is a bug in Sub::Exporter's test suite, and is fixed by rjbs/Sub-Exporter#17

Bisecting shows that this failure shows up in blead at this commit:

commit 2dcf3cf50d7cb67422ffe2c2a3e2d3dc404ea6c4 (HEAD, refs/bisect/bad)
Author:     Yves Orton <demerphq@gmail.com>
AuthorDate: Mon Feb 14 12:24:21 2022 +0100
Commit:     Yves Orton <demerphq@gmail.com>
CommitDate: Tue Jul 18 20:26:14 2023 +0200

    Fix assorted bugs related to not having a UNIVERSAL::import

... which is consistent with @demerphq's explanation. I suspect that since this commit went into blead just two days ago, we'll see other CPAN breakage connected to that commit and have to file tickets for other distros upstream.

[demerphq]

I suspect that since this commit went into blead just two days ago, we'll see other CPAN breakage connected to that commit and have to file tickets for other distros upstream.

Yep. Very very likely. This patch is specifically intended to detect people doing stuff that wouldn't be detected prior to it.

[rjbs]

Hey, thanks! I'm out of town on business, but this all looks like Yves is probably right, and I'll dig into it pretty soon.

[andk]

Also affected: LEONT/CPAN-Meta-Check-0.017.tar.gz
@Leont please take note

[demerphq]

Re CPAN::Meta::Check:

------------------------------
PROGRAM OUTPUT
------------------------------

Output from '/usr/bin/make test':

PERL_DL_NONLAZY=1 "/tmp/basesmoker-reloperl-uCLf/bin/perl" "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
Attempt to call undefined import method with arguments via package "CPAN::Meta::Prereqs" (Perhaps you forgot to load the package?) at /tmp/loop_over_bdir-101935-CrkcWo/CPAN-Meta-Check-0.017-0/blib/lib/CPAN/Meta/Check.pm line 12.
BEGIN failed--compilation aborted at /tmp/loop_over_bdir-101935-CrkcWo/CPAN-Meta-Check-0.017-0/blib/lib/CPAN/Meta/Check.pm line 12.
Compilation failed in require at t/10-basics.t line 8.
BEGIN failed--compilation aborted at t/10-basics.t line 8.
t/10-basics.t ......... 
Dubious, test returned 255 (wstat 65280, 0xff00)
No subtests run 
Attempt to call undefined import method with arguments via package "CPAN::Meta::Prereqs" (Perhaps you forgot to load the package?) at /tmp/loop_over_bdir-101935-CrkcWo/CPAN-Meta-Check-0.017-0/blib/lib/CPAN/Meta/Check.pm line 12.
BEGIN failed--compilation aborted at /tmp/loop_over_bdir-101935-CrkcWo/CPAN-Meta-Check-0.017-0/blib/lib/CPAN/Meta/Check.pm line 12.
Compilation failed in require at t/20-undef-version.t line 6.
BEGIN failed--compilation aborted at t/20-undef-version.t line 6.
t/20-undef-version.t .. 
Dubious, test returned 255 (wstat 65280, 0xff00)
No subtests run 

Test Summary Report
-------------------
t/10-basics.t       (Wstat: 65280 (exited 255) Tests: 0 Failed: 0)
  Non-zero exit status: 255
  Parse errors: No plan found in TAP output
t/20-undef-version.t (Wstat: 65280 (exited 255) Tests: 0 Failed: 0)
  Non-zero exit status: 255
  Parse errors: No plan found in TAP output
Files=2, Tests=0,  0 wallclock secs ( 0.01 usr  0.00 sys +  0.14 cusr  0.02 csys =  0.17 CPU)
Result: FAIL
Failed 2/2 test programs. 0/0 subtests failed.
make: *** [Makefile:873: test_dynamic] Error 255

[demerphq]

Re: CPAN::Meta::Check, the offending line of code is

use CPAN::Meta::Prereqs '2.132830';

I guess this is intended to be a version Check, but since it's not a number it gets treated as an argument to import. We could probably handle this case more gracefully in UNIVERSAL::import(), but I'm not sure if we should, it seems preferable to keep it very simple. But perhaps we could add a bespoke error for this case to make it easier to understand what is going on.

Again, given the highly experienced nature of the author of this code it seems likely we will discover quite a bit more code where people thought they were doing something that were actually no-ops in disguise.

[jkeenan]

The impact of this breakage is massive. On both FreeBSD and Linux, I built perls from the v5.39.1 tag, attempted to install Task::CPAN::Reporter against them and generate CPANtesters reports from a list of 500 modules. Because Sub::Exporter is a prereq to Test::Reporter::Transport::Metabase, even though the modules were actually tested, no reports were sent and all were marked DISCARD.

We should consider reverting the breaking commit until we can better assess the impact of the change.

[demerphq]

We should consider reverting the breaking commit until we can better assess the impact of the change.

That sounds like an over-reaction to me personally. When @rjbs can roll a new release of Sub-Exporter things will go back to normal. Presumably if he wasn't on holiday this would be fixed already. We can wait a few days IMO.

Just force install Sub::Exporter for now. The only thing broken there is a test that doesn't quite do what he thought it did.

[dur-randir]

Looks like Moose is also hit by this

#   Failed test '... got the right list of applicable methods for Foo'
#   at t/cmop/methods.t line 211.
#     Structures begin differing at:
#          $got->[28] = 'import'
#     $expected->[28] = 'isa'

#   Failed test '... got the right list of applicable methods for Bar'
#   at t/cmop/methods.t line 302.
#     Structures begin differing at:
#          $got->[28] = 'import'
#     $expected->[28] = 'isa'
# Looks like you failed 2 tests of 81.

Out of my to-install list, I see ~80 failures, but didn't dive into them thoroughly.