Bug 574267 (perl-Test-Simple)

*** Bug 230798 has been marked as a duplicate of this bug. ***

So, a couple extra thoughts on this one, as it's a little "weird" given its dual-life status.  All of the following directly follow the corresponding bits of perl.spec.

I've directly BR'd perl-devel.

Modules are installed to "perl" (aka core) dirs vs vendor.  Practically, this change has no real impact on F-13+ due to the core/vendor conflation, but it seemed to be a good idea to be consistent.

There's some commented-out bits relating to triggering a full re-test using the core perl test suite, as Iain suggested.  (I have the perl.spec mods in place to break out the test suite into a perl-tests package, but need to get something up and going so I can make sure it actually works as advertised.)

I haven't BR'ed either Test::Pod or Test::Pod::Coverage, as I didn't want to introduce a non-core dependency on a dual-life package.  (I'm open to being convinced otherwise, however :))

Am a bit confused here on F12.

yum provides 'perl(Test::Simple)'

perl-Test-Simple-0.92-87.fc12.x86_64 : Basic utilities for writing tests
Repo        : updates
Matched from:
Other       : perl(Test::Simple)

am I missing something?

Basically -- it's the new process for updating dual-lifed dists.  (e.g. Test-Simple ships both bundled with core Perl, and on the CPAN, so it has two lives).

The key things to keep in mind here:
* Files should be installed to "perl" (aka core) not "vendor"
* The dual-lifed dist should already be broken out into a subpackage of the "perl" package.
* perl_default_filter should be used
* packages must be upgradable by core when the core perl's package is upgraded and includes a newer version of the dual-lifed dist
* The core "perl" package maintainers must have access to update the package, and perl-sig must be on CC.

This is still a very new change to the updating policy...  Discussion is always welcome on the fedora perl mailing list (or here, but the mailing list seems easier for it :))

Thanks for the explanation this makes sense to me. As long as the package
updates cleanly from perl's own to this one it is good.

Review: perl-Test-Simple
Date:   31st March  2010
Mock Build: F14-x86_64 tested.

* COMMENT: rpmlint output
$ rpmlint SPECS/perl-Test-Simple.spec RPMS/noarch/perl-Test-Simple-0.94-1.fc12.noarch.rpm SRPMS/perl-Test-Simple-0.94-1.fc12.src.rpm 
perl-Test-Simple.noarch: E: devel-dependency perl-devel
perl-Test-Simple.noarch: W: spelling-error %description -l en_US perldoc -> periodic, Perl, perforce
perl-Test-Simple.noarch: W: spelling-error %description -l en_US lifed -> filed, life, lied
perl-Test-Simple.src: W: spelling-error %description -l en_US perldoc -> periodic, Perl, perforce
perl-Test-Simple.src: W: spelling-error %description -l en_US lifed -> filed, life, lied

Is the word "lifed" as in this dual-lifed now establised? It's really
not a word.

As for dependecy on perl-devel yes it comes from the main perl package
I see.
* PASS: Named according to the Package Naming Guidelines.
* PASS: spec file name same as  base package %{name}.
* PASS: Packaging Guidelines.
* PASS: Approved license in .spec file.
As perl.
* PASS: License on Source code.
As perl
* PASS: Include LICENSE file or similar if it exist.
Does not exist
* COMMENT: Written in American English.
"lifed"
* PASS: Spec file legible. 
* PASS: Included source must match upstream source.
$ md5sum Test-Simple-0.94.tar.gz ../SOURCES/Test-Simple-0.94.tar.gz 
e4e09d8bf2cc73124152ba2c45c95b5b  Test-Simple-0.94.tar.gz
e4e09d8bf2cc73124152ba2c45c95b5b  ../SOURCES/Test-Simple-0.94.tar.gz
* PASS: Build on one architecture.
* NOTCHECKED: Not building on an architecture must highlighted.
* PASS: Build dependencies must be listed in BuildRequires.
* PASS: Handle locales properly. 
None present
* PASS: ldconfig must be called on shared libs.
None present
* PASS: No bundled copies of system libraries.
None present
* PASS: Package must state why relocatable if relocatable.
Not Relocatable.
* PASS: A package must own all directories that it creates
Standard perl
* PASS:  No duplicate files in %files listings. 
None
* PASS:  Permissions on files must be set properly. %defattr
* PASS:  %clean section contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
* PASS:  Each package must consistently use macros.
* PASS:  The package must contain code, or permissable content.
* PASS:  Large documentation files must go in a -doc subpackage.  
None present.
* PASS:  %doc  must not affect the runtime of the application. 
* PASS:  Header files must be in a -devel package.
* PASS:  Static libraries must be in a -static package.
None present.
* PASS:  Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
None present.
* PASS:  Then library files that end in .so 
None present.
* PASS:  devel packages must require the exact base package
* PASS:  No .la libtool archives
* PASS:  GUI apps should have %{name}.desktop file
No Gui
* PASS:  No files or directories already owned by other packages. 
* PASS:  %install must run rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
* PASS:  All filenames in rpm packages must be valid UTF-8.

Summary:

Just the comment about "lifed".

 APPROVED.

(In reply to comment #5)

> Summary:
> 
> Just the comment about "lifed".

Thanks!  I can't claim the word, but I believe I captured it correctly.

New Package CVS Request
=======================
Package Name: perl-Test-Simple
Short Description: Basic utilities for writing tests
Owners: cweyl
Branches: F-11 F-12 F-13 devel
InitialCC: perl-sig

CVS done (by process-cvs-requests.py).

Thanks for the review! :-)