Re: [Qemu-devel] [PATCH] Remove bash-ism from configure

2007-02-12 Thread Krister Joas

On Feb 12, 2007, at 2:41 AM, Anthony Liguori wrote:

Johannes Schindelin wrote:

Hi,

On Sat, 10 Feb 2007, Anthony Liguori wrote:

This is one I introduced in the gcc-3.x autodetect.  It's against  
CVS from a little while ago so hopefully it's just fuzz.




AFAICT which is not really portable, either. But then, I always  
have a bash, so I really don't care.




On my system, which is an executable, not a shell command so it's  
outside the scope of bashism.  It's also used elsewhere within  
configure.


You should take a look at that executable.  On most systems I know, / 
usr/bin/which is a csh script.  It has to be because it also finds  
aliases.  It may or may not be portable to use 'which', I'm not  
really sure, but it's not very efficient.


Krister



___
Qemu-devel mailing list
Qemu-devel@nongnu.org
http://lists.nongnu.org/mailman/listinfo/qemu-devel


Re: [Qemu-devel] Remove bash-ism from configure

2007-02-12 Thread Krister Joas

On Feb 12, 2007, at 6:17 PM, Jernej Simončič wrote:


On Monday, February 12, 2007, 9:55:25, Krister Joas wrote:


You should take a look at that executable.  On most systems I know, /
usr/bin/which is a csh script.  It has to be because it also finds
aliases.  It may or may not be portable to use 'which', I'm not
really sure, but it's not very efficient.


On my Linux boxes, which is either a sh script (on Debian), or a  
(compiled)
program (on Gentoo and Slackware). On my firewall (based on  
FreeBSD), which

is also a program (and also a /bin/sh builtin).


I stand corrected.  NetBSD seems to have removed it completely and  
instead relies on shell built-in commands.  On OpenBSD it's also a  
binary.  My references were Solaris and Mac OS X, where it's  
unfortunately still a csh script.


Krister



___
Qemu-devel mailing list
Qemu-devel@nongnu.org
http://lists.nongnu.org/mailman/listinfo/qemu-devel