[to -= commits@]

On Tue, Jul 29, 2025 at 3:12 AM Daniel Sahlberg <daniel.l.sahlb...@gmail.com>
wrote:

> Den tis 29 juli 2025 kl 09:06 skrev <dsahlb...@apache.org>:
>
>> Author: dsahlberg
>> Date: Tue Jul 29 07:06:11 2025
>> New Revision: 1927514
>>
>> Log:
>> In site/staging
>>
>> * docs/community-guide/conventions.part.html
>>   (#crediting): search-svnlog.pl will run `svn log` itself if no
>> filename given
>>
>> Modified:
>>    subversion/site/staging/docs/community-guide/conventions.part.html
>>
>> Modified:
>> subversion/site/staging/docs/community-guide/conventions.part.html
>>
>> ==============================================================================
>> --- subversion/site/staging/docs/community-guide/conventions.part.html
>> Tue Jul 29 06:45:02 2025        (r1927513)
>> +++ subversion/site/staging/docs/community-guide/conventions.part.html
>> Tue Jul 29 07:06:11 2025        (r1927514)
>> @@ -1252,7 +1252,7 @@ these fields in practice.  This command
>>  working copy will help:</p>
>>
>>  <pre>
>> -svn log | contrib/client-side/search-svnlog.pl
>> "(Patch|Review|Suggested) by: "
>> +svn log | contrib/client-side/search-svnlog.pl -f -
>> "(Patch|Review|Suggested) by: "
>>  </pre>
>>
>>  <p><b>Note:</b> The "Approved&nbsp;by:&nbsp;" field seen in some
>>
>>
> I'm not quite sure about this change actually, I made it just to get a
> commit message to start discussing...
>
> The other option I considered (which is clearer in my opinion):
>
>  <pre>
> -svn log | contrib/client-side/search-svnlog.pl "(Patch|Review|Suggested)
> by: "
> + ./contrib/client-side/search-svnlog.pl "(Patch|Review|Suggested) by: "
>  </pre>
>
> On the other hand with the variant committed, you can check just the last
> 100 log messages like this:
> $ svn log -l100 | contrib/client-side/search-svnlog.pl -f -
> "(Patch|Review|Suggested) by: "
>
> Opinions?
>
> Compare the actual sites, the example is at the very end on the #crediting
> section:
>
> https://subversion.apache.org/docs/community-guide/conventions.html#crediting
>
> https://subversion-staging.apache.org/docs/community-guide/conventions.html#crediting
>
> Cheers,
> Daniel
>


So, without '-f -' to explicitly open stdin, search-svnlog.pl ignores
stdin and runs 'svn log' itself.

This means the command shown at HACKING before the change would run
'svn log' twice: first time to pipe its output to search-svnlog.pl,
which happily ignores it, and again from within search-svnlog.pl.

Good catch!

Cheers,
Nathan

Reply via email to