subversion-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Shahaf <...@daniel.shahaf.name>
Subject Re: Bug in mailer.py
Date Mon, 18 Feb 2013 21:36:41 GMT
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
> nick@peelman.us
> 
> 
> 
> 



Mime
View raw message