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

Re: 'svn list --search' feature complete?

From: Johan Corveleyn <jcorvel_at_gmail.com>
Date: Tue, 29 Aug 2017 12:22:08 +0200

On Tue, Aug 29, 2017 at 12:08 PM, Stefan Fuhrmann <stefan2_at_apache.org> wrote:
> On 18.08.2017 13:36, Johan Corveleyn wrote:
>>
>> On Sun, Jan 29, 2017 at 10:35 AM, Stefan Fuhrmann <stefan2_at_apache.org>
>> wrote:
>>>
>>> On 24.01.2017 12:20, Stefan Sperling wrote:
>>
>> ...
>>>>
>>>> I would like to get an 1.10.0 alpha1 released in February. Unless I hear
>>>> objections I will start rolling this alpha release from trunk and call a
>>>> vote on it soon.
>>>
>>>
>>>
>>> Originally, I wanted to complete the get_list API before the alpha
>>> release but I can't make any commitments. So, it is probably better
>>> to roll the alpha with whatever is on /trunk at that time.
>>>
>>> -- Stefan^2.
>>
>>
>> Hi Stefan2,
>>
>> Is the current state of the new get_list API complete on trunk now? Is
>> the 'svn list --search' feature useable / good enough for beta?
>>
>> Context: on IRC we were talking about releasing a 1.10 beta, so this
>> crossed my mind as one of the new features that might not be complete
>> yet.
>>
>> BTW: 'svn list --search' isn't described in the release notes yet (and
>> neither in CHANGES I believe).
>>
>
> Sorry that had fallen off the edge of the disk for
> the last couple of weeks. Been super busy and then
> went fully off the grid for the last two weeks.
>
> r1806548 fixes the inconsistency that Julian found.

Are you sure? As I understood from Julian's mail, the problem with the
case-sensitivity really seemed like a bug: files with uppercase
characters were simply not found, not even if the pattern was spelled
out in uppercase. See the last one here below:

> $ svn list | grep -i "^co"
> COMMITTERS
> configure.ac
> contrib/
>
> $ svn list --search 'co*'
> configure.ac
> contrib/
>
> $ svn list --search 'CO*'
> configure.ac
> contrib/

So that looks like: you tried to make it case-insensitive (by
converting the pattern to lowercase internally), but it doesn't work
(because the paths are not converted similarly or something).

(I haven't tested it myself)

Orthogonally: I think case-insensitive matching would be nice to have
as well (either always, or optionally with some flag / pattern prefix
/ ...). Use case: Windows user base, trying to search for files with
an extension, like *.PNG, *.png, ...

> r1806550 got the CHANGES updated.
>
> My plan is to complete the ra_serf implementation
> this week. Let's hope no higher-prio SVN issue
> interrupts that once more.

Great! Thanks for making this work :-).

-- 
Johan
Received on 2017-08-29 12:22:41 CEST

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