> Committed in r1917397.

Hi Daniel, the indentation is somehow off [1]. My editor is showing
that there are tabs there (in attached image).

[1] 
https://github.com/apache/subversion/commit/014bf1137790e38c5d9a9ebcaa8eae37a952e720#diff-a8f344ecc1feeea3eae75bf0bdce5784cb1c945e8748015d7d1c4a42e61c8f9e

-- Khairul

On Sun, Apr 28, 2024 at 4:23 PM Daniel Sahlberg
<daniel.l.sahlb...@gmail.com> wrote:
>
> Den sön 28 apr. 2024 kl 04:18 skrev Khairul Azhar Kasmiran 
> <kaza...@gmail.com>:
>>
>> Hi everyone!
>>
>> Following on from [1], this patch prevents the error "a peg revision
>> is not allowed here" when svn_apply_autoprops.py handles filenames
>> containing `@`. Stack Overflow [2] gives the impression that these
>> filenames are rare -- I have them because I have code that works with
>> npm type packages like [3].
>>
>> The patch still appears routine to me, but maybe I've put the fix in
>> the wrong place or something like that.
>>
>> [1] https://lists.apache.org/thread/k0o0ytr6h74wf92p4xyg8tvq5g7h4tj7
>> [2] 
>> https://stackoverflow.com/questions/757435/how-to-escape-characters-in-subversion-managed-file-names
>> [3] https://www.npmjs.com/package/@types/node
>>
>> [[[
>> svn_apply_autoprops.py: Support @-containing filenames.
>>
>> * contrib/client-side/svn_apply_autoprops.py
>>   (filter_walk): Append `@` to filenames containing `@`.
>> ]]]
>
>
> Thanks, I've tested and it seems to work fine. Committed in r1917397.
>
> Kind regards,
> Daniel
>
>>
>>
>> Best regards,
>> Khairul

Reply via email to