Use grabfile() instead of grablines(), to ignore empty lines.
authorZac Medico <zmedico@gentoo.org>
Thu, 13 Nov 2008 03:32:16 +0000 (03:32 -0000)
committerZac Medico <zmedico@gentoo.org>
Thu, 13 Nov 2008 03:32:16 +0000 (03:32 -0000)
svn path=/main/trunk/; revision=11872

pym/portage/news.py

index ed739c23e5b7509918a33526cbebc958c2c9731b..d26a3ab8df269d56a0b78a4cb47b295befcc8733 100644 (file)
@@ -11,7 +11,7 @@ import logging
 import os
 import re
 from portage.util import apply_permissions, ensure_dirs, \
-       grablines, normalize_path, write_atomic, writemsg_level
+       grabfile, normalize_path, write_atomic, writemsg_level
 from portage.data import portage_gid
 from portage.dep import isvalidatom
 from portage.locks import lockfile, unlockfile
@@ -100,9 +100,9 @@ class NewsManager(object):
                unread_filename = self._unread_filename(repoid)
                unread_lock = lockfile(unread_filename, wantnewlockfile=1)
                try:
-                       unread = set(grablines(unread_filename))
+                       unread = set(grabfile(unread_filename))
                        unread_orig = unread.copy()
-                       skip = set(grablines(skip_filename))
+                       skip = set(grabfile(skip_filename))
                        skip_orig = skip.copy()
 
                        updates = []
@@ -156,7 +156,7 @@ class NewsManager(object):
                except (InvalidLocation, OperationNotPermitted, PermissionDenied):
                        return 0
                try:
-                       return len(grablines(unread_filename))
+                       return len(grabfile(unread_filename))
                finally:
                        if unread_lock:
                                unlockfile(unread_lock)