svn commit: r240967 - head/release

[Garrett Cooper]

On Sep 26, 2012, at 12:04 PM, Alexey Dokuchaev <danfe at FreeBSD.org> wrote:

> On Wed, Sep 26, 2012 at 02:46:43PM -0400, Glen Barber wrote:
>> On Wed, Sep 26, 2012 at 11:44:07AM -0700, Garrett Cooper wrote:
>>>> -: ${RELSTRING=`chroot $2 uname -s`-`chroot $2 uname -r`-`chroot $2 uname -p`}
>>>> +if [ "x${OSVERSION}" == "x" ]; then
>>> 
>>> This should be `=` in order to be backwards compatible with older
>>> (pre-9.1) copies of /bin/sh
>> 
>> This won't be going into pre-9.1 versions of FreeBSD.
> 
> Still, there is no need to spread this bogus syntax which we had erroneously adopted.

If you aren't leading the pack, you have to follow the herd (linux).

This syntax was added to our copy of ash after some discussion on the posix list re: test(1) syntax.

Fighting this inconsistency (bash/zsh syntax) causes more unnecessary turmoil and churn in Linux centric projects than needed (and there are a number of bashisms/zshisms we still don't support). I think it's fine to go with the new syntax as long as things that are FreeBSD specific prefer the old syntax as long as 8.2/9.0 and their respective versions are still supported and one can upgrade from older versions (this is squishy, but rule of thumb the last I heard was 2-3 major versions).
Thanks!
-Garrett