Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Text-Aspell - spell check interface for perl Alias: perl-Text-Aspell https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235895 cweyl@xxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cweyl@xxxxxxxxxxxxxxx Alias| |perl-Text-Aspell ------- Additional Comments From cweyl@xxxxxxxxxxxxxxx 2007-04-15 19:11 EST ------- A couple comments -- *the canonical way to indicate requirements against perl modules is to cite the perl module itself, rather than the package (e.g. perl(Test::Pod) vs perl-Test-Pod) *due to the impending-maybe-sometime-who-knows perl/devel split, there are a couple deps that have to be listed as explicit BR's: perl(ExtUtils::MakeMaker), perl(Test::More) * typically, the license is written as "GPL or Artistic", rathern than "Artistic or GPL" (don't ask me -- just convention, really) *Requires(check) is neat, but doesn't work correctly. BuildRequires(check) just might (or just plain BuildRequires). -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review