Move rpm line formating into its own method

1 files changed, 6 insertions, 1 deletions

diff --git a/fedora-easy-karma.py b/fedora-easy-karma.py
index 71ef4c7..dd52112 100755
--- a/fedora-easy-karma.py
+++ b/fedora-easy-karma.py
@@ -203,7 +203,7 @@ class FedoraEasyKarma(object):
                                                  # remove version and release
                                                  ["-".join(b.split("-")[:-2]) for b in affected_builds]]):
                         continue
-                installed_rpms = ["{rpm.name}-{rpm.version}-{rpm.release}.{rpm.arch} - {rpm.summary} (installed {install_age.days} days ago)".format(rpm=pkg, install_age=(now - datetime.datetime.fromtimestamp(pkg.installtime))) for pkg in installed_pkgs]
+                installed_rpms = [self.format_rpm(pkg) for pkg in installed_pkgs]
                 if not self.options.list_rpms_only:
                     print self.bodhi_update_str(update, bodhi_base_url=bc.base_url, width=self.options.wrap_width, wrap_bugs=self.options.wrap_bugs)
                     if self.options.wrap_rpms:
@@ -349,6 +349,11 @@ class FedoraEasyKarma(object):
                 message = "%s - timestamp not updated" % message
             sys.stderr.write("%s\n" % message)
 
+    def format_rpm(self, rpm):
+        now = datetime.datetime.now()
+        install_age=(now - datetime.datetime.fromtimestamp(rpm.installtime))
+        format_str = "{rpm.name}-{rpm.version}-{rpm.release}.{rpm.arch} - {rpm.summary} (installed {install_age.days} days ago)"
+        return format_str.format(rpm=rpm, install_age=install_age)
 
     def info(self, message):
         sys.stderr.write("%s\n" % message)