Debian Bug report logs - #836296
ruby-dbus: could use dbus-run-session instead of reinventing it

Reply or subscribe to this bug.

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Simon McVittie <smcv@debian.org>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: ruby-dbus: could use dbus-run-session instead of reinventing it

Date: Thu, 1 Sep 2016 13:02:54 +0100

Source: ruby-dbus
Version: 0.11.0-1
Severity: wishlist
Tags: upstream

As described in <https://lists.debian.org/debian-devel/2016/08/msg00554.html>
I'm trying to reduce how much dbus-launch is used in Debian.
This package showed up in my search for mentions of dbus-launch, but
does not actually use dbus-launch, instead implementing its own script
resembling dbus-run-session.

This is really a minor upstream bug rather than anything in Debian,
but it's one of up to 300 in this mass bug filing, so I'm not going to
go looking for every upstream bug tracker - sorry!

Unlike dbus-launch, dbus-run-session is better-designed for the use
case of wrapping a regression test, and does not have an X11 dependency;
distributions are expected to ship it in the same package as dbus-daemon.
Now that dbus has had dbus-run-session for quite a long time (it
was new in 1.8.0, in January 2014) it should be safe to remove
test/tools/dbus-launch-simple, and replace

    ./test_server \
         ../service_newapi.rb \
         -- \
         ./dbus-launch-simple \
         "$@"

with

    ./test_server \
         ../service_newapi.rb \
         -- \
         dbus-run-session \
         -- \
         "$@"

in test/tools/test_env.

Thanks,
    S

Message #10 received at 836296@bugs.debian.org (full text, mbox, reply):

From: Martin Vidner <martin@vidner.net>

To: 836296@bugs.debian.org

Subject: ruby-dbus: could use dbus-run-session instead of reinventing it

Date: Tue, 13 Sep 2016 15:54:25 +0200

[Message part 1 (text/plain, inline)]

Hi!

I am the upstream maintainer of https://github.com/mvidner/ruby-dbus and
I've found this bug by accident (via a Twitter search)

The suggested fix works when I run the tests locally:
https://github.com/mvidner/ruby-dbus/compare/master...mvidner:use-dbus-run-session?expand=1

But not so in build environments.

>    Unlike dbus-launch, dbus-run-session is better-designed for the use
>    case of wrapping a regression test, and does not have an X11
dependency;
>    distributions are expected to ship it in the same package as
>    dbus-daemon.

Unfortunately that seems not to be the case. Travis on Ubuntu Trusty fails:
https://travis-ci.org/mvidner/ruby-dbus/builds/159596244
and also openSUSE-42.1 is still packaging dbus-run-session in the X11
subpackage:
$ rpm -qf /usr/bin/dbus-run-session
dbus-1-x11-1.8.16-7.1.x86_64

I think I will need to find a way to install dbus-run-session in the Travis
environment, and get the openSUSE package fixed.

[Message part 2 (text/html, inline)]

Send a report that this bug log contains spam.