Re: Building git-1.5.3.7 on HP-UX 11.00

From: Junio C Hamano <gitster@pobox.com>
Date: Dec 11, 2007, 08:26 UTC
Message-ID: <7vfxy9fxh3.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20071210145123.7c34af6d@pc09.procura.nl>

"H.Merijn Brand" <h.m.brand@xs4all.nl> writes:

Show 8 quoted lines

> Summary of the changes:
>
> 1 Added a section for HP-UX in the Makefile. Note that this will
>   cover most of HP-UX, but might need several changes for both
>   newer HP-UX versions and 64bit environments. Will come to that
>   once I've got it all running
>
> 2 HP-UX does not have <sys/select.h>. I mentioned this before

Unfortunate. It is even in POSIX. But nothing "make -DNO_SYS_SELECT=Unfortunately" cannot fix.

Show 7 quoted lines

> 3 I am willing to believe that HP_UX' vsnprintf () is broken, or
>   at least does not conform to the expectations in the GNU world,
>   but chickening out like the way strbuf_addf () does is maybe a
>   bit too rude, so I forced a minimum of 64bytes available. That
>   fixes a lot!
>
>   but it still breaks t4013 :(

I think Shawn had something similar for Solaris, and if it is a small cost to help portability, it does not feel so bad.

> 4 'tr' doesn't like '\0', but wants '\000' instead.
> 5 'tr' cannot deal with character classes

Ok, that's odd (and not old fashioned, which makes even odder), but "\000" is not too bad (unless we have to do that million places).

> 6 I don't know how to attack another HP specific problem: HP has
>   a system command called 'patch' and it is *very* incompatible
>   with GNU patch. Very.

I do not think there is any reason for us to rely on "GNU patch" anymore. 4109 can have precomputed test vector instead of comparing our output with what GNU patch of the day happens to do.

> 7 What do you expect for LOCALE's?

I am not opposed to a change that makes these tests to conditionally skip on systems (not necessarily "platforms" but individual "installations") that do not have locales necessary to run tests installed, just like cvs or svn tests are skipped when they are not installed. Especially you are not using iconv...

> + git-index-pack -o tmp.idx test-2-7f8ead892057e78576c0329a70cc83afb113f117.pack
> fatal: serious inflate inconsistency

That sounds like a broken zlib X-<.