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

Makefile does not test for compiler availability #139

Open
rwfranks opened this issue Oct 27, 2024 · 2 comments
Open

Makefile does not test for compiler availability #139

rwfranks opened this issue Oct 27, 2024 · 2 comments

Comments

@rwfranks
Copy link

Transcribed verbatim from CPAN RT#74705, warts and all.

Sun Feb 05 14:26:01 2012 ribasushi [...] leporine.io - Ticket created
Subject: Makefile does not test for compiler availability

As already mentioned in
http://search.cpan.org/~timb/DBI-1.617/lib/DBI/PurePerl.pm#INSTALLATION
the Make3file should simply skip all xs code building if a compiler is
not available. A well working implementation for such a check can be
found here:
https://metacpan.org/source/ADAMK/List-MoreUtils-0.33/Makefile.PL#L81

Sun Feb 05 14:47:29 2012 ribasushi [...] leporine.io - Dependency by #74706: [Meta] Necessary milestones for fat-packable DBIx::Class added

@rwfranks
Copy link
Author

Tue Feb 21 22:12:26 2012 bitcard.org [...] perl.tgape.org - Correspondence added
From: [email protected]

On Sun Feb 05 14:26:01 2012, RIBASUSHI wrote:

As already mentioned in
http://search.cpan.org/~timb/DBI-1.617/lib/DBI/PurePerl.pm#INSTALLATION
the Make3file should simply skip all xs code building if a compiler is
not available. A well working implementation for such a check can be
found here:
https://metacpan.org/source/ADAMK/List-MoreUtils-0.33/Makefile.PL#L81

This would also require a code update to the perl modules so that they
eval bootstrapping and fail over to loading pure perl even if the
DBI_PUREPERL environment variable is not set. I'm not saying that's a
big obstacle, just that it would need to be done as well.

Tue Feb 21 22:12:27 2012 The RT System itself - Status changed from 'new' to 'open'

@rwfranks
Copy link
Author

Mon Nov 19 18:16:27 2012 TIMB [...] cpan.org - Correspondence added

This isn't a priority for me. I'd happily take a patch though.

Mon Nov 19 18:16:28 2012 TIMB [...] cpan.org - Status changed from 'open' to 'stalled'

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