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 by: " 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
