do not expect executable permissions for in-tree shell scripts

scripts in the kernel source do not have executable permissions if they were created with patch(1) run mkuboot.sh with bash, its tagged as bash script.

/opt/cross/kernel/linux-2.6.33-cross-host-kernel-source/arch/powerpc/boot/wrapper: line 273: /opt/cross/kernel/linux-2.6.33-cross-host-kernel-source/scripts/mkuboot.sh: Permission denied

---

This happens with 2.6.33, other versions have this bug as well.

 arch/powerpc/boot/wrapper |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

@@ -270,7 +270,7 @@ membase=`${CROSS}objdump -p "$kernel" |
 case "$platform" in
 uboot)
     rm -f "$ofile"
-    ${MKIMAGE} -A ppc -O linux -T kernel -C gzip -a $membase -e $membase \
+    bash ${MKIMAGE} -A ppc -O linux -T kernel -C gzip -a $membase -e $membase \
 	$uboot_version -d "$vmz" "$ofile"
     if [ -z "$cacheit" ]; then
 	rm -f "$vmz"

@@ -330,7 +330,7 @@ coff)
     ;;
 cuboot*)
     gzip -f -9 "$ofile"
-    ${MKIMAGE} -A ppc -O linux -T kernel -C gzip -a "$base" -e "$entry" \
+    bash ${MKIMAGE} -A ppc -O linux -T kernel -C gzip -a "$base" -e "$entry" \
             $uboot_version -d "$ofile".gz "$ofile"
     ;;
 treeboot*)

Wouldn't it be better to use ${SHELL}? Not every system has bash.

Cheers,
Sean

Maybe.
As it stands right now, mkuboot.sh does not run without bash.

And:
Reality check please.
A _development system_ without bash, installed per default on every sane Linux distro, does most likely not exist.

Olaf

Hmmm, can't argue with that. I wouldn't consider building a kernel on the warp sane ;) And to be honest, I am not sure anyone has tried. We officially support it, but I have never done it and don't plan to try.

Cheers,
Sean

I still don't like it very much.. why not chmod'ing it +x instead ? :-)

Cheers,
Ben.

I looked at a few other scripts in the source tree, they are called with $(CONFIG_SHELL) <name>, or perl <name>, or awk <name> or even sh <name>.

So my change adds some sort of consistency and makes packaging more robust.

Olaf