Summary: | media-gfx/sane-backends-1.0.23 should set "+threads" as it's required by default enabled backends | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Pacho Ramos <pacho> |
Component: | Current packages | Assignee: | Patrick Kursawe (RETIRED) <phosphan> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | nikoli |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://bugs.gentoo.org/show_bug.cgi?id=868336 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | do not enable mustek_usb2 & kvs40xx backends by default |
Description
Pacho Ramos
2013-01-13 17:25:06 UTC
I picked a different solution: Default disabled these two backends if threads is not in use. I think it is better to first offer what they can get without changing their USE flags. Thanks for the hint! Seems to not work as expected : even with the *threads* useflag enabled, the two backend are default-disabled (maybe I'm wrong, but I'm not sure that you can do 'if use' outside of a phase function) : > $ emerge -pv sane-backends > > These are the packages that would be merged, in order: > > Calculating dependencies... done! > [ebuild R ] media-gfx/sane-backends-1.0.23 USE="threads (...)" > SANE_BACKENDS="(...) -kvs40xx* -mustek_usb2* -pnm" 0 kB > > Total: 1 package (1 reinstall), Size of downloads: 0 kB Oopsie. Looks like you are right. But maybe it's still better than before - at least you can emerge it with default flags. Pondering about a solution, suggestions are welcome. Why not just enable use threads by default? And remove 'mySign' changes. Created attachment 337888 [details, diff]
do not enable mustek_usb2 & kvs40xx backends by default
You should simply disable these two backends by default (see patch).
You should not try to force the user configuration. If the user wants one of these two backends, portage will then inform him that the *thread* useflag is required. Anyway a lot of packages already do this in this way.
Sounds reasonable. In CVS, thanks for the support. |