Bug 136498 - (perl-Archive-Tar) Outdated version and IO::Zlib
Summary: (perl-Archive-Tar) Outdated version and IO::Zlib
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-Archive-Tar
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Warren Togami
QA Contact:
URL:
Whiteboard:
: 122736 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2004-10-20 15:47 UTC by Jose Pedro Oliveira
Modified: 2007-11-30 22:10 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-04-26 01:39:33 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Update and specfile cleanup (3.32 KB, patch)
2005-04-03 13:00 UTC, Jose Pedro Oliveira
no flags Details | Diff

Description Jose Pedro Oliveira 2004-10-20 15:47:07 UTC
Description of problem:
perl-Archive-Tar should be updated to the latest CPAN version (1.10).
This new version also requires the non core module IO::Zlib
(Archive::Tar 1.08 only recommended the installation of IO::Zlib).

Version-Release number of selected component (if applicable):
perl-Archive-Tar-1.08-3.src.rpm

Additional info:
The module IO::Zlib already exists in the fedora.us mirrors
for RH9, FC1, and FC2 (perl-IO-Zlib-1.04-*).

Comment 1 Warren Togami 2005-03-31 23:05:58 UTC
*** Bug 122736 has been marked as a duplicate of this bug. ***

Comment 2 Warren Togami 2005-03-31 23:07:49 UTC
http://cvs.fedora.redhat.com/viewcvs/devel/perl-IO-Zlib/?root=extras
Someone want to update this old fedora.us version for FC4 inclusion?
Clean up perl-Archive-Tar.spec while you're at it too.  Attach unidiffs here please.



Comment 3 Jose Pedro Oliveira 2005-04-01 11:04:24 UTC
Just updated the devel branch of perl-IO-Zlib.

Comment 4 Warren Togami 2005-04-01 22:16:56 UTC
waiting on sopwith to add perl-IO-Zlib first

Comment 5 Jose Pedro Oliveira 2005-04-02 00:13:06 UTC
Warren,

Archive::Tar v1.23 also requires perl-IO-String (available in Extras).

Source: See version 1.20 changelog entry in
http://search.cpan.org/src/KANE/Archive-Tar-1.23/README

I'm also finishing a specfile for Archive::Tar 1.23 (still have to run a couple
more tests).

Comment 6 Warren Togami 2005-04-02 00:18:01 UTC
Thanks for the heads up.  Crap...

Comment 7 Ville Skyttä 2005-04-02 07:57:51 UTC
Has anyone checked what in core actually needs perl-Archive-Tar?   IIRC at least
perl-RPM-Specfile does.  perl-RPM-Specfile is IMO definitely a candidate for
moving into Extras, and if there are no other deps to perl-Archive-Tar in Core,
perhaps it'd be easier and better to move it to Extras too, the dependencies are
already there.

Comment 8 Warren Togami 2005-04-02 11:24:42 UTC
#rpm -q --qf '%{name}\t%{size}\n' perl-Archive-Tar perl-RPM-Specfile
perl-Archive-Tar        62286
perl-RPM-Specfile       37509

And the sizes of perl-IO-Zlib and perl-IO-String are even smaller.  Plus notting
said that he wants to keep perl-RPM-Specfile.

Comment 9 Warren Togami 2005-04-02 12:02:32 UTC
(from perl-IO-String.spec)
Note that perl-5.8 and better has built-in support for "in memory"
files, which are set up by passing a reference instead of a filename
to the open() call. The reason for using this module is that it makes
the code backwards compatible with older versions of Perl.

Doesn't this indicate that we really don't need this?  perl-Archive-Tar is hard
coded to always need perl-IO-String now, or is this something we can
filter-requires like we do for Win32?


Comment 10 Ville Skyttä 2005-04-02 15:54:19 UTC
http://search.cpan.org/src/KANE/Archive-Tar-1.23/lib/Archive/Tar.pm
[...]
use IO::String;

So yes, it is a "hard" dependency, and cannot be left out unless someone patches
Archive::Tar not to use it.

Comment 11 Jose Pedro Oliveira 2005-04-03 13:00:37 UTC
Created attachment 112635 [details]
Update and specfile cleanup

New requirements:
- perl-IO-Zlib (Fedora Extras - devel branch)
- perl-IO-String (Fedora Extras)

Comment 12 Warren Togami 2005-04-03 19:59:25 UTC
Freeze is tomorrow and rel-eng isn't around during the weekend, so it looks like
we need to add these after test2.

Comment 13 Warren Togami 2005-04-11 22:08:43 UTC
done.  make sure those modules are removed from Extras.

Comment 14 Jose Pedro Oliveira 2005-04-12 16:27:04 UTC
Warren,

The updated version of perl-Archive-Tar hasn't appeared in Rawhide
(only the new requirements have appeared: perl-IO-String and perl-IO-Zlib).

Rawhide: perl-Archive-Tar 1.08
Expected: perl-Archive-Tar 1.23

Shall I re-open this tickect?

Comment 15 Warren Togami 2005-04-13 00:44:18 UTC
hmmm, some kind of buildsystem problem

Comment 16 Warren Togami 2005-04-13 23:43:31 UTC
-BuildRequires:  perl(IO::String), perl(IO::Zlib)
+#BuildRequires:  perl(IO::String), perl(IO::Zlib)
+# beehive was filtering '(' deps by accident
+# temporary workaround, remove this in a few days
+BuildRequires: perl-IO-String, perl-IO-Zlib

ping me in a few days to remind me, keeping bug open until then

Comment 17 Jose Pedro Oliveira 2005-04-26 03:38:24 UTC
Warren,

Didn't you forgot to delete the line:
   BuildRequires: perl-IO-String, perl-IO-Zlib

Reference:
https://www.redhat.com/archives/fedora-cvs-commits/2005-April/msg01276.html

Comment 18 Warren Togami 2005-04-26 03:45:32 UTC
oops!  thanks for your attention to detail.



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