On Thu, 8 May 2008 08:27:48 +0100, "Mike Ralphson" <mike.ralphson@xxxxxxxxx> wrote: > 2008/5/7 Brandon Casey <casey@xxxxxxxxxxxxxxx>: > > > > Some systems define fopen as a macro based on compiler settings. > > The previous technique for reverting to the system fopen function > > by merely undefining fopen is inadequate in this case. Instead, > > avoid defining fopen entirely when compiling this source file. > > > > Signed-off-by: Brandon Casey <casey@xxxxxxxxxxxxxxx> > > > > Tested-by: Mike Ralphson <mike@xxxxxxxxxxxx> > > Both with and without -D_LARGE_FILES. Many thanks. > > H.Merijn, is this change also ok for your HP-UX? I'm not really actively following the ML anymore, as it is kinda busy :) I was able to compile/test/install 1.5.5.1 on HP-UX 11.00/32 and on HP-UX 11.23-ilp64 with a reasonable small set of additional changes. Main thing I found to make most tests that used to fail now pass is to use bash, instead of HP's native POSIX shell. http://www.xs4all.nl/~procura/git-1.5.5.1-11.00.diff http://www.xs4all.nl/~procura/git-1.5.5.1-11.23.diff We - as a company - now actively use git on HP-UX 11.00 and Linux > I guess there may still be a case for not defining _LARGE_FILES by > default on AIX as all the warnings may be off-putting or mask other > issues. I also have AIX, but I hate it, and don't really care about it. It's just that we have some poor customers whose IT people forced this OS upon them. > Maybe instead having a comment for those who need large pack-file > support? Will submit amended Makefile patch if there's interest. -- H.Merijn Brand Amsterdam Perl Mongers (http://amsterdam.pm.org/) using & porting perl 5.6.2, 5.8.x, 5.10.x on HP-UX 10.20, 11.00, 11.11, & 11.23, SuSE 10.1 & 10.2, AIX 5.2, and Cygwin. http://qa.perl.org http://mirrors.develooper.com/hpux/ http://www.test-smoke.org http://www.goldmark.org/jeff/stupid-disclaimers/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
