Merging App::Cpan

[andk]

Merging App::Cpan

[andk]

brian d foy [email protected] writes:

I'm ready to merge the App::Cpan stuff. Before, you grabbed the module and the
script, but I also have a lot of tests. How do you want me to merge those? I
like having subdirectories of t/ to hold groups of related tests.

I have no special wishes, as long as all tests pass.

Please note that currently I have a backlog of pull requests. I hope to
merge them over the weekend.

andreas

[andk]

All pull requests are now merged or rejected (or both:). I'm about to release a 2.01-TRIAL. Looking forward to your next pull request.

[briandfoy]

I got back to this with what now seems like a simple solution I should have done at first. It's not yet complete so I need some feedback.

• It's in my merge_cpan1 branch in https://github.com/briandfoy/cpanpm/tree/merge_cpan1 . I couldn't see how to do a pull request to a new branch in the origin.
• I've updated lib/App/Cpan.pm from the cpan-script repo, and merged back in the pre-5.008 changes
• I added the tests and prefixed them with 97-*
• I added t/lib_cpan1 and t/test-mirror-cpan1 for the support files for the tests

Some questions before I finish this:

• Are the locations of these files and directories okay? I didn't see where to add them to a no_index. I can create that, I guess.
• Some of the test files require a recent Test::More. What's the minimum I can use? I'll have to go back through them and undo new features, I guess.