Thread (5 messages) flat view 5 messages, 4 authors, 2016-06-15

Re: [PATCH] git-grep: convert from bash to sh

From: Timo Hirvonen <hidden>
Date: 2016-06-15 22:42:15

On Sun, 18 Dec 2005 11:57:26 -0800 (PST)
Linus Torvalds [off-list ref] wrote:
On Sun, 18 Dec 2005, Timo Hirvonen wrote:
quoted
sh does not support arrays so we have to use eval instead.
This seems horribly broken.

If I'm not mistaken, this breaks

	git grep "it's a happy coincidence"

badly. I didn't test, just looking at the patch.
Actually it works:

/usr/src/linux-2.6: git grep "it's a"
Documentation/Changes:debugfs.  Obviously, it's a good idea to upgrade.
Documentation/CodingStyle:and NOT read it.  Burn them, it's a great
symbolic gesture.


but it doesn't work with backslashes.  Need to use \\

/usr/src/linux-2.6: git grep 'e.\\n"'
Documentation/DMA-mapping.txt:		       "mydev: No suitable DMA
available.\n");
Documentation/DMA-mapping.txt:		       "mydev: No suitable DMA
available.\n");

-- 
http://onion.dynserv.net/~timo/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help