Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 618838

Summary: media-sound/abcde - add USE=curl to make wget optional
Product: Gentoo Linux Reporter: Scall <scall>
Component: Current packagesAssignee: Gentoo Sound Team <sound>
Status: UNCONFIRMED ---    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Scall 2017-05-18 15:27:49 UTC
The abcde ebuild should have a "curl" USE flag to optionally make the package depend on net-misc/curl instead of net-misc/wget (since users may want to use curl instead of wget by setting the HTTPGET and HTTPGETOPTS vars in ~/.abcde.conf).

Like so:
!curl? ( net-misc/wget )
curl? ( net-misc/curl )
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2017-05-18 15:43:02 UTC
(In reply to Scall from comment #0)
> The abcde ebuild should have a "curl" USE flag to optionally make the
> package depend on net-misc/curl instead of net-misc/wget (since users may
> want to use curl instead of wget by setting the HTTPGET and HTTPGETOPTS vars
> in ~/.abcde.conf).
> 
> Like so:
> !curl? ( net-misc/wget )
> curl? ( net-misc/curl )

If you can set HTTPGET to something other than wget, why would you restrict it to curl?

It looks like the net-misc/wget dependency should simply be removed.
Comment 2 Scall 2017-05-18 17:12:37 UTC
(In reply to Jeroen Roovers from comment #1)
> If you can set HTTPGET to something other than wget, why would you restrict
> it to curl?
> 
> It looks like the net-misc/wget dependency should simply be removed.


Yeah, removing net-misc/wget as dependency is probably better, and perhaps adding an "ewarn" like this example:
An HTTP retrieval program like wget, fetch or curl, is required to retrieve CDDB information about the CD and configurable through "HTTPGET". Alternatively, abcde-musicbrainz-tool can be used by setting "CDDBMETHOD=musicbrainz".