diff options
author | Bogdano Arendartchuk <bogdano@mandriva.org> | 2006-07-21 23:39:20 +0000 |
---|---|---|
committer | Bogdano Arendartchuk <bogdano@mandriva.org> | 2006-07-21 23:39:20 +0000 |
commit | a46cdc5b84843f6708c9db5c1b8752a84fac8d85 (patch) | |
tree | 26024f4cb04326f2ddd82756160653c9c86a33b6 | |
parent | abe7287b58251417d3966409d1f8c649f0e78fc8 (diff) | |
download | mgarepo-a46cdc5b84843f6708c9db5c1b8752a84fac8d85.tar mgarepo-a46cdc5b84843f6708c9db5c1b8752a84fac8d85.tar.gz mgarepo-a46cdc5b84843f6708c9db5c1b8752a84fac8d85.tar.bz2 mgarepo-a46cdc5b84843f6708c9db5c1b8752a84fac8d85.tar.xz mgarepo-a46cdc5b84843f6708c9db5c1b8752a84fac8d85.zip |
- changed ugly class by better ones for log.py
-rw-r--r-- | RepSys/log.py | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/RepSys/log.py b/RepSys/log.py index 2648b41..d218c64 100644 --- a/RepSys/log.py +++ b/RepSys/log.py @@ -77,7 +77,7 @@ def getrelease(pkgdirurl, rev=None): shutil.rmtree(tmpdir) -class _Revision: +class ChangelogRevision: lines = [] date = None raw_date = None @@ -89,14 +89,14 @@ class _Revision: self.__dict__.update(kwargs) -class _Release(_Revision): +class ChangelogRelease(ChangelogRevision): version = None release = None revisions = None def __init__(self, **kwargs): + ChangelogRevision.__init__(self, **kwargs) self.revisions = [] - _Revision.__init__(self, **kwargs) def format_lines(lines): @@ -122,7 +122,7 @@ def format_lines(lines): return entrylines -class _Author: +class ChangelogByAuthor: name = None email = None revisions = None @@ -138,7 +138,7 @@ def group_releases_by_author(releases): # all the mess below is to sort by author and by revision number decorated = [] for authorname, revs in authors.iteritems(): - author = _Author() + author = ChangelogByAuthor() author.name = revs[0].author_name author.email = revs[0].author_email revdeco = [(r.revision, r) for r in revs] @@ -148,10 +148,11 @@ def group_releases_by_author(releases): decorated.sort(reverse=1) release.authors = [t[1] for t in decorated] - # the difference between a released and a not released _Release is - # the way the release numbers is obtained. So, when this is a - # released, we already have it, but if we don't, we should get de - # version/release string using getrelease and then get the first + # the difference between a released and a not released + # ChangelogRelease is the way the release numbers is obtained. So, + # when this is a released, we already have it, but if we don't, we + # should get de version/release string using getrelease and then + # get the first first, release.authors = release.authors[0], release.authors[1:] release.author_name = first.name release.author_email = first.email @@ -168,7 +169,7 @@ emailpat = re.compile("(?P<name>.*?)\s*<(?P<email>.*?)>") def make_release(author=None, revision=None, date=None, lines=None, entries=[], released=True, version=None, release=None): - rel = _Release() + rel = ChangelogRelease() rel.author = author found = emailpat.match(config.get("users", author, author or "")) rel.author_name = (found and found.group("name")) or author @@ -180,7 +181,7 @@ def make_release(author=None, revision=None, date=None, lines=None, rel.lines = lines rel.released = released for entry in entries: - revision = _Revision() + revision = ChangelogRevision() revision.revision = entry.revision revision.lines = format_lines(entry.lines) revision.date = time.strftime("%a %b %d %Y", entry.date) |