Bug 157879 – Testing request for xfce-extra/xarchiver

• Status: RESOLVED DUPLICATE of bug 175749
• Product: Gentoo Linux

[Description Josh Nichols (RETIRED) 2006-12-11 19:46:08 UTC]

xarchiver is an archiver application for XFCE. It is a new package, so please keyword for your arch, thanks.

[Comment 1 Jeroen Roovers (RETIRED) 2006-12-12 06:24:27 UTC]

Marked ~hppa.

[Comment 2 Jeroen Roovers (RETIRED) 2006-12-13 11:09:56 UTC]

The rar USE flags pulls in app-arch/rar as dependency. I failed to notice this before as repoman (sys-apps/portage-2.1.2_rc3-r4) seems to be b0rked (is it somehow using /etc/portage?).

Anyhow, app-arch/rar is only available for x86 and amd64, so I have changed the ebuild to reflect that. You might want to change that in the near future because the rar USE flag is already in use:

# euses -csv rar
[...]
use.local.desc::media-gfx/comix:rar - Pulls app-arch/unrar for rar file support
use.local.desc::rox-extra/archive:rar - Enable .rar support via rar package
use.local.desc::xfce-extra/xarchiver:rar - Adds support for rar archives

and seems to pull in either rar or unrar depending on the package.

Besides that, nothing in the ebuild suggests that the package actually needs all of the stuff in DEPEND to build. The ebuild does not use any configure options to tell the build system what to do with these dependencies, so to me it seems these are runtime dependencies and they shouldn't be pulled in at any time. You cannot have the build system find these libraries on its own without reflecting that in the control over configure you put into the ebuild.

FWIW, with app-arch/rar not installed, xarchiver will simply fail gracefully and tell the user the appropriate utility to handle for instance *.rar is not installed.

hppa won't mask the rar use flag as it is useful elsewhere in the tree, so instead I would advise looking over the ebuild and fixing it instead, including doing away with all the weird USE flags that only serve to pull in dependencies the package doesn't actually require.


Kind regards,
     JeR

[Comment 3 Jeroen Roovers (RETIRED) 2006-12-13 11:34:15 UTC]

FWIW, for extracting files from .rar archives, app-arch/unrar is good enough, as xarchiver can actually use that, so you might want to default to that in any RDEPEND if you really really really must go that road.

[Comment 4 Peter Weller (RETIRED) 2006-12-13 13:33:34 UTC]

Moved all USE-based deps to RDEPEND, added arj USE flag and USE RDEPEND'd on app-arch/unrar rather than app-arch/rar. Bumped.

[Comment 5 Jakub Moc (RETIRED) 2006-12-15 11:18:32 UTC]

(In reply to comment #4)
> Moved all USE-based deps to RDEPEND, added arj USE flag and USE RDEPEND'd on
> app-arch/unrar rather than app-arch/rar. Bumped.

I already have app-arch/rar, thanks but I don't need unrar in addition... Why don't you just stick

rar? ( || ( app-arch/unrar app-arch/rar ) )

in there?

[Comment 6 Jakub Moc (RETIRED) 2006-12-15 11:21:33 UTC]

And same goes for app-arch/{arj,unarj}

[Comment 7 Peter Weller (RETIRED) 2006-12-16 02:02:56 UTC]

Jakub: done :)

[Comment 8 Jason Wever (RETIRED) 2006-12-19 18:27:47 UTC]

Can anyone provide a sample ace file?

[Comment 9 Markus Rothe (RETIRED) 2006-12-21 06:30:48 UTC]

added ~ppc64

[Comment 10 Bryan Østergaard (RETIRED) 2007-01-04 06:22:57 UTC]

Added ~alpha keyword.

[Comment 11 Jakub Moc (RETIRED) 2007-01-22 09:39:06 UTC]

(In reply to comment #8)
> Can anyone provide a sample ace file?

Not supported, dependency dropped wrt Bug 162890 :)

[Comment 12 Gustavo Zacarias (RETIRED) 2007-02-02 19:44:31 UTC]

~sparc'd.

[Comment 13 Raúl Porcel (RETIRED) 2007-03-25 18:16:09 UTC]

Added ~ia64 keyword

[Comment 14 Samuli Suominen (RETIRED) 2007-04-23 18:49:46 UTC]

*** This bug has been marked as a duplicate of bug 175749 ***