Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 556668 - media-tv/kodi: make dbus optional
Summary: media-tv/kodi: make dbus optional
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Gentoo Xbox project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-04 15:29 UTC by Kirill A. Shutemov
Modified: 2015-08-06 02:26 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch poposal (kodi-optional-dbus.patch,1.15 KB, patch)
2015-08-04 15:30 UTC, Kirill A. Shutemov
Details | Diff
Patch poposal v2 (kodi-optional-dbus.patch,1.26 KB, patch)
2015-08-05 09:10 UTC, Kirill A. Shutemov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kirill A. Shutemov 2015-08-04 15:29:24 UTC
Kodi supports build without dbus. Let's make dbus optional in ebuild.

Reproducible: Always
Comment 1 Kirill A. Shutemov 2015-08-04 15:30:20 UTC
Created attachment 408274 [details, diff]
Patch poposal
Comment 2 SpanKY gentoo-dev 2015-08-05 04:20:21 UTC
some of the features in there though require dbus (like upower).  i'm guessing REQUIRED_USE also needs to be updated to list the flags that require dbus ?
Comment 3 Kirill A. Shutemov 2015-08-05 09:10:15 UTC
Created attachment 408322 [details, diff]
Patch poposal v2

Update REQUIRED_USE to indicate dbus dependency of upower and udisk.
Comment 4 SpanKY gentoo-dev 2015-08-05 13:25:14 UTC
should be all set now in the tree; thanks for the report!

Commit message: Make dbus optional
http://sources.gentoo.org/media-tv/kodi/kodi-9999.ebuild?r1=1.17&r2=1.18
Comment 5 Kirill A. Shutemov 2015-08-05 13:49:07 UTC
Hm. Looks like changes to REQUIRED_USE was not applied.
Is it intended?
Comment 6 SpanKY gentoo-dev 2015-08-06 02:24:13 UTC
(In reply to Kirill A. Shutemov from comment #5)

the patch didn't apply cleanly, and i messed up merging the first hunk.  thanks for checking!
Comment 7 SpanKY gentoo-dev 2015-08-06 02:26:26 UTC
Commit message: Add REQUIRED_USE settings for dbus
http://sources.gentoo.org/media-tv/kodi/kodi-9999.ebuild?r1=1.18&r2=1.19