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

RT-84272 Translate test_requires() to TEST_REQUIRES #11

Open
kevindawson opened this issue Mar 10, 2014 · 1 comment
Open

RT-84272 Translate test_requires() to TEST_REQUIRES #11

kevindawson opened this issue Mar 10, 2014 · 1 comment

Comments

@kevindawson
Copy link
Member

RT-84272: https://rt.cpan.org/Public/Bug/Display.html?id=84272

Since 6.63_03 ExtUtils::MakeMaker accepts TEST_REQUIRES parameters to dump into META spec 2.0. It would be nice if Module::Install can be upgraded to translate test_requires() into that, as the documentation has been written in the first place.

This is my first stab to implement that, while some of the tests are failing because the test was expecting it to be merged with build_requires, or include() logic might have to be updated to reflect the change as well. Which I haven't got around to it yet. https://gist.github.com/miyagawa/5261788

Would be nice if someone can take a look and continue over to make it happen.

@kevindawson
Copy link
Member Author

I have been having a hack around with this :)
see: https://github.com/kevindawson/Module-Install/tree/RT-84272

O what fun the logic is, still coming to terms

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