Bug 662951

Summary: Review Request: perl_checker - Verify Perl code
Product: [Fedora] Fedora Reporter: Thierry Vignaud <thierry.vignaud>
Component: Package ReviewAssignee: 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: rawhideCC: 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 10:51:01 EST Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:

Description Thierry Vignaud 2010-12-14 04:12:38 EST
Spec URL: http://kenobi.mandriva.com/~tv/FC/perl_checker.spec
SRPM URL: http://kenobi.mandriva.com/~tv/FC/perl_checker-1.2.11-1.fc14.src.rpm
Description: This is a perl equivalent of clint which is widely used at Mandriva.

This is one of my first package for Fedora (I'm packaging for Mandriva Linux
since 1999), and I'm seeking a sponsor for Fedora.
Comment 1 Xavier Bachelot 2010-12-14 10:37:34 EST
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.
Comment 2 Steve Traylen 2011-01-22 13:31:55 EST
Hi Thierry, 

Are you able to provide new packages based on Xavier's comments. They are all valid.

Steve.
Comment 3 Michael Scherer 2011-11-26 10:51:01 EST
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.