Agreed. This was introduced in r1439592. You can use r1439591 for now; I'll commit your fix shortly. Thanks!
Peelman, Nick wrote on Mon, Feb 18, 2013 at 15:55:40 -0500: > Using mailer.py fetched from: > http://svn.apache.org/repos/asf/subversion/trunk/tools/hook-scripts/mailer/ > > Running: > > > root@svn:/home/svn/agsync/hooks# ./mailer.py commit /home/svn/agsync/ 8160 > > ../conf/mailer.conf > > Traceback (most recent call last): > > File "./mailer.py", line 1444, in <module> > > sys.argv[3:3+expected_args]) > > File "/usr/lib/python2.7/dist-packages/svn/core.py", line 281, in run_app > > return func(application_pool, *args, **kw) > > File "./mailer.py", line 102, in main > > {'author': author, > > UnboundLocalError: local variable 'author' referenced before assignment > > > Fix: > > ~ Line 102: > > > if cmd == 'commit': > > revision = int(cmd_args[0]) > > repos = Repository(repos_dir, revision, pool) > > cfg = Config(config_fname, repos, > > {'author': repos.author, > > 'repos_basename': os.path.basename(repos.repos_dir) > > }) > > messenger = Commit(pool, cfg, repos) > > Use repos.author instead of just 'author'. > > -nick > > -- > Peelman, Nick > n...@peelman.us > > > >