[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 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 > 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