Re: FYI: Reply from HP-UX

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jan 17, 2008, 16:58 UTC
Message-ID: <alpine.LSU.1.00.0801171655570.5731@racer.site>
In-Reply-To: <20080117141143.38a88c7a@pc09.procura.nl>

Hi,

On Thu, 17 Jan 2008, H.Merijn Brand wrote:

Show 9 quoted lines

> > I got git running on HP-UX 11.00, which still is our base for our Version
> > Control System (VCS)s. Most of the changes I made were incorporated into
> > the git mainline.  
> 
> I've had a look at the git 1.5.3.8 source code and there are several
> HP-UX issues with it:
> 
> * Use of gcc-specific features. Doesn't seem to want to compile nicely
>   with HP's ANSI C compiler.

If that would be a bit more specific, we might very well be able to fix it. After all, AFAIK we do support some non-gcc compilers.

Show 5 quoted lines

> * Failure of the configure/make to handle the lack of unsetenv() in
>   pre-11.31 HP-UX releases. Bizarrely, it does handle the lack of setenv()
>   and also has a compat/unsetenv.c for use by non-unsetenv()-capable OS'es,
>   but then configure and config.mak.in completely fail to use a NO_UNSETENV
>   variable like they should!

Didn't you provide them with a patch to Makefile making a run of ./configure unnecessary?

> * Use of "-include config.mak.autogen" in the top-level Makefile doesn't
>   seem to actually include that file. I had to remove the leading "-" to
>   include it. Also had to stop "gmake clean" from deleting config.mak.autogen
>   otherwise a second "gmake clean" would fail :-(

That is strange. The "-" in front means that the exit status of this command is ignored. So it is no surprise that it fails when you remove the "-". OTOH it is quite a surprise that it does not work when leaving it alone... What's the output?

> * Have to force use of /usr/local/bin/perl, otherwise it picks up /usr/bin/perl
>   which the build system doesn't like.

This should be handled in your patch to Makefile IMHO.

> * "gmake check" requires porting "sparse" at
>   http://www.kernel.org/pub/software/devel/sparse/ which could be "fun".

You don't need to make "check".  You need to make "test".

Ciao, Dscho