Bug 673172

Summary: Review Request: perl-Perl-Critic-Deprecated - Perl::Critic policies which have been superseded by others
Product: [Fedora] Fedora Reporter: Petr Pisar <ppisar>
Component: Package ReviewAssignee: Marcela Mašláňová <mmaslano>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting, rc040203
Target Milestone: ---Flags: mmaslano: fedora-review+
j: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: perl-Perl-Critic-Deprecated-1.108-1.fc16 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-03-07 15:51:31 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Petr Pisar 2011-01-27 16:10:51 UTC
Spec URL: http://ppisar.fedorapeople.org/perl-Perl-Critic-Deprecated/perl-Perl-Critic-Deprecated.spec
SRPM URL: http://ppisar.fedorapeople.org/perl-Perl-Critic-Deprecated/perl-Perl-Critic-Deprecated-1.108-1.fc15.src.rpm
Description: 
The included policies are:
  - Write "$my_variable = 42" instead of "$MyVariable = 42".
  - Write "sub my_function{}" instead of "sub MyFunction{}".

Comment 1 Marcela Mašláňová 2011-02-01 15:46:26 UTC
- rpmlint OK
- package must be named according to Guidelines OK
- spec file name must match the base package %{name} OK
- package must meet the Packaging Guidelines OK
- package must be licensed with Fedora approved license OK
- license field must match actual license OK
- text of the license in its own file must be included in %doc OK
- sources must match the upstream source OK
- package MUST successfully compile and build OK
 http://koji.fedoraproject.org/koji/taskinfo?taskID=2754775
- architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK
- build dependencies must be listed in BuildRequires OK
- handle locales properly with %find_lang macro OK
- shared library files must call ldconfig in %post(un) OK
- packages must NOT bundle system libraries OK
- package must own all directories that it creates OK
- permissions on files must be set properly OK
- package must consistently use macros OK
- package must contain code, or permissable content OK
- large documentation must go in a -doc OK
- %doc must not affect the runtime of the application OK
- header files must be in a -devel package OK
- static libraries must be in a -static package OK
- library files that end in .so (without suffix) must go in a -devel OK
- devel package usually require base package OK
- packages must NOT contain any .la libtool archives OK
- GUI applications must include a %{name}.desktop file OK
- packages must not own files or directories already owned by other packages OK

rpm -qp --provides ~/Downloads/perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch.rpm 
perl(Perl::Critic::Deprecated) = 1.108
perl(Perl::Critic::Policy::NamingConventions::ProhibitMixedCaseSubs) = 1.108
perl(Perl::Critic::Policy::NamingConventions::ProhibitMixedCaseVars) = 1.108
perl(Perl::Critic::Utils::PPIRegexp) = 1.108
perl-Perl-Critic-Deprecated = 1.108-1.fc15

rpm -qp --requires ~/Downloads/perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch.rpm 
perl(:MODULE_COMPAT_5.12.3)  
perl(Perl::Critic::Policy) >= 1.094
perl(Perl::Critic::Utils) >= 1.094
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(CompressedFileNames) <= 3.0.4-1
perl >= 0:5.006001
perl(base)  
perl(Carp)  
perl(English)  
perl(Exporter)  
perl(PPI::Node)  
perl(Readonly)  
perl(strict)  
perl(warnings)  
rpmlib(PayloadIsXz) <= 5.2-1

ACCEPT

Comment 2 Marcela Mašláňová 2011-02-01 15:55:36 UTC
rpm -i /home/marca/Downloads/perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch.rpm 
        file /usr/share/man/man3/Perl::Critic::Utils::PPIRegexp.3pm.gz from install of perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch conflicts with file from package perl-Perl-Critic-1.106-1.fc14.noarch
        file /usr/share/perl5/Perl/Critic/Utils/PPIRegexp.pm from install of perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch conflicts with file from package perl-Perl-Critic-1.106-1.fc14.noarch

You should solve this issue before pushing into Fedora.

Comment 3 Ralf Corsepius 2011-02-01 16:02:56 UTC
OK, Marcela, seemingly you are not interested in discussing the issues 
@redhat.cz submitted packages are introducing and seem to prefer to implement facts, instead:

This package does not install to vendor_dir:
  %{__perl} Build.PL installdirs=core

I guess, I don't have to mention what I think of this?

Comment 4 Petr Pisar 2011-02-01 17:20:58 UTC
(In reply to comment #2)
> rpm -i
> /home/marca/Downloads/perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch.rpm 
>         file /usr/share/man/man3/Perl::Critic::Utils::PPIRegexp.3pm.gz from
> install of perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch conflicts with file
> from package perl-Perl-Critic-1.106-1.fc14.noarch
>         file /usr/share/perl5/Perl/Critic/Utils/PPIRegexp.pm from install of
> perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch conflicts with file from
> package perl-Perl-Critic-1.106-1.fc14.noarch
> 
> You should solve this issue before pushing into Fedora.
You have out-dated machine probably as current
perl-Perl-Critic-1.111-1.fc15.noarch does not contain the module (because it's---deprecated). I'm targeting perl-Perl-Critic-Deprecated to F15 only, so I guess there is no problem.

Comment 5 Marcela Mašláňová 2011-03-02 13:08:10 UTC
(In reply to comment #4)
> (In reply to comment #2)
> > rpm -i
> > /home/marca/Downloads/perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch.rpm 
> >         file /usr/share/man/man3/Perl::Critic::Utils::PPIRegexp.3pm.gz from
> > install of perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch conflicts with file
> > from package perl-Perl-Critic-1.106-1.fc14.noarch
> >         file /usr/share/perl5/Perl/Critic/Utils/PPIRegexp.pm from install of
> > perl-Perl-Critic-Deprecated-1.108-1.fc15.noarch conflicts with file from
> > package perl-Perl-Critic-1.106-1.fc14.noarch
> > 
> > You should solve this issue before pushing into Fedora.
> You have out-dated machine probably as current
> perl-Perl-Critic-1.111-1.fc15.noarch does not contain the module (because
> it's---deprecated). I'm targeting perl-Perl-Critic-Deprecated to F15 only, so I
> guess there is no problem.

It's working with current perl-Perl-Critic.

Also could you change core back to vendor? Thank you.

Comment 7 Petr Pisar 2011-03-07 13:38:01 UTC
New Package SCM Request
=======================
Package Name: perl-Perl-Critic-Deprecated
Short Description: Perl::Critic policies which have been superseded by others
Owners: ppisar, mmaslano, psabata
Branches: 
InitialCC: perl-sig

Comment 8 Jason Tibbitts 2011-03-07 15:25:57 UTC
Git done (by process-git-requests).

Comment 9 Petr Pisar 2011-03-07 15:51:31 UTC
Thank you for review and repository.