[U-Boot] [PATCH 1/3] scripts: add scripts/gnu_make to not hard-code make command

Jeroen Hofstee jeroen at myspectrum.nl
Fri Jul 18 20:19:52 CEST 2014


Hello Masahiro,

On 17-07-14 11:18, Masahiro Yamada wrote:
> U-Boot is expected to be built on various platforms
> but makefiles are written for GNU Make.
>
> We should keep in mind that the command 'make' is not always GNU Make.
>
> For example, on Linux, people generally do:
>
>    make <board>_config; make
>
> But FreeBSD folks do
>
>    gmake <board>_config; gmake
>
> (The command 'make' on FreeBSD is BSD Make, not GNU Make)
>
> It is not a good idea to hard-code the command name 'make'
> in MAKEALL or buildman.

indeed, not a good idea.
> Instead, they should call the make command via this helper script
> to make sure it is GNU Make.

yup, or the name of the executable could be displayed, so
it only needs to be checked once at startup (and error out
if no GNU make is found at all).

> +++ b/scripts/gnu_make

nitpicking, most script use a dash instead of an underscore.

> @@ -0,0 +1,30 @@
> +#!/bin/sh
> +#
> +# Call GNU Make
> +#
> +# U-Boot is supposed to be built on various platforms.
> +# One problem is that the command 'make' is not always GNU Make.
> +# (For ex. the command name for GNU Make on FreeBSD is usually 'gmake'.)
> +# It is not a good idea to hard-code the command name 'make'
> +# in scripts where where GNU Make is expected.
> +# In that case, call this helper script to make sure to invoke GNU Make.
> +#
> +# SPDX-License-Identifier:	GPL-2.0+
> +#
> +
> +gnu_make=
> +
> +for m in make gmake
> +do
> +	if $m --version 2>/dev/null | grep -q GNU; then
> +		gnu_make=$m
> +		break
> +	fi
> +done
> +
> +if [ -z "$gnu_make" ]; then
> +	echo "GNU Make not found" >&2
> +	exit 1
> +fi
> +
> +$gnu_make "$@"
>

I guess, it could use exec here, since the script itself
is no longer needed.

Regards,
Jeroen




More information about the U-Boot mailing list