From dbbcb19507c558ba7e14e42b62f58cb0dcae39a9 Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Sat, 13 Sep 2008 18:10:15 +0000 Subject: Bug #237526 - Use portage.util.grablines() for exception handling when reading the unread file. svn path=/main/trunk/; revision=11510 --- pym/portage/news.py | 19 +++++-------------- 1 file changed, 5 insertions(+), 14 deletions(-) (limited to 'pym/portage/news.py') diff --git a/pym/portage/news.py b/pym/portage/news.py index 203ce3fec..e366287bb 100644 --- a/pym/portage/news.py +++ b/pym/portage/news.py @@ -10,7 +10,8 @@ __all__ = ["NewsManager", "NewsItem", "DisplayRestriction", import errno import os import re -from portage.util import ensure_dirs, apply_permissions, normalize_path, grabfile, write_atomic +from portage.util import apply_permissions, ensure_dirs, grabfile, \ + grablines, normalize_path, write_atomic from portage.data import portage_gid from portage.locks import lockfile, unlockfile from portage.exception import OperationNotPermitted @@ -141,19 +142,9 @@ class NewsManager(object): if os.access(os.path.dirname(unreadfile), os.W_OK): # TODO: implement shared readonly locks unread_lock = lockfile(unreadfile) - try: - f = open(unreadfile) - try: - unread = f.readlines() - finally: - f.close() - except EnvironmentError, e: - if e.errno != errno.ENOENT: - raise - del e - return 0 - if len(unread): - return len(unread) + + return len(grablines(unreadfile)) + finally: if unread_lock: unlockfile(unread_lock) -- cgit v1.2.3-1-g7c22