ExternalArchive(): Fixes proposed by Bernhard Reiter, specifically:

use the lower cased list name (i.e. the internal name) as the
%(listname)s expansion; log a message when the external archivers
exits with a non-zero status.

1 files changed, 6 insertions, 2 deletions

diff --git a/Mailman/Archiver/Archiver.py b/Mailman/Archiver/Archiver.py
index 5e3197768..935480242 100644
--- a/Mailman/Archiver/Archiver.py
+++ b/Mailman/Archiver/Archiver.py
@@ -179,11 +179,15 @@ class Archiver:
             post.SetHeader('Date', olddate)
 
     def ExternalArchive(self, ar, txt):
-        d = SafeDict({'listname': self.real_name})
+        d = SafeDict({'listname': self.internal_name()})
         cmd = ar % d
         extarch = os.popen(cmd, 'w')
         extarch.write(txt)
-        extarch.close()
+        status = extarch.close()
+        if status:
+            self.LogMsg('error',
+                        'external archiver non-zero exit status: %d\n' %
+                        (status & 0xff00) >> 8)
 
     #
     # archiving in real time  this is called from list.post(msg)