Skip to content
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

New sub(){$var} deprecation breaks CPAN #14243

Closed
p5pRT opened this issue Nov 15, 2014 · 19 comments
Closed

New sub(){$var} deprecation breaks CPAN #14243

p5pRT opened this issue Nov 15, 2014 · 19 comments

Comments

@p5pRT
Copy link

p5pRT commented Nov 15, 2014

Migrated from rt.perl.org#123213 (status was 'resolved')

Searchable as RT123213$

@p5pRT
Copy link
Author

p5pRT commented Nov 15, 2014

From @cpansprout

So far these two seem to be broken​:

HKOBA/YATT-Lite-0.0.9.tar.gz
ABW/Badger-0.09.tar.gz

I’m still testing other dists.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Nov 15, 2014

From @jkeenan

On Fri Nov 14 22​:03​:32 2014, sprout wrote​:

So far these two seem to be broken​:

HKOBA/YATT-Lite-0.0.9.tar.gz
ABW/Badger-0.09.tar.gz

I’m still testing other dists.

Searching through 'git log', I came across 'sub ()' more than once in the past month.

Is there a particular commit which describes this new deprecation? Or has there been an addition to the docs about it? (I'm having trouble figuring out what the change in functionality actually was.)

Thank you very much.

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Nov 15, 2014

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Nov 15, 2014

From @cpansprout

On Sat Nov 15 04​:18​:34 2014, jkeenan wrote​:

On Fri Nov 14 22​:03​:32 2014, sprout wrote​:

So far these two seem to be broken​:

HKOBA/YATT-Lite-0.0.9.tar.gz
ABW/Badger-0.09.tar.gz

I’m still testing other dists.

Searching through 'git log', I came across 'sub ()' more than once in
the past month.

Is there a particular commit which describes this new deprecation?

0ac016f

Or
has there been an addition to the docs about it? (I'm having trouble
figuring out what the change in functionality actually was.)

d3c633b

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Nov 16, 2014

From @cpansprout

On Fri Nov 14 22​:03​:32 2014, sprout wrote​:

So far these two seem to be broken​:

HKOBA/YATT-Lite-0.0.9.tar.gz
ABW/Badger-0.09.tar.gz

I’m still testing other dists.

HKOBA/YATT-v0.0.8.tar.gz
SAMV/Net-SSLeay-OO-0.02.tar.gz [rt.cpan.org #100364]

Test​::LeakTrace apparently broken by other blead changes (at least under debugging builds). Needed for testing SYOHEX/Text-Xslate-3.3.3.tar.gz.

SUPER broken in blead, which prevents testing BAREFOOT/Debuggit-2.05.tar.gz.

MONS/XML-Hash-LX-0.0603.tar.gz broken (blead-related?), which prevents testing MONS/XML-RPC-Fast-0.8.tar.gz

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Nov 16, 2014

From @andk

On Sat, 15 Nov 2014 22​:50​:35 -0800, "Father Chrysostomos via RT" <perlbug-followup@​perl.org> said​:

HKOBA/YATT-Lite-0.0.9.tar.gz
ABW/Badger-0.09.tar.gz
  > HKOBA/YATT-v0.0.8.tar.gz

For me all three already broken since v5.21.1-11-g4077a6b

  > SAMV/Net-SSLeay-OO-0.02.tar.gz [rt.cpan.org #100364]
  > Test​::LeakTrace apparently broken by other blead changes (at least under debugging builds). Needed for testing SYOHEX/Text-Xslate-3.3.3.tar.gz.

Leaktrace bisected to v5.21.5-396-ga635561

  > SUPER broken in blead, which prevents testing BAREFOOT/Debuggit-2.05.tar.gz.

Test-More/test-more#476
https://rt.cpan.org/Ticket/Display.html?id=97939

  > MONS/XML-Hash-LX-0.0603.tar.gz broken (blead-related?), which
  > prevents testing MONS/XML-RPC-Fast-0.8.tar.gz

https://rt.cpan.org/Ticket/Display.html?id=81645

new BBC


I would make this a new BBC, but you seem to prefer to have it here?
GPHAT/MooseX-Method-0.44.tar.gz broken by v5.21.5-413-gd0a32af

--
andreas

@p5pRT
Copy link
Author

p5pRT commented Nov 17, 2014

From @cpansprout

On Sun Nov 16 02​:31​:21 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

On Sat, 15 Nov 2014 22​:50​:35 -0800, "Father Chrysostomos via RT"
<perlbug-followup@​perl.org> said​:

HKOBA/YATT-Lite-0.0.9.tar.gz
ABW/Badger-0.09.tar.gz
HKOBA/YATT-v0.0.8.tar.gz

For me all three already broken since v5.21.1-11-g4077a6b

I didn’t actually do any bisecting. I just tested suspicious-looking modules that I found via CPAN grep.

SAMV/Net-SSLeay-OO-0.02.tar.gz [rt.cpan.org #100364]
Test​::LeakTrace apparently broken by other blead changes (at least
under debugging builds). Needed for testing SYOHEX/Text-Xslate-
3.3.3.tar.gz.

Leaktrace bisected to v5.21.5-396-ga635561

I have opened #123223. Fixing that will be rather involved, and may break other modules.

SUPER broken in blead, which prevents testing BAREFOOT/Debuggit-
2.05.tar.gz.

Test-More/test-more#476
https://rt.cpan.org/Ticket/Display.html?id=97939

MONS/XML-Hash-LX-0.0603.tar.gz broken (blead-related?), which
prevents testing MONS/XML-RPC-Fast-0.8.tar.gz

https://rt.cpan.org/Ticket/Display.html?id=81645

new BBC
-------

I would make this a new BBC, but you seem to prefer to have it here?
GPHAT/MooseX-Method-0.44.tar.gz broken by v5.21.5-413-gd0a32af

On the contrary, this one is different enough that I have opened #123224 for it.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Nov 17, 2014

From @karenetheridge

On Sat, Nov 15, 2014 at 10​:50​:35PM -0800, Father Chrysostomos via RT wrote​:

On Fri Nov 14 22​:03​:32 2014, sprout wrote​:

So far these two seem to be broken​:

HKOBA/YATT-Lite-0.0.9.tar.gz
ABW/Badger-0.09.tar.gz

I’m still testing other dists.

HKOBA/YATT-v0.0.8.tar.gz
SAMV/Net-SSLeay-OO-0.02.tar.gz [rt.cpan.org #100364]

Also, aliased.pm is affected​:
https://rt.cpan.org/Ticket/Display.html?id=100359

Test​::LeakTrace apparently broken by other blead changes (at least under debugging builds). Needed for testing SYOHEX/Text-Xslate-3.3.3.tar.gz.

We've encountered this in Moose's tests too​:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/41186826/log.txt

SUPER broken in blead, which prevents testing BAREFOOT/Debuggit-2.05.tar.gz.

I see chromatic just released a new SUPER today.

@p5pRT
Copy link
Author

p5pRT commented Nov 18, 2014

From @cpansprout

On Sun Nov 16 02​:31​:21 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

SUPER broken in blead, which prevents testing BAREFOOT/Debuggit-
2.05.tar.gz.

Test-More/test-more#476
https://rt.cpan.org/Ticket/Display.html?id=97939

Debuggit passes its tests, but warns. I’ve submitted a patch to <https://rt.cpan.org/Ticket/Display.html?id=90433>.

MONS/XML-Hash-LX-0.0603.tar.gz broken (blead-related?), which
prevents testing MONS/XML-RPC-Fast-0.8.tar.gz

https://rt.cpan.org/Ticket/Display.html?id=81645

With the patch from that ticket, I was able to test MONS/XML-RPC-Fast-0.8.tar.gz, which is fine.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Nov 22, 2014

From @andk

Also Affected​: JSWARTZ/Mason-2.22.tar.gz
Bisect​: v5.21.5-402-g0ac016f
Report​: http​://www.cpantesters.org/cpan/report/f5837424-7119-11e4-9a97-65c510862755
--
andreas

@p5pRT
Copy link
Author

p5pRT commented Nov 22, 2014

From @cpansprout

On Fri Nov 21 21​:36​:58 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

Also Affected​: JSWARTZ/Mason-2.22.tar.gz
Bisect​: v5.21.5-402-g0ac016f
Report​: http​://www.cpantesters.org/cpan/report/f5837424-7119-11e4-
9a97-65c510862755

Patch at <https://rt.cpan.org/Ticket/Display.html?id=100507>.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Dec 8, 2014

From @andk

Also affected​:

Exact bisect​: v5.21.5-402-g0ac016f
PREACTION/Statocles-0.026.tar.gz
http​://www.cpantesters.org/cpan/report/4704cc42-7e96-11e4-aaa8-d1ac52b25338

--
andreas

@p5pRT
Copy link
Author

p5pRT commented Dec 9, 2014

From @cpansprout

On Mon Dec 08 12​:27​:19 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

Also affected​:

Exact bisect​: v5.21.5-402-g0ac016f
PREACTION/Statocles-0.026.tar.gz
http​://www.cpantesters.org/cpan/report/4704cc42-7e96-11e4-aaa8-d1ac52b25338

It depends on Type​::Tiny, which fails assertions added by the multideref code​:

Assertion failed​: (!(o->op_flags & ~(OPf_WANT|OPf_KIDS|OPf_MOD))), function S_maybe_multideref, file op.c, line 12520.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Dec 9, 2014

From @cpansprout

On Mon Dec 08 16​:16​:11 2014, sprout wrote​:

On Mon Dec 08 12​:27​:19 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de
wrote​:

Also affected​:

Exact bisect​: v5.21.5-402-g0ac016f
PREACTION/Statocles-0.026.tar.gz
http​://www.cpantesters.org/cpan/report/4704cc42-7e96-11e4-aaa8-
d1ac52b25338

It depends on Type​::Tiny, which fails assertions added by the
multideref code​:

Assertion failed​: (!(o->op_flags & ~(OPf_WANT|OPf_KIDS|OPf_MOD))),
function S_maybe_multideref, file op.c, line 12520.

Type​::Tiny was fixed by 1553b1d, so I was able to test Statocles, which passes if I apply the aliased patch from <https://rt.cpan.org/Ticket/Display.html?id=100359>.  So there is nothing more I can do here.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Dec 13, 2014

From @andk

Also affected on the detour to O/OV/OVID/aliased-0.31.tar.gz​:
FLORA/MooseX-MultiMethods-0.10.tar.gz

But only if MooseX​::Declare and Test​::NoWarnings are installed. In this
case, when the patch https://rt.cpan.org/Ticket/Display.html?id=100359
is applied, MooseX-MultiMethods-0.10 passes its tests again.

--
andreas

@p5pRT
Copy link
Author

p5pRT commented Apr 8, 2015

From @tonycoz

Out of the dists mentioned so far, the following haven't had fixes released​:

HKOBA/YATT-Lite-0.0.9.tar.gz
HKOBA/YATT-v0.0.8.tar.gz - https://rt.cpan.org/Ticket/Display.html?id=95311 - author said he'd fix both dists in this ticket

ABW/Badger-0.09.tar.gz - https​://rt.cpan.org/Ticket/Display.html?id=99200 - fixed in git at https://github.com/abw/Badger

SAMV/Net-SSLeay-OO-0.02.tar.gz - https://rt.cpan.org/Ticket/Display.html?id=100364 - no response to ticket since November

JSWARTZ/Mason-2.22.tar.gz - https://rt.cpan.org/Ticket/Display.html?id=100507 - has a patch, has had a few recent releases without this being fixed

The following appears to be unrelated to sub () {$var}​:

MONS/XML-Hash-LX-0.0603.tar.gz - failure seems to be caused by hash randomization

The following have been fixed​:

BAREFOOT/Debuggit-2.05.tar.gz - fixed in 2.06

OVID/aliased-0.31.tar.gz - fixed in 0.33

The following have been fixed by the fix to aliased​:

PREACTION/Statocles-0.026.tar.gz

FLORA/MooseX-MultiMethods-0.10.tar.gz

@p5pRT
Copy link
Author

p5pRT commented May 6, 2015

From @iabyn

On Tue, Apr 07, 2015 at 05​:19​:51PM -0700, Tony Cook via RT wrote​:

Out of the dists mentioned so far, the following haven't had fixes released​:

HKOBA/YATT-Lite-0.0.9.tar.gz
HKOBA/YATT-v0.0.8.tar.gz - https://rt.cpan.org/Ticket/Display.html?id=95311 - author said he'd fix both dists in this ticket

There have since been releases of both these modules that claim to be
fixed.

ABW/Badger-0.09.tar.gz -
https://rt.cpan.org/Ticket/Display.html?id=99200 - fixed in git at
https://github.com/abw/Badger

Still no new release

SAMV/Net-SSLeay-OO-0.02.tar.gz - https://rt.cpan.org/Ticket/Display.html?id=100364 - no response to ticket since November

No new release since 2009.

JSWARTZ/Mason-2.22.tar.gz -
https://rt.cpan.org/Ticket/Display.html?id=100507 - has a patch, has had
a few recent releases without this being fixed

Still unfixed. I've just emailed the author asking if he's aware of the
ticket.

My feeling is that this ticket no longer needs to be blocker.

--
Dave's first rule of Opera​:
If something needs saying, say it​: don't warble it.

@p5pRT
Copy link
Author

p5pRT commented May 8, 2015

From @rjbs

Used to update Known Problems list.

--
rjbs

@p5pRT p5pRT closed this as completed May 8, 2015
@p5pRT
Copy link
Author

p5pRT commented May 8, 2015

@rjbs - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant