#!/usr/bin/python
-### forward-diffs.py --- forward emacs-elpa-diffs mails to maintainers
+### forward-diffs.py --- forward emacs-diffs mails to maintainers
-## Copyright (C) 2012 Free Software Foundation, Inc.
+## Copyright (C) 2012-2014 Free Software Foundation, Inc.
## Author: Glenn Morris <rgm@gnu.org>
+## Maintainer: emacs-devel@gnu.org
## This program is free software; you can redistribute it and/or modify
## it under the terms of the GNU General Public License as published by
### Commentary:
-## Forward emails from the emacs-elpa-diffs mailing list to the
+## Forward emails from an emacs-diffs style mailing list to the
## maintainer(s) of the modified files.
## Two modes of operation:
## :0c
## * ^TO_emacs-elpa-diffs@gnu\.org
-## | forward-diffs.py -p packagedir -m maintfile -l logfile -s sender
+## | forward-diffs.py -p packagedir -m maintfile -l logfile \
+## -o overmaint -s sender
## where
##
## package1/file1 email1
## package2/file2 email2,email3
+## package3 email4
##
## Use "nomail" for the email field to not send a mail.
+## An entry that is a directory applies to all files in that directory
+## that do not have specific maintainers.
##
-## overmaintfile = like maintfile, but takes precedence over it.
+## overmaint = like maintfile, but takes precedence over it.
### Code:
scan_file(path, maints)
## This would skip printing empty maints.
## That would mean we would scan the file each time for no reason.
-## if not maints: continue
+ ## But empty maintainers are an error at present.
+ if not maints: continue
path = re.sub( '^%s' % dir, '', path )
string = "%-50s %s\n" % (path, ",".join(maints))
if fd:
usage="""usage: %prog <-p /path/to/packages> <-m maintfile>
- <-l logfile -s sender|--create> [-o overmaintfile] [--sendmail] [--debug]
-Take a GNU ELPA diff on stdin, and forward it to the maintainer(s)."""
+ <-l logfile -s sender|--create> [-o overmaintfile] [--prefix prefix]
+ [--sendmail] [--debug]
+Take an emacs-diffs mail on stdin, and forward it to the maintainer(s)."""
parser = optparse.OptionParser()
parser.set_usage ( usage )
help="sender address for forwards")
parser.add_option( "--create", dest="create", default=False,
action="store_true", help="create maintfile")
+parser.add_option( "--no-scan", dest="noscan", default=True,
+ action="store_true",
+ help="don't scan for maintainers; implies --no-update")
+parser.add_option( "--no-update", dest="noupdate", default=False,
+ action="store_true",
+ help="do not update the maintfile")
+parser.add_option( "--prefix", dest="prefix", default="packages/",
+ help="prefix to remove from modified file name [default: %default]")
parser.add_option( "--sendmail", dest="sendmail", default=False,
action="store_true", help="use sendmail rather than smtp")
parser.add_option( "--debug", dest="debug", default=False,
parser.error('No sender specified')
-## Create the maintfile.
-if opts.create:
- scan_dir( opts.packagedir, opts.maintfile )
- sys.exit()
-
-
try:
lfile = open( opts.logfile, 'a' )
except Exception as err:
lfile.write('Error opening maintfile: %s\n' % str(err))
sys.exit(1)
+## Create the maintfile.
+if opts.create:
+ scan_dir( opts.packagedir, opts.maintfile )
+ sys.exit()
+
+
## Each element is package/file: maint1, maint2, ...
maints = {}
for line in mfile:
if re.match( '#| *$', line ): continue
+ ## FIXME error here if empty maintainer.
(pfile, maint) = line.split()
maints[pfile] = maint.split(',')
text = stdin.read()
-resent_via = 'GNU ELPA diff forwarder'
+resent_via = 'GNU Emacs diff forwarder'
message = email.message_from_string( text )
for line in text.splitlines():
- if re.match( 'modified:$', line ):
+ # Look for and process things that look like (Git):
+ #
+ # Summary of changes:
+ # packages/vlf/vlf.el | 2 +-
+ # 1 files changed, 1 insertions(+), 1 deletions(-)
+ #
+ # or things that look like (Git):
+ #
+ # ---
+ # packages/vlf/vlf.el | 2 +-
+ # 1 files changed, 1 insertions(+), 1 deletions(-)
+
+ #BZR: if re.match( 'modified:$', line ):
+ if re.match( '---|Summary of changes:$', line ):
start = True
continue
if not start: continue
- if re.match( ' *$', line ): break
-
-
- reg = re.match( 'packages/([^ ]+)', line.strip() )
- if not reg: break
-
+ ## An empty line or a line with non-empty first character.
+ if re.match( '( *$|[^ ])', line ): break
+ # Any line that doesn't match the diffstat format (Git).
+ if not re.match( ' [^ ]+ +\| ', line ):
+ lfile.write('Stop scanning at: %s\n' % line)
+ break
+
+ if opts.prefix:
+ #BZR: reg = re.match( '%s([^ ]+)' % opts.prefix, line.strip() )
+ reg = re.match( ' %s([^ ]+)' % opts.prefix, line )
+ if not reg:
+ lfile.write('Skip: %s\n' % line)
+ continue
+ pfile = reg.group(1)
+ else:
+ pfile = line.strip()
- pfile = reg.group(1)
lfile.write('File: %s\n' % pfile)
if not pfile in maints:
- lfile.write('Unknown maintainer, scanning file...\n')
+ lfile.write('Unknown maintainer\n')
- thismaint = []
- thisfile = os.path.join( opts.packagedir, pfile )
+ if not opts.noscan:
- scan_file( thisfile, thismaint )
+ lfile.write('Scanning file...\n')
+ thismaint = []
+ thisfile = os.path.join( opts.packagedir, pfile )
+ # scan_file( thisfile, thismaint )
- if not thismaint: continue
+ if thismaint:
+ maints[pfile] = thismaint
- maints[pfile] = thismaint
+ ## Append maintainer to file.
+ if not opts.noupdate:
+ try:
+ mfile = open( opts.maintfile, 'a' )
+ string = "%-50s %s\n" % (pfile, ",".join(thismaint))
+ mfile.write(string)
+ mfile.close()
+ lfile.write('Appended to maintfile\n')
+ except Exception as err:
+ lfile.write('Error appending to maintfile: %s\n' %
+ str(err))
- ## Append maintainer to file.
- try:
- mfile = open( opts.maintfile, 'a' )
- string = "%-50s %s\n" % (pfile, ",".join(thismaint))
- mfile.write(string)
- mfile.close()
- lfile.write('Appended to maintfile\n')
- except Exception as err:
- lfile.write('Error appending to maintfile: %s\n' % str(err))
+ ## Didn't scan, or scanning did not work.
+ ## Look for a directory maintainer.
+ if not pfile in maints:
+ lfile.write('No file maintainer, trying directories...\n')
+ while True:
+ (pfile, tail) = os.path.split(pfile)
+ if not pfile: break
+ if pfile in maints: break
+
+
+ if not pfile in maints:
+ lfile.write('No maintainer, skipping\n')
+ continue
for maint in maints[pfile]: