Skip to content

Remove PERL_CREATE_GVSV#24177

Open
xenu wants to merge 1 commit intoPerl:bleadfrom
xenu:xenu/remove_create_gvsv
Open

Remove PERL_CREATE_GVSV#24177
xenu wants to merge 1 commit intoPerl:bleadfrom
xenu:xenu/remove_create_gvsv

Conversation

@xenu
Copy link
Member

@xenu xenu commented Feb 7, 2026

PERL_DONT_CREATE_GVSV has been the default for more than 20 years now (since bdf3085).

There's no real reason to enable PERL_CREATE_GVSV and it's almost undocumented, perlapi mentions it once in passing. It doesn't seem that any code on CPAN checks for PERL_CREATE_GVSV or PERL_DONT_CREATE_GVSV.

This set of changes does not require a perldelta entry.

PERL_DONT_CREATE_GVSV has been the default for more than 20 years now
(since bdf3085).

There's no real reason to enable PERL_CREATE_GVSV and it's almost
undocumented, perlapi mentions it once in passing. It doesn't seem that
any code on CPAN checks for PERL_CREATE_GVSV or PERL_DONT_CREATE_GVSV.
@xenu xenu force-pushed the xenu/remove_create_gvsv branch from a93754b to 0c11533 Compare February 7, 2026 20:45
@jkeenan
Copy link
Contributor

jkeenan commented Feb 7, 2026

It doesn't seem that any code on CPAN checks for PERL_CREATE_GVSV or PERL_DONT_CREATE_GVSV.

What's the best way for someone to confirm that?

@mauke
Copy link
Contributor

mauke commented Feb 7, 2026

I searched https://grep.metacpan.org for PERL_(?:DONT_)?CREATE_GVSV and found (apart from perl itself and boilerplate like ppport.h) only one match: Method::Lexical by CHOCOLATE. However, that module seems to have been deleted from CPAN.

@xenu
Copy link
Member Author

xenu commented Feb 7, 2026

What's the best way for someone to confirm that?

By grepping, for example using grep.cpanauthors.org.

Keep in mind in queries like this, there always will be lots of results from the copies of PPPort, these are irrelevant. One way to avoid it is to limit the searches to XS files.

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

Successfully merging this pull request may close these issues.

3 participants