115842 - Use "fgrep" instead of "grep" in build code where usefull

Status: VERIFIED WONTFIX

(SeaMonkey :: Build Config, enhancement)

Description

[Roland Mainz]

Looking at configure.in:
There are a lot of places where we do a simple check for a string via "grep"
without using a regex.
It may be nice to use "fgrep" instead of "grep" because it's slightly faster (no
regex precompilation).

For example:
-- snip --
if test -z "$MOZ_JSDEBUGGER" && test `echo "$MOZ_EXTENSIONS" | grep -c venkman`
-ne 0; then
    AC_MSG_WARN([Cannot build venkman without JavaScript debug library. Removing
venkman from MOZ_EXTENSIONS.])
    MOZ_EXTENSIONS=`echo $MOZ_EXTENSIONS | sed -e 's|venkman||'`
fi
-- snip --
Can be turned into:
-- snip --
if test -z "$MOZ_JSDEBUGGER" && test `echo "$MOZ_EXTENSIONS" | fgrep -c venkman`
-ne 0; then
    AC_MSG_WARN([Cannot build venkman without JavaScript debug library. Removing
venkman from MOZ_EXTENSIONS.])
    MOZ_EXTENSIONS=`echo $MOZ_EXTENSIONS | sed -e 's|venkman||'`
fi
-- snip --

Comment 1

[hacker formerly known as [e-mail]]

I'll fix occurrences as I find them but I'm not dedicating time to this.

Comment 2

[Jon Granrose]

verified wontfix.