On 24/03/2008, at 11:34 PM, Nathan Scott wrote:
On Tue, 2008-03-25 at 16:59 +1100, Mark Goodwin wrote:
Yes that's pretty much the explanation, thanks Max. Nowdays I think
we could probably nuke the search for awk in configure and just
statically define it as PCP_AWK_PROG=awk in pcp.conf.in. That assumes
awk will be found in the default path on all required distros .. is
that true?
Pretty much. That would break the ability to set the
location/binary via the AWK environment variable though
(at configure/build time). It might be better to do it
the same way as sed...
test -z "$SED" && AC_PATH_PROG(SED, sed, /bin/sed)
sed=$SED
AC_SUBST(sed)
This fix works for me ....
diff --git a/configure.in b/configure.in
index 3cd5336..0f2f7d6 100644
--- a/configure.in
+++ b/configure.in
@@ -410,7 +410,7 @@ lex=$LEX
AC_SUBST(lex)
dnl check if user wants their own awk, sed and echo
-test -z "$AWK" && AC_PROG_AWK
+test -z "$AWK" && AC_PATH_PROG(AWK, awk, /usr/bin/awk)
if test $target_os = irix -a "$AWK" = "gawk"
then
dnl always use nawk on IRIX
diff --git a/src/include/pcp.env b/src/include/pcp.env
index dc6c96d..ca657fc 100644
--- a/src/include/pcp.env
+++ b/src/include/pcp.env
@@ -63,7 +63,7 @@ PATH=$PATH/usr/sbin:/sbin:/bin:/usr/bin:/etc:$
{PCP_BIN_DIR}:${PCP_BINADM_DIR}:${
#
for dir in /usr/bsd /usr/etc /usr/bin/X11 /usr/local/bin \
/opt/sfw/bin /usr/ccs/bin /cygdrive/c/WINDOWS/system32 \
- /usr/contrib/bin /opt/local/bin
+ /usr/contrib/bin
do
[ -d $dir ] || continue
echo "$PATH" | egrep ":$dir(:|\$)" >/dev/null && continue
|