Re: [RFH] Solaris portability

From: Guido Ostkamp <git@ostkamp.fastmail.fm>
Date: Nov 18, 2007, 12:08 UTC
Message-ID: <Pine.LNX.4.64.0711181302360.3945@bianca.dialin.t-online.de>
In-Reply-To: <7vtznlww0t.fsf_-_@gitster.siamese.dyndns.org>

On Fri, 16 Nov 2007, Junio C Hamano wrote:

> This makes me wonder if treating it just like strcasestr() might be 
> simpler.  Could folks with access to Solaris boxes of different vintages 
> please see if the attached patch makes sense?

I think the patch makes sense as neither Solaris 8, 9 nor 10 supports mkdtemp().

> Can we also unify UNSETENV, SETENV, C99_FORMAT and STRTOUMAX, by the 
> way?

No.

I've just checked on our Solaris Sparc systems, and found that the C-library provides unsetenv(), setenv() and strtoumax() beginning with Solaris 10; also the 'man sprintf' page mentions the 'z' and 't' specifiers for printf (which is what is behind C99_FORMAT) only beginning with Solaris 10.

So workarounds are needed for all 4 cases for Solaris 8 and 9 but not 10.

Regards

Guido