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

Is stable "isa" actually stable in 5.32? #17

Open
JRaspass opened this issue Feb 2, 2023 · 7 comments
Open

Is stable "isa" actually stable in 5.32? #17

JRaspass opened this issue Feb 2, 2023 · 7 comments

Comments

@JRaspass
Copy link

JRaspass commented Feb 2, 2023

Congrats on shipping stable 🎉

I wanted to try it out at $work with Perl 5.32 from Debian stable, this meant the only available stable feature I could use was isa.

However I quickly ran into issues like:

Can't locate object method "isa" via package XXX

I believe this is the exact same issue raised in https://www.effectiveperlprogramming.com/2020/01/use-the-infix-class-instance-operator/

And fixed in Perl/perl5@a85f515.

As such I think isa's Perl version ought to be raised to 5.34.

@Leont
Copy link
Member

Leont commented Feb 3, 2023

Sounds reasonable to me. @rjbs @leonerd what do you think?

@haarg
Copy link
Member

haarg commented Feb 5, 2023

Aren't there other significant bugs with isa even in 5.36?

@leonerd
Copy link
Contributor

leonerd commented Feb 5, 2023

It had a bugfix between 5.32 and 5.34, yes. That fixed a fairly big observable bug in its behaviour. Is it an intended change? Probably not. Does it mean the thing wasn't "stable" at 5.32? I can't say.

You probably don't want to be using isa before 5.34, that's true. But are we going to retroactively declare various other bugs in features that you shouldn't use before later perl versions fixed them? I don't think we're likely to do that.

@leonerd
Copy link
Contributor

leonerd commented Feb 5, 2023

@haarg

Aren't there other significant bugs with isa even in 5.36?

If there are, I don't think I'm aware of any.

@haarg
Copy link
Member

haarg commented Feb 5, 2023

Perl/perl5#20354

@leonerd
Copy link
Contributor

leonerd commented Feb 5, 2023

@haarg Oooh I see. Still present in 5.36 but fixed in blead, so it'll be fine in 5.38.

@tonycoz
Copy link
Member

tonycoz commented Jun 8, 2023

The fix for Perl/perl5#20354 was included in 5.36.1

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

5 participants