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@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #1 from Petr Pisar <ppisar@xxxxxxxxxx> 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@xxxxxxxxxx> - 0.024-2 -- Mass rebuild with perl-5.12.0 - -* Sun Feb 21 2010 Iain Arnell <iarnell@xxxxxxxxx> 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review