[OE-core] [PATCH 1/2] base-files: profile: Do not assume that the 'command' command exists

[Mike Looijmans]

On 18-09-17 15:08, Burton, Ross wrote:
> On 18 September 2017 at 12:31, Mike Looijmans <mike.looijmans at topic.nl 
> <mailto:mike.looijmans at topic.nl>> wrote:
> 
>         This is basically the same change as I first sent a patch for in
>         April, and
>         last pinged this Friday... The only real difference is that this one
>         misses
>         passing error output from resize to /dev/null (which it should do to
>         handle
>         the case where tty exists, but resize does not).
> 
> 
>     Yeah, indeed.
> 
> 
> Apologies for missing that patch!
> 
>     Other problem is that "resize" outputs shell script on stdout to be
>     executed, so the proper "total" invokation would be:
> 
>        /dev/tty[A-z]*) eval `resize 2>/dev/null` ;;
> 
>     The "eval" part is missing in your version...
> 
> 
> Who is going to submit the One True patch with all the fixes in?  I promise to 
> merge it.

I'll send the one ring, eh, patch, in a few minutes. I'll merge the two into a 
single as well.



Kind regards,

Mike Looijmans
System Expert

TOPIC Products
Materiaalweg 4, NL-5681 RJ Best
Postbus 440, NL-5680 AK Best
Telefoon: +31 (0) 499 33 69 79
E-mail: mike.looijmans at topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail