Re: - search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Oleg Verych
On Thu, Feb 08, 2007 at 03:35:34PM -0500, Mike Frysinger wrote: [] > there's a difference between NACKing my changes because the patch > breaks things versus because you think the whole system should be > taking some other form. After all, i was giving negative acknowledgement to only-good-branch

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Mike Frysinger
On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: On Thu, Feb 08, 2007 at 09:28:03AM -0500, Mike Frysinger wrote: > On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >proposition is to substitute: > > "$(CONFIG_SHELL) $(MKIMAGE)" > >with > > "mkimage" > > this isnt a one-to-one change

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Oleg Verych
On Thu, Feb 08, 2007 at 09:28:03AM -0500, Mike Frysinger wrote: > On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >proposition is to substitute: > > "$(CONFIG_SHELL) $(MKIMAGE)" > >with > > "mkimage" > > this isnt a one-to-one change ... let's look at the typical > mkimage-missing

Re: + search-a-little-harder-for-mkimage.patch

2007-02-08 Thread Oleg Verych
On Thu, Feb 08, 2007 at 05:38:28PM +0100, Roman Zippel wrote: > Hi, > > On Thu, 8 Feb 2007, Oleg Verych wrote: > > > BTW, Debian shell policy is to be bashizms-clear. Purpose -- to have > > compatibility with any `sh'. Shall we? > > Wrong. > >

Re: + search-a-little-harder-for-mkimage.patch

2007-02-08 Thread Roman Zippel
Hi, On Thu, 8 Feb 2007, Oleg Verych wrote: > BTW, Debian shell policy is to be bashizms-clear. Purpose -- to have > compatibility with any `sh'. Shall we? Wrong. http://www.debian.org/doc/debian-policy/ch-files.html#s-scripts bye, Roman - To unsubscribe from this list: send the line

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Mike Frysinger
On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: proposition is to substitute: "$(CONFIG_SHELL) $(MKIMAGE)" with "mkimage" this isnt a one-to-one change ... let's look at the typical mkimage-missing scenario ... with mkuboot.sh you'd get output like: ... UIMAGE

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Oleg Verych
> From: [EMAIL PROTECTED] > Newsgroups: gmane.linux.kernel.commits.mm > Subject: + search-a-little-harder-for-mkimage.patch added to -mm tree > Date: Wed, 07 Feb 2007 22:07:38 -0800 [] > -- > Subject: search a little harder

Re: + search-a-little-harder-for-mkimage.patch

2007-02-08 Thread Jan Engelhardt
On Feb 8 2007 07:43, Oleg Verych wrote: >> >> >> >> > If that matter, `type -path' is bashizm (BloAted SHell), and "blackbox" >> >> > with "dash" (very good `sh' equivalents) will fail. >> >> >> >> Does the kernel presently build with that shell? >> > >> >build - yes, with dash being `$(shell)'.

Re: + search-a-little-harder-for-mkimage.patch

2007-02-08 Thread Jan Engelhardt
On Feb 8 2007 07:43, Oleg Verych wrote: If that matter, `type -path' is bashizm (BloAted SHell), and blackbox with dash (very good `sh' equivalents) will fail. Does the kernel presently build with that shell? build - yes, with dash being `$(shell)'. All scripts with `#!/bin/bash'

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Oleg Verych
From: [EMAIL PROTECTED] Newsgroups: gmane.linux.kernel.commits.mm Subject: + search-a-little-harder-for-mkimage.patch added to -mm tree Date: Wed, 07 Feb 2007 22:07:38 -0800 [] -- Subject: search a little harder for mkimage From: Mike

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Mike Frysinger
On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: proposition is to substitute: $(CONFIG_SHELL) $(MKIMAGE) with mkimage this isnt a one-to-one change ... let's look at the typical mkimage-missing scenario ... with mkuboot.sh you'd get output like: ... UIMAGE

Re: + search-a-little-harder-for-mkimage.patch

2007-02-08 Thread Roman Zippel
Hi, On Thu, 8 Feb 2007, Oleg Verych wrote: BTW, Debian shell policy is to be bashizms-clear. Purpose -- to have compatibility with any `sh'. Shall we? Wrong. http://www.debian.org/doc/debian-policy/ch-files.html#s-scripts bye, Roman - To unsubscribe from this list: send the line unsubscribe

Re: + search-a-little-harder-for-mkimage.patch

