C. Michael Pilato wrote on Mon, Jul 01, 2013 at 13:23:59 -0400: > On 07/01/2013 11:54 AM, Daniel Shahaf wrote: > > SVN_RA_CAPABILITY_EPHEMERAL_TXNPROPS seems to be queried only by the > > libsvn_ra_serf and libsvn_ra_svn. Is there a reason not to either: > > > > - Make ra_local set those properties too > > or > > - Move the logic querying that capability (and when available setting > > svn:txn-* revprops) to ra_loader.c > > > > ? > > There's no reason to query the capability in ra_local (tautology?). But > yeah, I think it makes sense for ra_local to set the properties, too.
Done that in r1498608. There's room for more code sharing (lift the get_client_string() call to ra_loader.c), but I won't do it right now. Cheers, Daniel