> From: Junio C Hamano [mailto:gitster@xxxxxxxxx] > Sent: Friday, August 10, 2012 6:21 PM > To: Joachim Schmitz > Cc: git@xxxxxxxxxxxxxxx; rsbecker@xxxxxxxxxxxxx > Subject: Porting to a new platform > > "Joachim Schmitz" <jojo@xxxxxxxxxxxxxxxxxx> writes: > > > - HP NonStop is lacking poll(),... > > - HP NonStop is lacking getrlimit(), fsync(), setitimer()... > > I would check compat/win32 and friends and see what other platforms that lack > this and that do, if I were you. Hmm, in compat/win32/poll.c I found exactly the same code I stole' for my implementation (GNUlib's implementation) so I just managed to reinvent the wheel :-( Thanks anyway for telling me about it. For getrlimit(RLIMIT_NOFILE, ...), I'm now using sysconf(_SC_OPEN_MAX), does that sound reasonable? I found no replacement for fsync() and setitimer(), but have my ones since long, so no real need. Also would compat/pread.c, another API HP NonStop is missing, but I had my own implementation for that since quite a while already (and it looks pretty similar to git's one). I don't quite understand though why neither compat/pread.c nor compat/win32/poll.c are used automatically after having been proven absent in configure? Ahh, I see, it could be done by adding a HP NonStop specific section in Makefile ("ifeq ($(uname_S),NONSTOP_KERNEL)"), right? I'll have a deeper look and see whether I can come up with something useful to feed back into git. > > so telling configure to search for c99 should help here. > > In general, the top-level Makefile is designed to be usable without ever > worrying about "configure" mess. Just define CC for the platform section there, Yes, that's what I did, sort of, I just set CC to c99 prior to executing configure. I've seen other configure though, that explicitly test for C99, so why not this one? > and optionally add a support to flip the same in configure.ac. This applies > equally to other conditional compilation options you may have to add to > support your platform. Bye, Jojo -- 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