On Thu, Jul 31, 2025 at 11:14 AM Daniel Sahlberg <
daniel.l.sahlb...@gmail.com> wrote:

> Den tors 31 juli 2025 kl 15:46 skrev Nathan Hartman <
> hartman.nat...@gmail.com>:
>
>> [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.
>>
>
> That is correct.
>
>
>>
>> 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.
>>
>
> Also correct.
>
> Do you have a preference for one of the two possible ways to change? I
> really like that I'm able to limit the amount of messages to search with
> the "-f -" variation, but it is not so intuitive...
>

Not a strong preference, but I prefer the change as committed because,
as you point out, you can limit the amount of messages. You could even
get fancy with things like 'svn log --revision { DATE } : HEAD'.

If you aren't happy with the change, you could show both variations:

Near the end of Crediting [1]

[[[

Look over Subversion's existing log messages to see how to use these
fields in practice. This command from the top of your trunk working
copy will help:

$ contrib/client-side/search-svnlog.pl "(Patch|Review|Suggested) by: "

The script runs 'svn log' internally. You can customize the invocation
of 'svn log' by piping its output to the script; the '-f -' arguments
are required for this. For example, to search only the last 100 log
messages:

$ svn log -l 100 | contrib/client-side/search-svnlog.pl -f -
"(Patch|Review|Suggested) by: "

]]]

Or something like that...

[1]
https://subversion-staging.apache.org/docs/community-guide/conventions.html#crediting

Cheers,
Nathan

Reply via email to