Code review comment for lp://staging/~pitti/indicator-session/autopkgtest

Revision history for this message
Charles Kerr (charlesk) wrote :

The accountsservice dependency looks like a leftover from ~2011 when it was used to provide the user list.

Likewise, the polkit-1 dependency looks like a leftover from ~2009 when it was used to ask whether or not given actions could be performed. (However, see discussion in bug #474392).

Neither of these dependencies is new in this MP -- pitti's test is just a victim of pre-existing cruft and shouldn't block on it IMO.

I've opened a separate cleanup "remove unneeded dependencies" ticket @ bug #1378023

review: Approve

« Back to merge proposal