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.PL's TEST_REQUIRES requires a modern ExtUtils::MakeMaker #33

Closed
DrHyde opened this issue Nov 7, 2020 · 2 comments
Closed

Makefile.PL's TEST_REQUIRES requires a modern ExtUtils::MakeMaker #33

DrHyde opened this issue Nov 7, 2020 · 2 comments

Comments

@DrHyde
Copy link

DrHyde commented Nov 7, 2020

File::Copy::Recursive, and anything that depends on it, won't build on a fresh install of some old versions of perl, because the EU::MM bundled with them doesn't understand TEST_REQUIRES and so the dependencies on things like Test::File and Test::Warnings won't be spotted. In particular this affects Test::File::ShareDir, which affects DateTime::Locale, which affects DateTime, which affects, well, just about everything.

The fix is to either merge TEST_REQUIRES into PREREQ_PM or, I think, declare the requirement for a modern EU::MM in META.json.

To see this bug in action, search for Unpacking File-Copy-Recursive-0.45.tar.gz here and watch as it fails to build

@tomhukins
Copy link
Contributor

This looks like a duplicate of #27

@DrHyde
Copy link
Author

DrHyde commented Nov 18, 2020

You're right, it is. Naughty me!

@DrHyde DrHyde closed this as completed Nov 18, 2020
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

2 participants