SF.net SVN: gar:[23442] csw/mgar/gar/v2/lib/python

[wahwah at users.sourceforge.net]

Revision: 23442
          http://sourceforge.net/p/gar/code/23442
Author:   wahwah
Date:     2014-04-20 19:11:44 +0000 (Sun, 20 Apr 2014)
Log Message:
-----------
Maintainer activity: Removed an obsolete comment

Modified Paths:
--------------
    csw/mgar/gar/v2/lib/python/find_obsolete_pkgs.py
    csw/mgar/gar/v2/lib/python/maintainer_activity_report.py

Modified: csw/mgar/gar/v2/lib/python/find_obsolete_pkgs.py
===================================================================
--- csw/mgar/gar/v2/lib/python/find_obsolete_pkgs.py	2014-04-20 18:42:54 UTC (rev 23441)
+++ csw/mgar/gar/v2/lib/python/find_obsolete_pkgs.py	2014-04-20 19:11:44 UTC (rev 23442)
@@ -74,9 +74,13 @@
       {% for dep in pkg.dependlist %}
         {% if dep in obsolete %}
           <span class="obsolete">
+          <a href="http://www.opencsw.org/packages/{{ obsolete[dep].catalogname }}/"
+              title="{{ dep }} is obsolete; {{ pkg.pkgname }} needs to be rebuilt
+              with {{ dep }} removed from RUNTIME_DEP_PKGS"
+            >{{ dep }}</a>
           <a href="http://buildfarm.opencsw.org/pkgdb/srv4/{{ obsolete[dep].md5_sum }}/"
-             title="{{ dep }} is obsolete; {{ pkg.pkgname }} needs to be rebuilt with {{ dep }} removed from RUNTIME_DEP_PKGS"
-          >{{ dep }}</a>
+            title="{{ dep }} in the buildfarm database"
+            >[in pkgdb]</a>
           </span>
         {% endif %}
       {% endfor %}

Modified: csw/mgar/gar/v2/lib/python/maintainer_activity_report.py
===================================================================
--- csw/mgar/gar/v2/lib/python/maintainer_activity_report.py	2014-04-20 18:42:54 UTC (rev 23441)
+++ csw/mgar/gar/v2/lib/python/maintainer_activity_report.py	2014-04-20 19:11:44 UTC (rev 23442)
@@ -76,15 +76,8 @@
 
   <h2>Maintainers to retire</h2>
 
-  <p>Limitations: This report doesn't know which maintainers are new and which ones are old.
-  New maintainers are reported as "to retire" until they release their first
-  package. The script only detects activity in the package catalogs. There are
-  people who are active in the project, even though they don't release
-  packages. They might be wrongly listed here as "to retire". Incorporating the
-  mailing list activity should help, but is not implemented right now.</p>
-
   <p>The following list contains maintainers with no detected activity within
-  the last 2 years.</p>
+  the last {{ counts.inactive_maintainer_cutoff }} years.</p>
 
   <p>
   {% for username in analysis_by_username|sort %}
@@ -161,7 +154,9 @@
       </td>
       <td>
         {% if maintainers[username].active and maintainers[username].csw_db_status != 'Active' %}
-          inconsistent status
+          <span style="color: brown;">
+            inconsistent status
+          </span>
         {% endif %}
         {% if analysis_by_username[username].new and not maintainers[username].pkgs %}
           maybe needs help
@@ -288,7 +283,10 @@
     if username not in csw_db_m_by_username:
       d['bogus'] = True
       counts['bogus'] += 1
-    if maintainers[username].active:
+
+    # Some maintainers have bogus activity due to impersonation. If a
+    # maintainer is retired in the database, don't count them as active.
+    if maintainers[username].active and maintainers[username].csw_db_status != 'Retired':
       counts['active'] += 1
     else:
       if not d['bogus']:
@@ -297,9 +295,10 @@
           counts['to_retire'] += 1
         else:
           counts['retired'] += 1
-
     analysis_by_username[username] = d
 
+  counts['inactive_maintainer_cutoff'] = activity.INACTIVE_MAINTAINER_CUTOFF
+
   with open(args.output, 'w') as outfd:
     template = jinja2.Template(REPORT_TMPL)
     rendered = template.render(

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.