Bug 244987

Summary:	games-rpg/nwn-cep needs an OR dependency on app-arch/rar
Product:	Gentoo Linux	Reporter:	drantin <drantin>
Component:	New packages	Assignee:	Denis Dupeyron (RETIRED) <calchan>
Status:	RESOLVED FIXED
Severity:	normal	CC:	games, JakesAddress
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	updated ebuild to depend on app-arch/rar optionally

Description drantin 2008-10-30 17:47:09 UTC

currently it depends on app-arch/unrar when app-arch/rar can do the job as well

Reproducible: Always

Steps to Reproduce:
1. install app-arch/rar
2. try to install games-rpg/nwn-cep with the rar USE flag enabled


Actual Results:  
app-arch/unrar gets pulled when it isn't needed

Expected Results:  
app-arch/unrar does not get pulled

Comment 1 drantin 2008-10-30 17:47:27 UTC

Created attachment 170342 [details]
updated ebuild to depend on app-arch/rar optionally

Comment 2 John Moe 2009-10-28 09:39:50 UTC

Mplayer pulls in unrar-gpl, which I assume should be able to handle the "unrar" dep as well.  Trying to emerge nwn-cep results in a block for me between app-arch/unrar (pulled in by nwn-cep) and app-arch/unrar-gpl (pulled in by mplayer).

So the ebuild should probably read:

DEPENDS="|| (app-arch/rar app-arch/unrar app-arch/unrar-gpl)"

PS: I hope to try it tomorrow after I download the 1.5+ GB of files needed to install NWN...

Comment 3 drantin 2009-10-28 09:47:08 UTC

(In reply to comment #2)
> Mplayer pulls in unrar-gpl, which I assume should be able to handle the "unrar"
> dep as well.  Trying to emerge nwn-cep results in a block for me between
> app-arch/unrar (pulled in by nwn-cep) and app-arch/unrar-gpl (pulled in by
> mplayer).
> 
> So the ebuild should probably read:
> 
> DEPENDS="|| (app-arch/rar app-arch/unrar app-arch/unrar-gpl)"


So that people that don't have any of the three installed will get the gpl version, a better order might be:

DEPENDS="|| ( app-arch/unrar-gpl app-arch/unrar app-arch/rar )"

Comment 4 Pacho Ramos gentoo-dev

2012-03-19 09:25:01 UTC

This ebuild should use unpacker.eclass

Comment 5 Pacho Ramos gentoo-dev

2012-10-07 11:26:14 UTC

+  07 Oct 2012; Pacho Ramos <pacho@gentoo.org> nwn-cep-1.68-r1.ebuild,
+  nwn-cep-2.0.ebuild:
+  Rely on unpack.eclass to pull unrar deps, if other RAR alternatives are found
+  to really fully replace unrar, this needs to be discussed centrally with
+  eclass authors instead of in every ebuild using rar files (#244987 by
+  drantin).
+