2007-02-08 Thread Oleg Verych
On Thu, Feb 08, 2007 at 05:38:28PM +0100, Roman Zippel wrote: Hi, On Thu, 8 Feb 2007, Oleg Verych wrote: BTW, Debian shell policy is to be bashizms-clear. Purpose -- to have compatibility with any `sh'. Shall we? Wrong.

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Oleg Verych
On Thu, Feb 08, 2007 at 09:28:03AM -0500, Mike Frysinger wrote: On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: proposition is to substitute: $(CONFIG_SHELL) $(MKIMAGE) with mkimage this isnt a one-to-one change ... let's look at the typical mkimage-missing scenario ... You are

Re: + search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Mike Frysinger
On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: On Thu, Feb 08, 2007 at 09:28:03AM -0500, Mike Frysinger wrote: On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: proposition is to substitute: $(CONFIG_SHELL) $(MKIMAGE) with mkimage this isnt a one-to-one change ... let's look at

Re: - search-a-little-harder-for-mkimage.patch added to -mm tree

2007-02-08 Thread Oleg Verych
On Thu, Feb 08, 2007 at 03:35:34PM -0500, Mike Frysinger wrote: [] there's a difference between NACKing my changes because the patch breaks things versus because you think the whole system should be taking some other form. After all, i was giving negative acknowledgement to only-good-branch

Re: search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Thu, Feb 08, 2007 at 02:40:28AM -0500, Mike Frysinger wrote: [] > by this argument, why does mkuboot.sh exist at all then ? let's > simply delete mkuboot.sh and change all arch Makefile's like so: > -$(CONFIG_SHELL) $(MKIMAGE) ... > +-mkimage ... Yes (with TOOLCHAIN prefix). > i of

Re: search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Mike Frysinger
On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: On Thu, Feb 08, 2007 at 12:56:17AM -0500, Mike Frysinger wrote: > On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >> Check to see if `${CROSS_COMPILE}mkimage` exists and if not, fall back to > >> the standard `mkimage` > > > >Why this can't

Re: search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Thu, Feb 08, 2007 at 12:56:17AM -0500, Mike Frysinger wrote: > On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >> Check to see if `${CROSS_COMPILE}mkimage` exists and if not, fall back to > >> the standard `mkimage` > > > >Why this can't be done by > > > > PATH=$CROSS_COMPILE:$PATH > > >

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Mike Frysinger
On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: On Thu, Feb 08, 2007 at 12:58:26AM -0500, Mike Frysinger wrote: > On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >Trivial shell scripts, especially for embedded archs (Blackfin is > >one, IIRC), may be easily converted. > > yes, but Blackfin

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Thu, Feb 08, 2007 at 12:58:26AM -0500, Mike Frysinger wrote: > On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >On Wed, Feb 07, 2007 at 09:19:21PM -0800, Andrew Morton wrote: > >> On Thu, 8 Feb 2007 06:24:40 +0100 Oleg Verych <[EMAIL PROTECTED]> > >wrote: > >> > >> > If that matter, `type

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Mike Frysinger
On 2/8/07, Oleg Verych <[EMAIL PROTECTED]> wrote: On Wed, Feb 07, 2007 at 09:19:21PM -0800, Andrew Morton wrote: > On Thu, 8 Feb 2007 06:24:40 +0100 Oleg Verych <[EMAIL PROTECTED]> wrote: > > > If that matter, `type -path' is bashizm (BloAted SHell), and "blackbox" > > with "dash" (very good

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Wed, Feb 07, 2007 at 09:19:21PM -0800, Andrew Morton wrote: > On Thu, 8 Feb 2007 06:24:40 +0100 Oleg Verych <[EMAIL PROTECTED]> wrote: > > > If that matter, `type -path' is bashizm (BloAted SHell), and "blackbox" > > with "dash" (very good `sh' equivalents) will fail. > > Does the kernel

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Wed, Feb 07, 2007 at 09:19:21PM -0800, Andrew Morton wrote: On Thu, 8 Feb 2007 06:24:40 +0100 Oleg Verych [EMAIL PROTECTED] wrote: If that matter, `type -path' is bashizm (BloAted SHell), and blackbox with dash (very good `sh' equivalents) will fail. Does the kernel presently build

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Mike Frysinger
On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: On Wed, Feb 07, 2007 at 09:19:21PM -0800, Andrew Morton wrote: On Thu, 8 Feb 2007 06:24:40 +0100 Oleg Verych [EMAIL PROTECTED] wrote: If that matter, `type -path' is bashizm (BloAted SHell), and blackbox with dash (very good `sh' equivalents)

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Thu, Feb 08, 2007 at 12:58:26AM -0500, Mike Frysinger wrote: On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: On Wed, Feb 07, 2007 at 09:19:21PM -0800, Andrew Morton wrote: On Thu, 8 Feb 2007 06:24:40 +0100 Oleg Verych [EMAIL PROTECTED] wrote: If that matter, `type -path' is bashizm

Re: + search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Mike Frysinger
On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: On Thu, Feb 08, 2007 at 12:58:26AM -0500, Mike Frysinger wrote: On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: Trivial shell scripts, especially for embedded archs (Blackfin is one, IIRC), may be easily converted. yes, but Blackfin is no-mmu

Re: search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Thu, Feb 08, 2007 at 12:56:17AM -0500, Mike Frysinger wrote: On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: Check to see if `${CROSS_COMPILE}mkimage` exists and if not, fall back to the standard `mkimage` Why this can't be done by PATH=$CROSS_COMPILE:$PATH in your environment?

Re: search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Mike Frysinger
On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: On Thu, Feb 08, 2007 at 12:56:17AM -0500, Mike Frysinger wrote: On 2/8/07, Oleg Verych [EMAIL PROTECTED] wrote: Check to see if `${CROSS_COMPILE}mkimage` exists and if not, fall back to the standard `mkimage` Why this can't be done by

Re: search-a-little-harder-for-mkimage.patch

2007-02-07 Thread Oleg Verych
On Thu, Feb 08, 2007 at 02:40:28AM -0500, Mike Frysinger wrote: [] by this argument, why does mkuboot.sh exist at all then ? let's simply delete mkuboot.sh and change all arch Makefile's like so: -$(CONFIG_SHELL) $(MKIMAGE) ... +-mkimage ... Yes (with TOOLCHAIN prefix). i of