Re: [PATCH] [powerpc] do not expect executable permissions for in-tree shell scripts

2010-06-05 Thread Sean MacLennan
On Sat, 5 Jun 2010 10:10:39 +0200
Olaf Hering  wrote:

> 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.

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

Cheers,
   Sean
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] [powerpc] do not expect executable permissions for in-tree shell scripts

2010-06-05 Thread Olaf Hering
On Sat, Jun 05, Sean MacLennan wrote:

> On Sat, 5 Jun 2010 10:10:39 +0200
> Olaf Hering  wrote:
> 
> > 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.
> 
> Wouldn't it be better to use ${SHELL}? Not every system has bash.

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
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] [powerpc] do not expect executable permissions for in-tree shell scripts

2010-06-05 Thread Sean MacLennan
On Sat, 5 Jun 2010 19:44:37 +0200
Olaf Hering  wrote:

> 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.

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
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] [powerpc] do not expect executable permissions for in-tree shell scripts

2010-06-09 Thread Benjamin Herrenschmidt
On Sat, 2010-06-05 at 14:00 -0400, Sean MacLennan wrote:
> On Sat, 5 Jun 2010 19:44:37 +0200
> Olaf Hering  wrote:
> 
> > 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.
> 
> 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.

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

Cheers,
Ben.


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] [powerpc] do not expect executable permissions for in-tree shell scripts

2010-06-10 Thread Olaf Hering
On Thu, Jun 10, Benjamin Herrenschmidt wrote:

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

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

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


Olaf
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev