Bug 662951
Summary: | Review Request: perl_checker - Verify Perl code | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Thierry Vignaud <thierry.vignaud> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED INSUFFICIENT_DATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | fedora-package-review, misc, notting, steve.traylen, xavier |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2011-11-26 15:51:01 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Thierry Vignaud
2010-12-14 09:12:38 UTC
I'm not a sponsor but from a quick glance, here are a couple comments that will hopefully help. - Don't define version and release twice. Just use plain "Version: 1.2.3", for example. - Source0 must be a full URL. - BuildRequires/Obsolotes/Provides for perl-MDK-Common looks dubious and are probably uneeded for the Fedora package. - Why disable AutoReqProv ? - %defattr should be %defattr(-,root,root,-) - Watch out for potentially unowned files/dirs or missing Requires (%{_datadir}/vim/ftplugin ; %{_sysconfdir}/emacs/site-start.d) - You might want to trim the changelog not too include the full Mandriva history. It is obviously fine to keep a comment about the source of this spec though. Hi Thierry, Are you able to provide new packages based on Xavier's comments. They are all valid. Steve. The url is no longer valid ( afaik, the home was trashed on the server at mandriva ), so i am closing the bug report until a new spec is posted. |