[PATCH] Fix `grep' command to handle filenames starting with '-'.

[PATCH] Fix `grep' command to handle filenames starting with '-'.

[Xi Lu]

* lisp/org-mouse.el (org-mouse-context-menu): Fix `grep' command to
handle filenames starting with '-' by adjusting wildcard usage to
prevent misinterpretation as options.
---
 lisp/org-mouse.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/org-mouse.el b/lisp/org-mouse.el
index 322d98682..7f85653f2 100644
--- a/lisp/org-mouse.el
+++ b/lisp/org-mouse.el
@@ -627,7 +627,7 @@ This means, between the beginning of line and the point."
 	   ["Sparse Tree" (org-occur ',region-string)]
 	   ["Find in Buffer" (occur ',region-string)]
 	   ["Grep in Current Dir"
-	    (grep (format "grep -rnH -e '%s' *" ',region-string))]
+	    (grep (format "grep -rnH -e '%s' ./*" ',region-string))]
 	   ["Grep in Parent Dir"
 	    (grep (format "grep -rnH -e '%s' ../*" ',region-string))]
 	   "--"
-- 
2.45.2

Re: [PATCH] Fix `grep' command to handle filenames starting with '-'.

[Rens Oliemans]

Xi Lu <lx@shellcodes.org> writes:

> -	    (grep (format "grep -rnH -e '%s' *" ',region-string))]
> +	    (grep (format "grep -rnH -e '%s' ./*" ',region-string))]

This works so my comment might be a bit too bikesheddy, but another method would be to
append the argument `--` after the list of options to ensure that the rest of the
arguments aren't parsed as options; perhaps that's preferred.

Re: [PATCH] Fix `grep' command to handle filenames starting with '-'.

[lux]

On Thu, 2024-11-07 at 11:40 +0100, Rens Oliemans wrote:
> Xi Lu <lx@shellcodes.org> writes:
> 
> > -	    (grep (format "grep -rnH -e '%s' *" ',region-string))]
> > +	    (grep (format "grep -rnH -e '%s' ./*" ',region-string))]
> 
> This works so my comment might be a bit too bikesheddy, but another method
> would be to
> append the argument `--` after the list of options to ensure that the rest of
> the
> arguments aren't parsed as options; perhaps that's preferred.

Yes, but I haven't tested it on other Unix-like platforms, so I'm not sure if "-
-" will work properly on all of them.