From dd925df160691bb68e65bc08383661698eb03083 Mon Sep 17 00:00:00 2001
From: Christoph Reiter <reiter.christoph@gmail.com>
Date: Sat, 16 Sep 2017 17:53:14 +0200
Subject: [PATCH] outofdate: show recently updated packages first

---
 main.py                  | 21 ++++++++++-----------
 templates/outofdate.html |  8 +++++---
 2 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/main.py b/main.py
index e48ee2d..6667581 100755
--- a/main.py
+++ b/main.py
@@ -100,7 +100,7 @@ class Package:
                  makedepends, md5sum, name, pgpsig, sha256sum, arch,
                  base_url, repo, repo_variant, provides, conflicts, replaces,
                  version, base, desc, groups, licenses):
-        self.builddate = builddate
+        self.builddate = int(builddate)
         self.csize = csize
         self.depends = depends
         self.filename = filename
@@ -212,11 +212,9 @@ class Source:
 
     @property
     def date(self):
-        """One of the package build dates"""
+        """The build date of the newest package"""
 
-        for n, p in sorted(self.packages.items()):
-            return p.builddate
-        return 0
+        return [p.builddate for p in self.packages.values()][-1]
 
     @property
     def source_url(self):
@@ -581,9 +579,9 @@ def update_versions():
                 if p.name in arch_versions:
                     old_ver = arch_versions[p.name][0]
                     if version_is_newer_than(msys_ver, old_ver):
-                        arch_versions[p.name] = (msys_ver, url)
+                        arch_versions[p.name] = (msys_ver, url, p.builddate)
                 else:
-                    arch_versions[p.name] = (msys_ver, url)
+                    arch_versions[p.name] = (msys_ver, url, p.builddate)
     print("done")
 
     print("update versions from AUR")
@@ -615,8 +613,9 @@ def update_versions():
         name = result["Name"]
         if name not in aur_packages or name in arch_versions:
             continue
+        last_modified = result["LastModified"]
         url = "https://aur.archlinux.org/packages/%s" % name
-        arch_versions[name] = (result["Version"], url)
+        arch_versions[name] = (result["Version"], url, last_modified)
     print("done")
 
     versions = arch_versions
@@ -658,16 +657,16 @@ def outofdate():
             missing.append((s, get_arch_name(s.realname)))
             continue
 
-        arch_version, url = arch_info
+        arch_version, url, date = arch_info
         arch_version = extract_upstream_version(arch_version)
         msys_version = extract_upstream_version(s.version)
 
         if version_is_newer_than(arch_version, msys_version):
-            to_update.append((s, msys_version, arch_version, url))
+            to_update.append((s, msys_version, arch_version, url, date))
 
     # show packages which have recently been build first.
     # assumes high frequency update packages are more important
-    to_update.sort(key=lambda i: i[0].date, reverse=True)
+    to_update.sort(key=lambda i: (i[-1], i[0].name), reverse=True)
 
     missing.sort(key=lambda i: i[0].name)
 
diff --git a/templates/outofdate.html b/templates/outofdate.html
index 974057a..dd36305 100644
--- a/templates/outofdate.html
+++ b/templates/outofdate.html
@@ -20,16 +20,18 @@
       <thead>
         <tr>
           <th>Package</th>
-          <th>MSYS2</th>
-          <th>Arch Linux</th>
+          <th>MSYS2 Version</th>
+          <th>Arch Linux Version</th>
+          <th>Arch Linux Update Date</th>
         </tr>
       </thead>
       <tbody>
-          {% for (s, myver, ver, url) in to_update %}
+          {% for (s, myver, ver, url, date) in to_update %}
            <tr>
             <td><a href="{{ url_for('base', name=s.name) }}">{{ s.name }}</a></td>
             <td>{{ myver }}</td>
             <td><a href="{{ url }}">{{ ver }}</a></td>
+            <td>{{ date|timestamp }}</td>
            </tr>
           {% endfor %}
       </tbody>