bug#44362: Grep pipe doc needs to warn about --null

bug#44362: Grep pipe doc needs to warn about --null

[積丹尼 Dan Jacobson]

(info "(emacs) Grep Searching") says

        Your command need not simply run ‘grep’; you can use any shell
     command that produces output in the same format.  For instance, you can
     chain ‘grep’ commands, like this:

          grep -nH -e foo *.el | grep bar | grep toto

OK, but it also needs to mention:

To have this currently work, be sure to remove the "--null" currently
added to today's prompt,

    Run grep (like this): grep --color -nH --null -e

Else you will end up with

    Binary file (standard input) matches

emacs-version "27.1"

bug#44362: Grep pipe doc needs to warn about --null

[Lars Ingebrigtsen]

積丹尼 Dan Jacobson <jidanni@jidanni.org> writes:

> (info "(emacs) Grep Searching") says
>
>         Your command need not simply run ‘grep’; you can use any shell
>      command that produces output in the same format.  For instance, you can
>      chain ‘grep’ commands, like this:
>
>           grep -nH -e foo *.el | grep bar | grep toto
>
> OK, but it also needs to mention:
>
> To have this currently work, be sure to remove the "--null" currently
> added to today's prompt,
>
>     Run grep (like this): grep --color -nH --null -e

I don't think the manual can warn you about all invalid things a user
may decide to put in the command line.  The --null is part of the
default command line, so I see your point, but I don't think mentioning
this stuff in the user manual would make much sense, and I'm closing
this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

bug#44362: Grep pipe doc needs to warn about --null

[積丹尼 Dan Jacobson]

I'll make the change for you.

But I checked on
https://www.gnu.org/software/emacs/
https://www.gnu.org/software/tasklist/howto-volunteer.html
and can't find where I can edit the source (hopefully like it is easy to do on
GitHub too.)

>>>>> "LI" == Lars Ingebrigtsen <larsi@gnus.org> writes:
LI> 積丹尼 Dan Jacobson <jidanni@jidanni.org> writes:

>> (info "(emacs) Grep Searching") says
>> 
>> Your command need not simply run ‘grep’; you can use any shell
>> command that produces output in the same format.  For instance, you can
>> chain ‘grep’ commands, like this:
>> 
>> grep -nH -e foo *.el | grep bar | grep toto
>> 
>> OK, but it also needs to mention:
>> 
>> To have this currently work, be sure to remove the "--null" currently
>> added to today's prompt,
>> 
>> Run grep (like this): grep --color -nH --null -e

LI> I don't think the manual can warn you about all invalid things a user
LI> may decide to put in the command line.  The --null is part of the
LI> default command line, so I see your point, but I don't think mentioning
LI> this stuff in the user manual would make much sense, and I'm closing
LI> this bug report.