please use dbus-run-session, and/or dbus-daemon directly, in tests

Bug #1624861 reported by Laurent Bigonville
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Déjà Dup
Fix Released
Low
Unassigned
deja-dup (Debian)
Fix Released
Unknown

Bug Description

Hi,

As explained in this debian bug[0], could it be possible to update the tests to use dbus-run-session and/or dbus-daemon instead of dbus-launch?

[0] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=836086

Changed in deja-dup (Debian):
status: Unknown → Confirmed
Vej (vej)
Changed in deja-dup:
status: New → Triaged
importance: Undecided → Low
Vej (vej)
Changed in deja-dup:
assignee: nobody → Vej (vej)
status: Triaged → In Progress
Revision history for this message
Vej (vej) wrote :

Hello everyone.

I worked myself through the mass bug filing [1] and some related manpages and want to sum this up as:

They want to keep the usage of dbus-x11 as low as possibly (at least on debian).
There might be some changes coming to dbus-launch regarding the Autolaunching feature.

But dbus-launch will be supported for a while although it seems to have some nasty side effects (but we are only using it for testing).

The source [0] from Laurent Bigonville suggests concrete changes in the ./tests/shell and a few Cmake-files. I will try them in the next weeks and give a patch if it works as expected.

[1] https://lists.debian.org/debian-devel/2016/08/msg00554.html

Best
Vej

Revision history for this message
Vej (vej) wrote :

Hello again!

I implemented the mentioned changes. The unit-tests seem to work as before, but the test-shell is not longer able to clean the created environment. I will try to find a way around this next week.

Best

Vej

Revision history for this message
Vej (vej) wrote :

Unassigning myself because I does not find a good workaround around the problem mentioned in comment 2.

Changed in deja-dup:
assignee: Vej (vej) → nobody
status: In Progress → Triaged
Michael Terry (mterry)
Changed in deja-dup:
status: Triaged → Fix Released
Changed in deja-dup (Debian):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.