[Bug 35896] Use XDG directories

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Sep 11 11:14:46 CEST 2012


https://bugs.freedesktop.org/show_bug.cgi?id=35896

--- Comment #30 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2012-09-11 09:14:46 UTC ---
Comment on attachment 66613
  --> https://bugs.freedesktop.org/attachment.cgi?id=66613
[3/3] McdAccount: migrate avatars to a XDG path

Review of attachment 66613:
 --> (https://bugs.freedesktop.org/page.cgi?id=splinter.html&bug=35896&attachment=66613)
-----------------------------------------------------------------

::: tests/twisted/account-manager/avatar-persist.py
@@ +150,5 @@
> +    avatar_filename = account_id
> +    avatar_filename = avatar_filename.replace('/', '-') + '.avatar'
> +    avatar_filename = (os.environ['XDG_DATA_HOME'] +
> +        '/telepathy/mission-control/' + avatar_filename)
> +    assertEquals('MJ12', ''.join(open(avatar_filename, 'r').readlines()))

Why do you do ''.join(open('filename')).readlines()) and not just
open('filename').read()?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.



More information about the telepathy-bugs mailing list