On Tue, Apr 27, 2010 at 16:39, Gary V. Vaughan <git@xxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > On Tue, Apr 27, 2010 at 04:10:56PM +0200, Tor Arntsen wrote: >> but (ref. a patch I sent earlier, in its >> first revision) I needed several other changes in order to compile on >> 5.1 (I also needed those you have above), e.g. >> -D_POSIX_C_SOURCE=1 NO_STRCASESTR = YesPlease NO_STRTOUMAX = YesPlease >> NO_STRLCPY = YesPlease NO_MEMMEM = YesPlease NO_UINTMAX_T = YesPlease >> NO_C99_FORMAT = YesPlease >> and >> NO_TCLTK = YesPlease > > Indeed, I don't mean to imply that this patch is all that is needed > for OSF1, merely that the changes it contains benefit only OSF1. I see. In that case, as I have verified that those changes are indeed necessary for OSF1: Acked-by: Tor Arntsen <tor@xxxxxxxxxxx> > Please bear in mind that I build using configure, and may have missed > something necessary for a make-only driven build. Ah, Ok, that would explain it. I too found that with ./configure fewer changes were needed. The patch I was working on I tested both with and without configure. If your patch goes in I could follow up with the changes needed for when not using configure, if you don't wish to tacke that. >> because V5.1 comes with tcl/tk 8.2 and gitk refuses to start with >> anything older than 8.4. >> I had some other too, but those could be worked around by building and >> installing own versions of iconv, openssl, zlib, python, expat. > > Ack. And since that is our business, we do indeed build git against > our own packages of relatively cutting edge releases of prerequisite > OS software on all our hosts (including all of the above). > > Cheers, > -- > Gary V. Vaughan (gary@xxxxxxxxxxxxxxxxxx) -Tor -- 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