Bugzilla Bug 157879

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

Marked ~hppa.

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

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.

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.

(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?

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

Jakub: done :)

Can anyone provide a sample ace file?

added ~ppc64

Added ~alpha keyword.

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

Not supported, dependency dropped wrt Bug 162890 :)

~sparc'd.

Added ~ia64 keyword

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