Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Andreas Schwab
Michael Buesch [EMAIL PROTECTED] writes:

 +if ! [ -x mkimage ]; then

What is this supposed to test?

Andreas.

-- 
Andreas Schwab, SuSE Labs, [EMAIL PROTECTED]
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
And now for something completely different.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Michael Buesch
On Wednesday 26 December 2007 15:47:05 Andreas Schwab wrote:
 Michael Buesch [EMAIL PROTECTED] writes:
 
  +if ! [ -x mkimage ]; then
 
 What is this supposed to test?

from man test:
   -x FILE
  FILE exists and execute (or search) permission is granted

-- 
Greetings Michael.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Michael Buesch
On Wednesday 26 December 2007 16:33:57 Andreas Schwab wrote:
 Michael Buesch [EMAIL PROTECTED] writes:
 
  On Wednesday 26 December 2007 15:47:05 Andreas Schwab wrote:
  Michael Buesch [EMAIL PROTECTED] writes:
  
   +if ! [ -x mkimage ]; then
  
  What is this supposed to test?
 
  from man test:
 
 You didn't answer my question.

I'm not sure how I can make the manual text any clearer

   -x FILE
  FILE exists and execute (or search) permission is granted

It tests if the program exists and is executable.
Could you please be more specific about what you don't understand?
That would help a lot not wasting my time.

-- 
Greetings Michael.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Andreas Schwab
Michael Buesch [EMAIL PROTECTED] writes:

 On Wednesday 26 December 2007 15:47:05 Andreas Schwab wrote:
 Michael Buesch [EMAIL PROTECTED] writes:
 
  +if ! [ -x mkimage ]; then
 
 What is this supposed to test?

 from man test:

You didn't answer my question.

Andreas.

-- 
Andreas Schwab, SuSE Labs, [EMAIL PROTECTED]
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
And now for something completely different.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Andreas Schwab
Michael Buesch [EMAIL PROTECTED] writes:

 On Wednesday 26 December 2007 16:33:57 Andreas Schwab wrote:
 Michael Buesch [EMAIL PROTECTED] writes:
 
  On Wednesday 26 December 2007 15:47:05 Andreas Schwab wrote:
  Michael Buesch [EMAIL PROTECTED] writes:
  
   +if ! [ -x mkimage ]; then
  
  What is this supposed to test?
 
  from man test:
 
 You didn't answer my question.

 I'm not sure how I can make the manual text any clearer

I know very well what it *does*, thank you.  But that does mean that is
does what it is *supposed* to do.

-x FILE
   FILE exists and execute (or search) permission is granted

 It tests if the program exists and is executable.
  ^^^
Observe the fine difference in the choice of words.

 That would help a lot not wasting my time.

How about testing your changes first?

Andreas.

-- 
Andreas Schwab, SuSE Labs, [EMAIL PROTECTED]
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
And now for something completely different.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Michael Buesch
On Wednesday 26 December 2007 16:56:57 Andreas Schwab wrote:
  That would help a lot not wasting my time.
 
 How about testing your changes first?

Stop the bullshit, please.
I tested this and it obviously works very well for me.
I even posted it for review on this list and nobody complained,
including you.

But I see what you are complaining about, now.
It would also error out, if mkimage was installed (which is not the
case for me. That's the whole reason for me making this patch!).

Are you OK with the new version I just sent?

-- 
Greetings Michael.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Andreas Schwab
Michael Buesch [EMAIL PROTECTED] writes:

 I tested this and it obviously works very well for me.

Obviously your tests were incomplete.

 I even posted it for review on this list and nobody complained,
 including you.

???  I did point out an error in your patch just now.

Andreas.

-- 
Andreas Schwab, SuSE Labs, [EMAIL PROTECTED]
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
And now for something completely different.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Michael Buesch
On Wednesday 26 December 2007 17:08:55 Andreas Schwab wrote:
  I even posted it for review on this list and nobody complained,
  including you.
 
 ???  I did point out an error in your patch just now.

I posted it for review a few days ago and the only response
I got was This is probably OK for now... from Josh.

-- 
Greetings Michael.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH] powerpc: Do not fail build if mkimage is not available

2007-12-26 Thread Andreas Schwab
Michael Buesch [EMAIL PROTECTED] writes:

 On Wednesday 26 December 2007 17:08:55 Andreas Schwab wrote:
  I even posted it for review on this list and nobody complained,
  including you.
 
 ???  I did point out an error in your patch just now.

 I posted it for review a few days ago and the only response
 I got was This is probably OK for now... from Josh.

And what does this have to do with me?  I didn't see your mail at all.

Andreas.

-- 
Andreas Schwab, SuSE Labs, [EMAIL PROTECTED]
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
And now for something completely different.
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev