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

Re: Change in svn/repos.py for recognition of replaced paths in mailer.py

From: Mathias Weinert <mathias.weinert_at_gfa-net.de>
Date: 2006-09-12 09:01:15 CEST

Daniel Rall wrote:
> On Thu, 07 Sep 2006, Mathias Weinert wrote:
> > C. Michael Pilato wrote:
> > > C. Michael Pilato wrote:
> > > > Mathias Weinert wrote:
> > > >> Hi,
> > > >>
> > > >> I still like to implement the recognition of replaced paths in
> > > >> mailer.py. In order to achieve this I need a change of
> > > >> subversion/bindings/swig/python/svn/repos.py which I already posted
> > > >> to the dev list and for which I already got replies - but no commit.
> > > >>
> > > >> So if there is anybody out there willing to commit my patch (with or
> > > >> without the change proposed by cmpilato) please commit it.
> > > >>
> > > >> Original patch/post:
> > > >> http://svn.haxx.se/dev/archive-2006-08/0612.shtml
> > > >>
> > > >> Last patch provided by me:
> > > >> http://svn.haxx.se/dev/archive-2006-08/0956.shtml
> > > >>
> > > >> cmpilato's reply:
> > > >> http://svn.haxx.se/dev/archive-2006-08/0969.shtml
> > > >>
> > > >> Thank you very much!
> > > >
> > > > Attached is an updated patch that has my proposed change. I'm not able
> > > > to clearly think through the compatibility issues here, but I *think*
> > > > that making the new parameter optional buys us all that we need.
> > >
> > > Ooh. And attached to *this* mail is alternative (much older) patch that
> > > I just found in my ~/misc/PATCHES directory which takes this problem a
> > > different direction altogether. Rather than continuing to grow these
> > > custom action fields (added, replaced, etc.) it just adds an 'action'
> > > field and a list of testable actions.
> > >
> > [...]
> >
> > I don't mind which version of the patch will be applied. I will just be
> > happy if any will be applied ;) so that I can go on with some other
> > changes to mailer.py.
> >
> > Who will decide which one to apply? Is it yourself?
> > And who will then commit it?
>
> Mathias, you've commit access. With Mike now out of town, feel free
> to use your best judgement as to which patch to apply. For a change
> like this, typically the bias would be towards ease of maintenance
> (which probably leans towards Mike's last posted patch?).

I only have commit access to mailer.py so I didn't dare to commit
anything else (although I already asumed that the commit access is not
restricted by the repository itself).

I'll do some tests with cmpilato's patch and then commit it.

Mathias

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Sep 12 09:02:02 2006

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

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