[Bug 602042] Review Request (rename): perl-Config-MVP-Reader-INI - MVP config reader for .ini files

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=602042

Petr Pisar <ppisar at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #1 from Petr Pisar <ppisar at redhat.com> 2010-06-21 09:48:21 EDT ---
$ rpmlint -i perl-Config-MVP-Reader-INI-2.101460-1.fc14.noarch.rpm
../../SPECS/perl-Config-MVP-Reader-INI.spec 
perl-Config-MVP-Reader-INI.noarch: W: obsolete-not-provided perl-Config-INI-MVP
If a package is obsoleted by a compatible replacement, the obsoleted package
should also be provided in order to not cause unnecessary dependency breakage.
If the obsoleting package is not a compatible replacement for the old one,
leave out the Provides.

1 packages and 1 specfiles checked; 0 errors, 1 warnings.

As perl module name is part of API and the name has changed, reviewed package
is not compatible with old one and thus old name must not be provided.

Provides and obsoletes OK.

Spec files compared each to other:

--- perl-Config-INI-MVP.spec 2010-04-30 14:17:43.000000000 +0200
+++ /home/petr/rpmbuild/SPECS/perl-Config-MVP-Reader-INI.spec 2010-06-09
05:08:13.000000000 +0200
@@ -1,31 +1,37 @@
-Name:           perl-Config-INI-MVP
-Version:        0.024
-Release:        2%{?dist}
-Summary:        Multi-value capable .ini file reader (for plugins)
+Name:           perl-Config-MVP-Reader-INI
+Version:        2.101460
+Release:        1%{?dist}
+Summary:        MVP config reader for .ini files
 License:        GPL+ or Artistic
 Group:          Development/Libraries
-URL:            http://search.cpan.org/dist/Config-INI-MVP/
-Source0:       
http://search.cpan.org/CPAN/authors/id/R/RJ/RJBS/Config-INI-MVP-%{version}.tar.gz
+URL:            http://search.cpan.org/dist/Config-MVP-Reader-INI/
+Source0:       
http://www.cpan.org/authors/id/R/RJ/RJBS/Config-MVP-Reader-INI-%{version}.tar.gz
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 BuildArch:      noarch
 BuildRequires:  perl(Config::INI::Reader)
-BuildRequires:  perl(Config::MVP) >= 0.092060
+BuildRequires:  perl(Config::MVP) >= 2
+BuildRequires:  perl(Config::MVP::Reader)
+BuildRequires:  perl(Config::MVP::Reader::Findable::ByExtension)
 BuildRequires:  perl(ExtUtils::MakeMaker)
-BuildRequires:  perl(Moose) >= 0.65
+BuildRequires:  perl(Moose)
 BuildRequires:  perl(Test::More)
-Requires:       perl(Config::MVP) >= 0.092060
-Requires:       perl(Moose) >= 0.65
+# not automatically detected
+Requires:       perl(Config::INI::Reader)
+Requires:       perl(Config::MVP::Reader)
+Requires:       perl(Config::MVP::Reader::Findable::ByExtension)
 Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo
$version))

+# this replaces perl-Config-INI-MVP-0.024
+Obsoletes:      perl-Config-INI-MVP < 0.025
+
 %{?perl_default_filter}

 %description
-The MVP INI file reader reads INI files, but can handle properties with
-multiple values. The identification of properties that may have multiple
-entries is done by section, on a plugin basis. 
+Config::MVP::Reader::INI reads .ini files containing MVP-style
+configuration.

 %prep
-%setup -q -n Config-INI-MVP-%{version}
+%setup -q -n Config-MVP-Reader-INI-%{version}

 %build
 %{__perl} Makefile.PL INSTALLDIRS=vendor
@@ -49,15 +55,11 @@

 %files
 %defattr(-,root,root,-)
-%doc Changes LICENSE README
+%doc Changes dist.ini eg LICENSE META.json README
 %{perl_vendorlib}/*
 %{_mandir}/man3/*

 %changelog
-* Fri Apr 30 2010 Marcela Maslanova <mmaslano at redhat.com> - 0.024-2
-- Mass rebuild with perl-5.12.0
-
-* Sun Feb 21 2010 Iain Arnell <iarnell at gmail.com> 0.024-1
+* Tue Jun 08 2010 Iain Arnell 2.101460-1
 - Specfile autogenerated by cpanspec 1.78.
-- use perl_default_filter and DESTDIR
-- tweak {build,}requires
+- obsoletes perl-Config-INI-MVP < 0.025


Spec file is in line with other packaging rules. (Beside the version number
looks scary :)

Package rename accepted.

Hint for package builder: Please build the package against dist-f14.
dist-f14-perltest build root has been merged into rawhide and will be removed
from Koji soon.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.