Spec URL: http://washington.kelkoo.net/fedora/SPECS/perl-Linux-Pid.spec SRPM URL: http://washington.kelkoo.net/fedora/SRPMS/perl-Linux-Pid-0.04-1.fc8.src.rpm Description: Linux::Pid gets the native PID and the PPID on Linux. It's useful with multithreaded programs. Linux's C library returns different values of the PID and the PPID from different threads. This module forces Perl to call the underlying C functions getpid() and getppid().
new version, fix missing BR : http://washington.kelkoo.net/fedora/SPECS/perl-Linux-Pid.spec http://washington.kelkoo.net/fedora/SRPMS/perl-Linux-Pid-0.04-2.fc8.src.rpm
make test should execute all tests successfully But here I see that t/threaded....ok 2/9 skipped: thread model is NPTL 2.7.90 All tests successful, 2 subtests skipped.
I believe this is regular behavior. The tests are skipped, not failed.
I think only Author/Developer tests that are skipped are ok. But all tests should be executed successfully.
Not sure what you call 'Author/Developer tests'. Anyway, I'm asking some perl devs here to take a look at the issue.
This test is OK. It is checking the runtime environment for a specific pthread model and skips some subtests if it doesn't find the pthread model this it wants to test the behavior of. What is not OK with this package's spec (MUSTFIX): 1. %check ||: We want to see breakdowns in testsuites => remove the ||: 2. the "BuildRequires:" BuildRequires: perl(ExtUtils::MakeMaker) BuildRequires: perl(Test::More) seems sufficient. All others seem superfluous to me.
thanks Ralf. Xavier, Can you please provide new updates SRPM that fix issues in above comment?
Fixed in : http://washington.kelkoo.net/fedora/SPECS/perl-Linux-Pid.spec http://washington.kelkoo.net/fedora/SRPMS/perl-Linux-Pid-0.04-3.fc8.src.rpm
Review: + package builds in mock (rawhide i386). + rpmlint is silent for SRPM and for RPM. + source files match upstream url 130c4d299f827abf1f2285fddf03fccb Linux-Pid-0.04.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test outputs PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/basic.......ok t/import......ok t/threaded....ok 2/9 skipped: thread model is NPTL 2.7.90 All tests successful, 2 subtests skipped. Files=3, Tests=13, 0 wallclock secs ( 0.08 cusr + 0.03 csys = 0.11 CPU) + Package perl-Linux-Pid-0.04-3.fc9 => Provides: Pid.so perl(Linux::Pid) = 0.04 Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) perl(:MODULE_COMPAT_5.8.8) perl(XSLoader) perl(strict) perl(warnings) APPROVED.
Thx Parag. New Package CVS Request ======================= Package Name: perl-Linux-Pid Short Description: Get the native PID and the PPID on Linux Owners: xavierb Branches: F-8 InitialCC: Cvsextras Commits: yes
cvs done.
Imported and built for devel and F-8.