Re: [Rpm-maint] [rpm-software-management/rpm] Do not automagically depend on dbus (#361)

2018-02-26 Thread Florian Festi
Closed #361 via 449ed5d9d772d588566748c26ead8c8fbaae71e1. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Do not automagically depend on dbus (#361)

2018-02-05 Thread Daniel M. Weeks
This seemed like the intended approach from the start since building the systemd_inhibit only depends on DBUS and nothing else does (enable_plugins has no granularity the way it's implemented). If there is a preferred way to implements this, then please do, because the current build logic is

Re: [Rpm-maint] [rpm-software-management/rpm] Do not automagically depend on dbus (#361)

2018-02-05 Thread Panu Matilainen
pmatilai requested changes on this pull request. NAK in this form, see comments by @ffesti -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Do not automagically depend on dbus (#361)

2018-02-05 Thread Florian Festi
Name and description are beside the point. This is should not be about dbus support but about whether to build the dbus-inhibit-plugin. The check should still honour the enable_plugins setting and not requiring dbus if plugins are not enabled. -- You are receiving this because you are

Re: [Rpm-maint] [rpm-software-management/rpm] Do not automagically depend on dbus (#361)

2017-11-17 Thread Igor Gnatenko
ignatenkobrain approved this pull request. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/361#pullrequestreview-77534527___

[Rpm-maint] [rpm-software-management/rpm] Do not automagically depend on dbus (#361)

2017-11-17 Thread Daniel M. Weeks
You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/361 -- Commit Summary -- * Do not automagically depend on dbus -- File Changes -- M configure.ac (7) -- Patch Links --