Re: ephemeral txnprops

2013-07-01 Thread C. Michael Pilato
On 07/01/2013 02:16 PM, Daniel Shahaf wrote: > C. Michael Pilato wrote on Mon, Jul 01, 2013 at 14:13:50 -0400: >> I saw the "user-agent" change go through. What about "compat-version"? Did >> you intentionally omit that one? > > No; svn:txn-client-compat-version was already being set over ra_loc

Re: ephemeral txnprops

2013-07-01 Thread Daniel Shahaf
C. Michael Pilato wrote on Mon, Jul 01, 2013 at 14:13:50 -0400: > On 07/01/2013 02:01 PM, Daniel Shahaf wrote: > > 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

Re: ephemeral txnprops

2013-07-01 Thread C. Michael Pilato
On 07/01/2013 02:01 PM, Daniel Shahaf wrote: > 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 eit

Re: ephemeral txnprops

2013-07-01 Thread Daniel Shahaf
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 properti

Re: ephemeral txnprops

2013-07-01 Thread C. Michael Pilato
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 availabl

ephemeral txnprops

2013-07-01 Thread Daniel Shahaf
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 ? The te