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

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

On Sun, Sep 15, 2013 at 02:05:59PM -0400, Christos Zoulas wrote:
> On Sep 15,  5:59pm, ast%NetBSD.org@localhost (Adrian Steinmann) wrote:
> -- Subject: Re: CVS commit: src/distrib/utils/embedded
> 
> | 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.
> 
> Yes, so go ahead and define all the other variables those scripts
> assume that are set, like "release" "srcs" etc. Preferably with a
> different definition than the one we have in mkimage to make
> debugging more interesting. There is a reason those are called .conf;
> they are not and they were not meant to be standalone -- by design.
> Duplicating definitions should be always the solution of last resort.

So you're suggesting I remove the

GZIP_CMD=${TOOL_GZIP:-gzip} # ${GZIP} is special to gzip(1)

line and just use

    ${TOOL_GZIP} -dc "${bbone_kernel}" > "${mnt}/boot/${bbone_kernelimg}"

further below in beagleboard.conf, rpi.conf, and rpi_inst.conf ?

By that logic we should have also used ${TOOL_GZIP} for gzip instead
of the ${GZIP} indirection in mkimage which started this thread and
was actually causing a problem.

Tell me what you think is best.

Thanks
Ast