Bug 207067

Summary:	rox-extra/archive - fix dependencies
Product:	Gentoo Linux	Reporter:	Jakub Moc (RETIRED) <jakub>
Component:	New packages	Assignee:	Rox Herd (OBSOLETE) <rox+disabled>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	High
Version:	2007.0
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	archive-2.1-r3.ebuild.diff

Description Jakub Moc (RETIRED) gentoo-dev

2008-01-22 18:13:29 UTC

This pulls in tons for redundant deps due to missing || in RDEPEND. Patch attached.

Comment 1 Jakub Moc (RETIRED) gentoo-dev

2008-01-22 18:23:57 UTC

Created attachment 141594 [details, diff]
archive-2.1-r3.ebuild.diff

- add || for zip/rpm USE flags
- remove USE=bzip2, well you couldn't even unpack this without bzip2 :) 
- Removed virtual/gzip and app-arch/tar from RDEPEND, those are part of basic system and always installed.

Comment 2 Jim Ramsay (lack) (RETIRED) gentoo-dev

2008-01-22 19:33:50 UTC

Unfortunately these are not redundant dependencies:

- RPM support requires both 'rpm2cpio' from app-arch/rpm and then app-arch/cpio to finish unpacking the result.

- ZIP support uses app-arch/unzip to unzip, and app-arch/zip to create zip files.

Granted, RPM could (and I argue, should) use 'rpm2targz' instead, but I haven't bothered to write a patch that does this yet, since I don't use the RPM support.

Though I take your points about the USE flag and system dependencies, which I will clean up.

Comment 3 Jim Ramsay (lack) (RETIRED) gentoo-dev

2009-05-04 02:59:03 UTC

As fixed as it can be, anyway :)