[Bug 849829] Review Request: perl-Algorithm-SVM - Perl bindings for the libsvm Support Vector Machine library

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--- Comment #1 from Petr Pisar <ppisar@xxxxxxxxxx> ---
Source tar ball is original (verified from
<http://search.cpan.org/CPAN/authors/id/L/LA/LAIRDM/Algorithm-SVM-0.13.tar.gz>,
SHA-256: f72572df107eb45cd2ff6165c183164f38c7da98417d2242e334e6999fb7e422).
Summary verified from lib/Algorithm/SVM.pm. Ok.
Description verified from lib/Algorithm/SVM.pm. Ok.
Description verified from README. Ok.
Patches are Ok.
There is XS code, default BuildArch is Ok.

TODO: You can drop removing empty directories from %install section. This is
default behaviour.

FIX: Build-require `perl(Carp)' for running tests
(lib/Algorithm/SVM/DataSet.pm:5).

TODO: Build-require `perl(Exporter)' which can dual-live in the future
(lib/Algorithm/SVM.pm:8, http://search.cpan.org/~toddr/Exporter/).
TODO: Build-require `perl(AutoLoader)' which can dual-live in the future
(lib/Algorithm/SVM.pm:9, http://search.cpan.org/~smueller/AutoLoader/).
TODO: Build-require `perl(Test)' which can dual-live in the future (test.pl:8,
http://search.cpan.org/~sburke/Test/).

All tests pass. Ok.

$ rpmlint perl-Algorithm-SVM.spec
../SRPMS/perl-Algorithm-SVM-0.13-1.fc19.src.rpm
../RPMS/x86_64/perl-Algorithm-SVM-*
perl-Algorithm-SVM.src: W: spelling-error Summary(en_US) libsvm -> Librium
perl-Algorithm-SVM.src: W: spelling-error %description -l en_US classifcation
-> classification, clarification, ossification
perl-Algorithm-SVM.x86_64: W: spelling-error Summary(en_US) libsvm -> Librium
perl-Algorithm-SVM.x86_64: W: spelling-error %description -l en_US
classifcation -> classification, clarification, ossification
perl-Algorithm-SVM.x86_64: W: private-shared-object-provides
/usr/lib64/perl5/vendor_perl/auto/Algorithm/SVM/SVM.so SVM.so()(64bit)
perl-Algorithm-SVM.x86_64: W: private-shared-object-provides
/usr/lib64/perl5/vendor_perl/auto/Algorithm/SVM/SVM.so SVM.so()(64bit)
3 packages and 1 specfiles checked; 0 errors, 6 warnings.

TODO: Correct typo `classifcation' in description.

FIX: Do not export private library. Insert `%{?perl_default_filter}' code
before %description section
(https://fedoraproject.org/wiki/Perl/Tips#private-shared-object-provides).

$ rpm -q -lv -p ../RPMS/x86_64/perl-Algorithm-SVM-0.13-1.fc19.x86_64.rpm 
drwxr-xr-x    2 root    root                        0 Sep 12 16:08
/usr/lib64/perl5/vendor_perl/Algorithm/SVM
-rw-r--r--    1 root    root                    12972 Jan 22  2008
/usr/lib64/perl5/vendor_perl/Algorithm/SVM.pm
-rw-r--r--    1 root    root                     3934 Jan 15  2006
/usr/lib64/perl5/vendor_perl/Algorithm/SVM/DataSet.pm
drwxr-xr-x    2 root    root                        0 Sep 12 16:08
/usr/lib64/perl5/vendor_perl/auto/Algorithm/SVM
-rwxr-xr-x    1 root    root                    57352 Sep 12 16:08
/usr/lib64/perl5/vendor_perl/auto/Algorithm/SVM/SVM.so
-rw-r--r--    1 root    root                       92 Sep 12 16:08
/usr/lib64/perl5/vendor_perl/auto/Algorithm/SVM/autosplit.ix
drwxr-xr-x    2 root    root                        0 Sep 12 16:09
/usr/share/doc/perl-Algorithm-SVM-0.13
-rw-r--r--    1 root    root                     1864 Jan 22  2008
/usr/share/doc/perl-Algorithm-SVM-0.13/Changes
-rw-r--r--    1 root    root                     4446 Jan 22  2008
/usr/share/doc/perl-Algorithm-SVM-0.13/README
-rw-r--r--    1 root    root                    61530 Dec 23  2005
/usr/share/doc/perl-Algorithm-SVM-0.13/sample.model
-rw-r--r--    1 root    root                    61530 Sep 12 16:09
/usr/share/doc/perl-Algorithm-SVM-0.13/sample.model.1
-rw-r--r--    1 root    root                     4541 Sep 12 16:08
/usr/share/man/man3/Algorithm::SVM.3pm.gz
-rw-r--r--    1 root    root                     2873 Sep 12 16:08
/usr/share/man/man3/Algorithm::SVM::DataSet.3pm.gz
File permissions and layout are Ok.

$ rpm -q --requires -p ../RPMS/x86_64/perl-Algorithm-SVM-0.13-1.fc19.x86_64.rpm
| sort | uniq -c
      1 libc.so.6()(64bit)
      1 libc.so.6(GLIBC_2.14)(64bit)
      1 libc.so.6(GLIBC_2.2.5)(64bit)
      1 libgcc_s.so.1()(64bit)
      1 libgcc_s.so.1(GCC_3.0)(64bit)
      1 libm.so.6()(64bit)
      1 libstdc++.so.6()(64bit)
      1 libstdc++.so.6(CXXABI_1.3)(64bit)
      1 libstdc++.so.6(GLIBCXX_3.4)(64bit)
      1 libsvm.so.2()(64bit)
      1 perl >= 0:5.006
      1 perl(Algorithm::SVM)
      1 perl(AutoLoader)
      1 perl(Carp)
      1 perl(DynaLoader)
      1 perl(Exporter)
      1 perl(:MODULE_COMPAT_5.16.1)
      1 perl(strict)
      1 perl(vars)
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsXz) <= 5.2-1
      1 rtld(GNU_HASH)
Binary requires are Ok.

$ rpm -q --provides -p ../RPMS/x86_64/perl-Algorithm-SVM-0.13-1.fc19.x86_64.rpm
| sort | uniq -c
      1 perl(Algorithm::SVM) = 0.13
      1 perl-Algorithm-SVM = 0.13-1.fc19
      1 perl(Algorithm::SVM::DataSet)
      1 perl-Algorithm-SVM(x86-64) = 0.13-1.fc19
      1 SVM.so()(64bit)
FIX: Do not export private library `SVM.so()'. Insert `%{?perl_default_filter}'
code before %description section
(https://fedoraproject.org/wiki/Perl/Tips#private-shared-object-provides).

$ resolvedeps rawhide ../RPMS/x86_64/perl-Algorithm-SVM-0.13-1.fc19.x86_64.rpm
Binary dependencies resolvable. Ok.

Package builds in F19
(http://koji.fedoraproject.org/koji/taskinfo?taskID=4477851). Ok.

Otherwise the package is in line with Fedora and Perl packaging guidelines.


Please correct all `FIX' issues, consider fixing `TODO' items and provide new
spec file.
Resolution: Package NOT approved.

-- 
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



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]