[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Bug in mailer.py

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Mon, 18 Feb 2013 23:36:41 +0200

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_at_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_at_peelman.us
>
>
>
>
Received on 2013-02-18 22:37:18 CET

This is an archived mail posted to the Subversion Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.