[Merge] lp:~michael-sheldon/content-hub/fix-1390095 into lp:content-hub

Oliver Grawert ogra at ubuntu.com
Sat Nov 8 11:30:46 UTC 2014


Review: Needs Fixing

clear NACK

please get rid of this dependency it caused a lot of havoc in the past (multiple running dbus seeion daemons with different addresses etc) and breaks AP testing completely ... we took quite some effort to get rid of it from the image in trusty.

as i said on IRC, it is ok to temporary still use /run/user/$UID/dbus-session ... but long term we need a clean solution that does not expose a security relevant var in a file and simply hands it over from the calling environment.
-- 
https://code.launchpad.net/~michael-sheldon/content-hub/fix-1390095/+merge/241129
Your team Ubuntu Phablet Team is subscribed to branch lp:content-hub.



More information about the Ubuntu-reviews mailing list