Perl-Toolchain-Gang / CPAN-Meta-Requirements

a set of version requirements for a CPAN dist
5 stars 15 forks source link

Descriptive error messages (hopefully an object!) to get which package gives conflicts #24

Closed miyagawa closed 1 year ago

miyagawa commented 9 years ago

Right now when merging two requirements, all I get is a string error saying "illegal requirements: minimum exceeds maximum at ..." with a giant stacktrace.

This isn't really helpful to present to users (using cpanm or Carmel), I'd like to display which requirement for which module gets a conflict. That should actually be doable if I walk down each module individually (because I have access to old and new requirement to add), but when calling ->add_requirements, the error message doesn't give any clue which module has the conflict.

For now like said above, I'll write my own merge wrapper to try to get them, but would be nice if the error message is more descriptive from CMR!

rjbs commented 9 years ago

I've just pushed up a "detailed-errors" branch. Could you let me know what you think? Give it a spin.

miyagawa commented 9 years ago

Looks good :+1:

rjbs commented 8 years ago

This is now in a trial release on the CPAN.

https://metacpan.org/release/RJBS/CPAN-Meta-Requirements-2.137-TRIAL

Leont commented 1 year ago

Closing this as it has been fixed in 2015