Bug 226240 - Merge Review: perl-Archive-Zip
Summary: Merge Review: perl-Archive-Zip
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Steven Pritchard
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-01-31 20:21 UTC by Nobody's working on this, feel free to take it
Modified: 2011-01-19 09:39 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-06-28 03:33:34 UTC
Type: ---
Embargoed:
steve: fedora-review+


Attachments (Terms of Use)
spec patch (2.37 KB, patch)
2007-03-20 21:27 UTC, Steven Pritchard
no flags Details | Diff

Description Nobody's working on this, feel free to take it 2007-01-31 20:21:55 UTC
Fedora Merge Review: perl-Archive-Zip

http://cvs.fedora.redhat.com/viewcvs/devel/perl-Archive-Zip/
Initial Owner: rnorwood

Comment 1 Steven Pritchard 2007-03-20 21:27:51 UTC
Created attachment 150536 [details]
spec patch

- Update to 1.18.
- Fix find option order.
- Use fixperms macro instead of our own chmod incantation.
- Remove check macro cruft.
- Update build dependencies.
- Package LICENSE.
- BR unzip, zip for better test coverage.

Comment 2 Steven Pritchard 2007-03-20 21:31:01 UTC
I don't see where any of that are blockers.

APPROVED

Comment 3 Steven Pritchard 2007-03-20 21:32:37 UTC
Oh, I forgot to mention that the newer version introduces a build dependency on
File::Which, which is in Extras.

Comment 4 Warren Togami 2007-03-20 22:04:30 UTC
It doesn't appear to be any important bug fixes here.  Could we wait on the
version upgrade until after Fedora 7?  (Moving a package now after the feature
freeze is a bit of a pain.)

1.18 Wed 25 Oct 2006 - Adam Kennedy
  - Changing to a production version for final release
  - No other changes of any kind

1.17_05 Tue 19 Sep 2006 - Adam Kennedy
  - Seperated the classes from the main file into seperate packages.
  - Merged the Zip.pod into the main Zip.pm file.
  - Applied default Perl::Tidy to all of the source files, to improve
    the readability and maintainability of the files.
  - Added license in Makefile.PL
  - Added some additional entries to the realclean files

Could you resubmit with a cleaned up spec for the 1.16 version instead?

Comment 5 Steven Pritchard 2007-03-24 21:02:24 UTC
(In reply to comment #4)
> Could you resubmit with a cleaned up spec for the 1.16 version instead?

I'd rather just see the patch applied whenever it is convenient.  It's all minor
cleanup and changes related to the version upgrade.

The patch can't really be applied until after the merge anyway given the new
dependency on File::Which.

Comment 6 Steven Pritchard 2007-06-27 22:52:21 UTC
It's after the merge...  Ignoring the part about updating to the new version, 
my patch would still apply (and give better test coverage).

And then this bug could be closed too.  :-)

Comment 7 Robin Norwood 2007-06-28 03:33:34 UTC
Done - perl-Archive-Zip-1_20-2_fc8

Comment 8 Stepan Kasal 2008-11-03 16:28:28 UTC
Package Change Request
======================
Package Name: perl-Archive-Zip
New Branch: EL-5
Owner: skasal

Comment 9 Stepan Kasal 2008-11-03 17:30:38 UTC
oops, I withdraw the request from comment #8, the package is going to appear in RHEL-5. Thanks.

Comment 10 manuel wolfshant 2008-11-08 03:14:24 UTC
If/when it will appear in RHEL-5 (that is at least 3 months from now) it should have a newer e-v-r than what EPEL provides and everybody will be happy. Meanwhile we have a bunch of packages in EPEL-5 which cannot be installed due to this package not being available.

Comment 11 Mathieu Bridon 2011-01-19 09:10:42 UTC
This package is in EPEL5, but not in EPEL6. Is there any reason for that?

I would like to see this package in EPEL6, and I am willing to help co-maintain it.

Comment 12 Mathieu Bridon 2011-01-19 09:39:53 UTC
Nevermind, I just saw it's in the RHEL6 Client repository (I was only looking at the Server one).

Sorry for the spam.


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