Branko Čibej wrote:
> On 17.12.2014 17:13, Julian Foad wrote:
>> * 'svnadmin load' always rewrites mergeinfo, which can change its form
>> even if it doesn't need to change it semantically. (Except, since
>> r1643074, if it's unparsable, it loads it unchanged instead of bombing
>> out.)
>>
>>
On 17.12.2014 17:13, Julian Foad wrote:
> Branko Čibej wrote:
>> On 17.12.2014 15:52, Julian Foad wrote:
>>> In http://svn.apache.org/r1646250 I committed a version that also works by
>>> textual substitution, but (hopefully) without the above-mentioned issues.
>>> This
>>> makes svnsync remove
Branko Čibej wrote:
> On 17.12.2014 15:52, Julian Foad wrote:
>> In http://svn.apache.org/r1646250 I committed a version that also works by
>> textual substitution, but (hopefully) without the above-mentioned issues.
>> This
>> makes svnsync remove r0 references before trying to commit.
>
> Doe
On 17.12.2014 15:52, Julian Foad wrote:
> I (Julian Foad) wrote on 4th December:
>> Julian Foad wrote:
>>> ... 'svnsync' ... my current thought is it
>>> should 'correct' the mergeinfo by removing the r0 reference.
>> I have written a test for this, and hacked up code to do this by textual
>> s
I (Julian Foad) wrote on 4th December:
> Julian Foad wrote:
>> ... 'svnsync' ... my current thought is it
>> should 'correct' the mergeinfo by removing the r0 reference.
>
> I have written a test for this, and hacked up code to do this by textual
> substitution. It isn't quite right -- for exam
the regular parser.
Here's the patch in progress...
- Julian
Fix the 'svnsync' part of issue #4476 "Mergeinfo containing r0 makes
svnsync and svnadmin dump fail".
Make 'svnsync' adjust mergeinfo containing r0 to remove the r0 before trying
to commit it to the targ
On 04.12.2014 19:40, Julian Foad wrote:
> I (Julian Foad) wrote:
>
>>> I'll come to 'svnsync' later, but basically my current thought is it
>>> should 'correct' the mergeinfo by removing the r0 reference.
>> I have written a test for this, and hacked up code to do this by textual
>> substitutio
I (Julian Foad) wrote:
>> I'll come to 'svnsync' later, but basically my current thought is it
>> should 'correct' the mergeinfo by removing the r0 reference.
>
> I have written a test for this, and hacked up code to do this by textual
> substitution. It isn't quite right -- for example it wo
Julian Foad wrote:
> http://subversion.tigris.org/issues/show_bug.cgi?id=4476
[...]
> PROPOSAL
>
> In the presence of a mergeinfo property that refers to r0, at the
> libsvn_repos
> API:
>
> 'dump'
> shall dump the property verbatim (with nowarning). (Done.)
>
> 'load', with 'validate proper
http://subversion.tigris.org/issues/show_bug.cgi?id=4476
There's another part to this issue: 'svnadmin load'.
When mergeinfo refers to r0, not only does 'svnadmin dump' fail (fixed in
r1574868) and 'svnsync' fail, but 'svnadmin load' also fails, as I found when I
started writing a test for the
10 matches
Mail list logo
