[PATCH] Fix typos and shell for MacOSX packaging scripts
Cristian Ionescu-Idbohrn
cristian.ionescu-idbohrn at axis.com
Tue Oct 2 02:00:53 PDT 2012
On Tue, 2 Oct 2012, Henrik Brautaset Aronsen wrote:
> Den 02.10.12 10:32, skrev Cristian Ionescu-Idbohrn:
> > On Tue, 2 Oct 2012, Henrik Brautaset Aronsen wrote:
> > > /bin/sh can be "anything", so I've forced /bin/bash for good
> > > measure.
> > What do you mean with "good measure"? That script should be
> > portable, ie. work with any posix shell, shouldn't it?
>
> This doesn't have to be very portable, since it's supposed to run on
> MacOSX. Apple has changed /bin/sh earlier, I just thought it would
> be smart to fix the shell to something that's known to work.
To be honest, I don't really care too much about MacOSX but, as a
matter of principal, the "smarter" way to have scripts work with any
posix shell is to write portable code. Even on MacOSX ;) If Apple
breaks the shell, than more than packaging scripts will be broken
anyway.
Cheers,
--
Cristian
More information about the subsurface
mailing list