Re: CVS commit: src/distrib/utils/embedded

[Adrian Steinmann <ast%NetBSD.org@localhost>]

On Sun, Sep 15, 2013 at 01:26:51PM +0000, Christos Zoulas wrote:
> In article <20130914125127.7648E46%cvs.netbsd.org@localhost>,
> Adrian Steinmann <source-changes-d%NetBSD.org@localhost> wrote:
> 
> 1. This is useless in the .conf files since mkimage sets it alreay.
> >+GZIP_CMD=${TOOL_GZIP:-gzip} # ${GZIP} is special to gzip(1)
Well yes, but it is not said that this config file might eventually
be pulled in by something else not defining TOOL_GZIP. I could have
indeed left the naked "gzip" in there, but I would think that when
TOOL_GZIP is defined, it is what should be used.  In other words,
I thought about it a bit and found this to be the best choice.

> 2. You might as well rename the rest of the variables _CMD for conistency.
I used distrib/sets/sets.subr as an example, which does this:

: ${AWK:=awk}
: ${CKSUM:=cksum}
: ${COMM:=comm}
: ${DATE:=date}
: ${DB:=db}
: ${EGREP:=egrep}
: ${ENV_CMD:=env}       # ${ENV} is special to sh(1), ksh(1), etc.
: ${FGREP:=fgrep}
: ${FIND:=find}
: ${GREP:=grep}
: ${GZIP_CMD:=gzip}     # ${GZIP} is special to gzip(1)
: ${HOSTNAME_CMD:=hostname}     # ${HOSTNAME} is special to bash(1)
: ${HOST_SH:=sh}
: ${IDENT:=ident}
: ${JOIN:=join}
: ${LS:=ls}
: ${MAKE:=make}
: ${MKTEMP:=mktemp}
: ${MTREE:=mtree}
: ${PASTE:=paste}
: ${PAX:=pax}
: ${PRINTF:=printf}
: ${SED:=sed}
: ${SORT:=sort}
: ${STAT:=stat}
: ${TSORT:=tsort}
: ${UNAME:=uname}
: ${WC:=wc}
: ${XARGS:=xargs}

Ast