Re: svn commit: r1927514 - subversion/site/staging/docs/community-guide

Den tis 29 juli 2025 kl 09:06 skrev <[email protected]>:

> 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