-
Notifications
You must be signed in to change notification settings - Fork 542
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
ExtUtils-ParseXS can redefine croak_xs_usage #18040
Comments
Fix Perl#18040 Do not redefine croak_xs_usage when already set.
EU::PXS already protects against redefining over the core definition of croak_xs_usage() by checking for |
This would work too, view Dual-Life/Devel-PPPort#196 |
@atoomic , @tonycoz, Are there any more issues to be handled in this ticket? Thank you very much. |
I do not think we should close this ticket, we should merge at least one of these two:
Right now both are stale, I'm not sure what action to do at this stage. |
The #196 change should be applied to blead (blead is upstream, not Dual-Life). It should probably include a comment to indicate why it is the way it is. |
Apply patch already applied to Dual-Life/Devel-PPPort: https://github.com/Dual-Life/Devel-PPPort/pull/196.patch For: #18040
The commit message should be the one from Dual-Life/Devel-PPPort#196 (except referencing #18040). |
Apply patch already applied to Dual-Life/Devel-PPPort: https://github.com/Dual-Life/Devel-PPPort/pull/196.patch This is matching what's done in ExtUtils::ParseXS::Utilities and avoids need to redefine croak_xs_usage later. For: #18040
Applied to blead in 3b75ef9. Closing ticket. |
Apply patch already applied to Dual-Life/Devel-PPPort: https://github.com/Dual-Life/Devel-PPPort/pull/196.patch This is matching what's done in ExtUtils::ParseXS::Utilities and avoids need to redefine croak_xs_usage later. For: Perl#18040
I noticed this issue while compiling Devel-PPPort-3.58 using Perl 5.8.8
ExtUtils::ParseXS::Utilities provides a definition of croak_xs_usage
which can conflict with the definition provided by Devel-PPPort.
It defines
#define croak_xs_usage S_croak_xs_usage
without checking if it's already define or undef it.This leads to warnings like this when generating RealPPPort.c
RealPPPort.c:458:1: warning: "croak_xs_usage" redefined
In truth fixing this issue without publishing a new version of the dual life module
ExtUtils-ParseXS
has a low value, but this does not mean we should not fix this issue in blead.
Note that Devel-PPPort is already using a protection for
#ifndef croak_xs_usage
The text was updated successfully, but these errors were encountered: