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

steal all outstanding features from Module::Manifest #11

Open
karenetheridge opened this issue Feb 8, 2015 · 2 comments
Open

steal all outstanding features from Module::Manifest #11

karenetheridge opened this issue Feb 8, 2015 · 2 comments

Comments

@karenetheridge
Copy link
Member

Module::Manifest was written to check existing MANIFEST files from anywhere in the filesystem -- see https://metacpan.org/pod/Module::Manifest#Comparison-to-ExtUtil::Manifest

However Module::Manifest does not handle #! directives, which therefore causes failures with everything using it downstream (e.g. Test::DistManifest). Let's try to deprecate Module::Manifest, which means adding features for everything it does that EUM cannot.

@Leont
Copy link
Member

Leont commented Feb 9, 2015

Wouldn't it be possible to just reimplement EU::M on top of M::M? (or reimplement it for all I care) EU::M's API and interface are both stretched far beyond their original requirements, it's starting to look like a duct-tape castle.

@karenetheridge
Copy link
Member Author

Yes, I'm really coming around to the view that a full rewrite is going to have to happen.

@kentnl is currently renovating the test suite, which is great as once we know we've got some reasonable tests in place, we can start hacking with less fear of breaking something (at least something that we were always testing before. Untested regressions will probably happen too, so we'll need to be writing more tests as well.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants