Bug 641341 - Review Request: perl-File-Map - Memory mapping made simple and safe
Summary: Review Request: perl-File-Map - Memory mapping made simple and safe
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Petr Pisar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 1547052 (view as bug list)
Depends On: 641339
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-10-08 13:29 UTC by Lubomir Rintel
Modified: 2018-02-20 15:23 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-11-05 18:18:36 UTC
Type: ---
Embargoed:
ppisar: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Lubomir Rintel 2010-10-08 13:29:38 UTC
SPEC: http://v3.sk/~lkundrak/SPECS/perl-File-Map.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/perl-File-Map-0.31-1.el5_5.src.rpm

Description:

File::Map maps files or anonymous memory into perl variables.

Comment 2 Petr Pisar 2010-11-03 16:52:07 UTC
The standalone spec file and the embedded one in SRPM differ. I will use the standalone one for reference.

Source tar ball is original. Ok.

> Patch0:         perl-File-Map-0.31-pod.patch
FIX: This is wrong as the bug is in Test::Pod that has been fixed in 1.41 (http://cpansearch.perl.org/src/DWHEELER/Test-Pod-1.41/Changes). No Fedora is  affected, drop this patch or apply it conditionally on older RHEL distributions (if necessary).

TODO: Submit the perl-File-Map-0.31-tie.patch patches to upstream.

License verified from README and lib/File/Map.pm. Ok.

FIX: BuildRequire perl(IO::Handle) (Build.PL and tests) as it can be dual-lived package in the future <http://search.cpan.org/~gbarr/IO/>.
FIX: BuildRequire perl(Time::Hires) (Build.PL and tests) as it can be dual-lived package in the future <http://search.cpan.org/~jhi/Time-HiRes/>.

All tests pass. Ok.

$ rpmlint perl-File-Map.spec ../SRPMS/perl-File-Map-0.31-2.fc15.src.rpm ../RPMS/x86_64/perl-File-Map-0.31-2.fc15.x86_64.rpm 
perl-File-Map.x86_64: W: private-shared-object-provides /usr/lib64/perl5/vendor_perl/auto/File/Map/Map.so Map.so()(64bit)
perl-File-Map.x86_64: W: private-shared-object-provides /usr/lib64/perl5/vendor_perl/auto/File/Map/Map.so Map.so()(64bit)
perl-File-Map.x86_64: W: spurious-executable-perm /usr/share/doc/perl-File-Map-0.31/examples/fastsearch.pl
perl-File-Map.x86_64: W: doc-file-dependency /usr/share/doc/perl-File-Map-0.31/examples/fastsearch.pl /usr/bin/perl
2 packages and 1 specfiles checked; 0 errors, 4 warnings.

FIX: Remove the private library from provides. You can do that by calling
`%{?perl_default_filter}' macro right before %description section. See <https://fedoraproject.org/wiki/Perl_default_filter>.
FIX: Remove executable bit from fastsearch.pl.

$ rpm -pq -lv ../RPMS/x86_64/perl-File-Map-0.31-2.fc15.x86_64.rpm 
drwxr-xr-x    2 root    root                        0 Nov  3 17:16 /usr/lib64/perl5/vendor_perl/File
-rw-r--r--    1 root    root                    15566 Nov  3 17:16 /usr/lib64/perl5/vendor_perl/File/Map.pm
drwxr-xr-x    2 root    root                        0 Nov  3 17:16 /usr/lib64/perl5/vendor_perl/auto/File
drwxr-xr-x    2 root    root                        0 Nov  3 17:16 /usr/lib64/perl5/vendor_perl/auto/File/Map
-rwxr-xr-x    1 root    root                    28760 Nov  3 17:16 /usr/lib64/perl5/vendor_perl/auto/File/Map/Map.so
drwxr-xr-x    2 root    root                        0 Nov  3 17:16 /usr/share/doc/perl-File-Map-0.31
-rw-r--r--    1 root    root                     4818 Sep 20 17:15 /usr/share/doc/perl-File-Map-0.31/Changes
-rw-r--r--    1 root    root                    17979 Sep 20 17:15 /usr/share/doc/perl-File-Map-0.31/LICENSE
-rw-r--r--    1 root    root                     2292 Sep 20 17:15 /usr/share/doc/perl-File-Map-0.31/README
drwxr-xr-x    2 root    root                        0 Sep 20 17:15 /usr/share/doc/perl-File-Map-0.31/examples
-rwxr-xr-x    1 root    root                      355 Sep 20 17:15 /usr/share/doc/perl-File-Map-0.31/examples/fastsearch.pl
-rw-r--r--    1 root    root                     6899 Nov  3 17:16 /usr/share/man/man3/File::Map.3pm.gz

File layout Ok.
Permissions except the fastsearch.pl Ok.

$ rpm -pq --requires ../RPMS/x86_64/perl-File-Map-0.31-2.fc15.x86_64.rpm |sort |uniq -c
      1 libc.so.6()(64bit)
      1 libc.so.6(GLIBC_2.2.5)(64bit)
      1 libc.so.6(GLIBC_2.3.2)(64bit)
      1 libc.so.6(GLIBC_2.4)(64bit)
      1 perl >= 0:5.008
      1 perl >= 1:5.010
      1 perl(Carp)
      1 perl(Const::Fast)
      1 perl(File::Map)
      1 perl(:MODULE_COMPAT_5.12.2)
      1 perl(PerlIO)
      1 perl(strict)
      1 perl(Sub::Exporter)
      1 perl(warnings)
      1 perl(XSLoader)
      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 rpmlib(VersionedDependencies) <= 3.0.3-1
      1 rtld(GNU_HASH)
      1 /usr/bin/perl
Binary requires Ok.

$ rpm -pq --provides ../RPMS/x86_64/perl-File-Map-0.31-2.fc15.x86_64.rpm |sort |uniq -c
      1 Map.so()(64bit)  
      1 perl(File::Map) = 0.31
      1 perl-File-Map = 0.31-2.fc15
      1 perl-File-Map(x86-64) = 0.31-2.fc15
FIX: Remove the Map.so() as stated in rpmlint.

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

> t/pod.t ........... skipped: Test::Pod 1.22 required for testing POD
FIX: BuildRequire perl(Test::Pod) >= 1.22

Otherwise, package is in line with Fedora and Perl packaging guildelines.

Please correct all `FIX' prefixed lines and submit new spec file.
Resolution: Package NOT APPROVED.

Comment 3 Lubomir Rintel 2010-11-03 18:33:16 UTC
(In reply to comment #2)
> > Patch0:         perl-File-Map-0.31-pod.patch
> FIX: This is wrong as the bug is in Test::Pod that has been fixed in 1.41
> (http://cpansearch.perl.org/src/DWHEELER/Test-Pod-1.41/Changes). No Fedora is 
> affected, drop this patch or apply it conditionally on older RHEL distributions
> (if necessary).

Done.

> TODO: Submit the perl-File-Map-0.31-tie.patch patches to upstream.

Not done. This is a compat patch for older version of Perl, upstream may not be necessarily interested in it. Dropped it from the package instead.

> FIX: BuildRequire perl(IO::Handle) (Build.PL and tests) as it can be dual-lived
> package in the future <http://search.cpan.org/~gbarr/IO/>.
> FIX: BuildRequire perl(Time::Hires) (Build.PL and tests) as it can be
> dual-lived package in the future <http://search.cpan.org/~jhi/Time-HiRes/>.

Done.

> FIX: Remove the private library from provides. You can do that by calling
> `%{?perl_default_filter}' macro right before %description section. See
> <https://fedoraproject.org/wiki/Perl_default_filter>.

Done.

> FIX: Remove executable bit from fastsearch.pl.

Done.

> > t/pod.t ........... skipped: Test::Pod 1.22 required for testing POD
> FIX: BuildRequire perl(Test::Pod) >= 1.22

Done.

SPEC: http://v3.sk/~lkundrak/SPECS/perl-File-Map.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/perl-File-Map-0.31-3.fc14.src.rpm

Comment 4 Petr Pisar 2010-11-04 10:27:37 UTC
Spec file changes:

 --- perl-File-Map.spec	2010-11-04 11:10:18.571439897 +0100
+++ perl-File-Map.spec.1	2010-11-03 19:31:17.000000000 +0100
@@ -1,13 +1,11 @@
 Name:           perl-File-Map
 Version:        0.31
-Release:        2%{?dist}
+Release:        3%{?dist}
 Summary:        Memory mapping made simple and safe
 License:        GPL+ or Artistic
 Group:          Development/Libraries
 URL:            http://search.cpan.org/dist/File-Map/
 Source0:        http://www.cpan.org/authors/id/L/LE/LEONT/File-Map-%{version}.tar.gz
-Patch0:         perl-File-Map-0.31-pod.patch
-Patch1:         perl-File-Map-0.31-tie.patch
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 BuildRequires:  perl >= 0:5.008
 BuildRequires:  perl(Const::Fast)
@@ -18,16 +16,20 @@
 BuildRequires:  perl(Test::More)
 BuildRequires:  perl(Test::Warn)
 BuildRequires:  perl(Test::Pod::Coverage)
+BuildRequires:  perl(Test::Pod) >= 1.22
+BuildRequires:  perl(IO::Handle)
+BuildRequires:  perl(Time::Hires) 
 Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
 
+%{?perl_default_filter}
+
 %description
 File::Map maps files or anonymous memory into perl variables.
 
 
 %prep
 %setup -q -n File-Map-%{version}
-%patch0 -p1
-%patch1 -p1
+chmod -x examples/fastsearch.pl
 
 
 %build
@@ -62,6 +64,10 @@
 
 
 %changelog
+* Wed Nov 03 2010 Lubomir Rintel (GoodData) <lubo.rintel> 0.31-3
+- Drop el5 secific patches
+- Add more BuildRequires (Petr Pisar)
+
 * Mon Oct 11 2010 Lubomir Rintel (GoodData) <lubo.rintel> 0.31-2
 - Fix build on el5
 

> > > Patch0:         perl-File-Map-0.31-pod.patch
> > FIX: This is wrong as the bug is in Test::Pod that has been fixed in 1.41
> > (http://cpansearch.perl.org/src/DWHEELER/Test-Pod-1.41/Changes). No Fedora
> > is affected, drop this patch or apply it conditionally on older RHEL
> > distributions (if necessary).
>
> Done.
Ok.

> > FIX: BuildRequire perl(IO::Handle) (Build.PL and tests) as it can be
> > dual-lived package in the future <http://search.cpan.org/~gbarr/IO/>.
Ok.

> > FIX: BuildRequire perl(Time::Hires) (Build.PL and tests) as it can be
> < dual-lived package in the future <http://search.cpan.org/~jhi/Time-HiRes/>.
> 
> Done.
> +BuildRequires:  perl(Time::Hires)
Fix: Proper name is `perl(Time::HiRes)' (capital `R'). I'm sorry I got you wrong name. Use this one please, otherwise the package cannot be built.

> > FIX: Remove the private library from provides. You can do that by calling
> > `%{?perl_default_filter}' macro right before %description section. See
> > <https://fedoraproject.org/wiki/Perl_default_filter>.
>
> Done.
Ok.

> > FIX: Remove executable bit from fastsearch.pl.
>
> Done.
Ok.

> > > t/pod.t ........... skipped: Test::Pod 1.22 required for testing POD
> > FIX: BuildRequire perl(Test::Pod) >= 1.22
>
> Done.
Ok.

FIX: Package does not build in F15 because of perl(Time::HiRes) typo (http://koji.fedoraproject.org/koji/taskinfo?taskID=2576028).


Please correct all `FIX' previxed notes and submit new spec file.
Resolution: Package NOT APPROVED.

Comment 5 Lubomir Rintel 2010-11-05 09:18:36 UTC
> FIX: Package does not build in F15 because of perl(Time::HiRes) typo
> (http://koji.fedoraproject.org/koji/taskinfo?taskID=2576028).

I would swear I did built it...

SPEC: http://v3.sk/~lkundrak/SPECS/perl-File-Map.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/perl-File-Map-0.31-4.fc14.src.rpm

Comment 6 Petr Pisar 2010-11-05 12:17:27 UTC
Spec file changes:

--- perl-File-Map.spec	2010-11-03 19:31:17.000000000 +0100
+++ perl-File-Map.spec.1	2010-11-05 10:17:13.000000000 +0100
@@ -1,6 +1,6 @@
 Name:           perl-File-Map
 Version:        0.31
-Release:        3%{?dist}
+Release:        4%{?dist}
 Summary:        Memory mapping made simple and safe
 License:        GPL+ or Artistic
 Group:          Development/Libraries
@@ -18,7 +18,7 @@
 BuildRequires:  perl(Test::Pod::Coverage)
 BuildRequires:  perl(Test::Pod) >= 1.22
 BuildRequires:  perl(IO::Handle)
-BuildRequires:  perl(Time::Hires) 
+BuildRequires:  perl(Time::HiRes) 
 Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
 
 %{?perl_default_filter}
@@ -64,6 +64,9 @@
 
 
 %changelog
+* Fri Nov 05 2010 Lubomir Rintel (GoodData) <lubo.rintel> 0.31-4
+- Fix a BR typo
+
 * Wed Nov 03 2010 Lubomir Rintel (GoodData) <lubo.rintel> 0.31-3
 - Drop el5 secific patches
 - Add more BuildRequires (Petr Pisar)


> > +BuildRequires:  perl(Time::Hires)
> Fix: Proper name is `perl(Time::HiRes)' (capital `R'). I'm sorry I got you
> wrong name. Use this one please, otherwise the package cannot be built.
Ok.

> FIX: Package does not build in F15 because of perl(Time::HiRes) typo
> (http://koji.fedoraproject.org/koji/taskinfo?taskID=2576028).
Package builds in F15 (http://koji.fedoraproject.org/koji/taskinfo?taskID=2579040). Ok.


Resolution: Package APPROVED.

Comment 7 Lubomir Rintel 2010-11-05 16:18:15 UTC
New Package SCM Request
=======================
Package Name: perl-File-Map
Short Description: Memory mapping made simple and safe
Owners: lkundrak
Branches: f13 f14 el6

Comment 8 Jason Tibbitts 2010-11-05 17:23:05 UTC
Git done (by process-git-requests).

Comment 9 Lubomir Rintel 2010-11-05 18:18:36 UTC
Thanks.
Imported and built.

Comment 10 Petr Pisar 2018-02-20 15:23:14 UTC
*** Bug 1547052 has been marked as a duplicate of this bug. ***


Note You need to log in before you can comment on or make changes to this bug.