[PATCHv2] powerpc: comvert relocs_check to a shell script using grep

David Laight David.Laight at ACULAB.COM
Sat Mar 14 01:18:10 AEDT 2015


From: Stephen Rothwell
> This runs a bit faster and removes another use of perl from
> the kernel build.
...
> diff --git a/arch/powerpc/relocs_check.sh b/arch/powerpc/relocs_check.sh
> new file mode 100755
> index 000000000000..182eae9cc40d
> --- /dev/null
> +++ b/arch/powerpc/relocs_check.sh
> @@ -0,0 +1,59 @@
> +#!/bin/sh
...
> +if [ ${#*} -lt 2 ]; then
> +	echo "$0 [path to objdump] [path to vmlinux]" 1>&2
> +	exit 1
> +fi
> +
> +# Have Kbuild supply the path to objdump so we handle cross compilation.
> +objdump="$1"
> +vmlinux="$2"

I'm not 100% sure of the requirements on /bin/sh for kernel builds.
But ${#*} is probably not completely portable (and may not be safe
if there are embedded spaces in the arguments).

Might be better to check:
	if [ -z "$objdump" -o -z "$vmlinux" ]
instead.

Or even:
	if [ ! -x "$objdump" -o ! -r "$vmlinux" ]

	David



More information about the Linuxppc-dev mailing list