[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