Re: [PATCH 1/4] Support builds when sys/param.h is missing

2012-12-15 Thread Junio C Hamano
David Michael writes: > On Fri, Dec 14, 2012 at 6:41 PM, Junio C Hamano wrote: >> I have this suspicion that nobody would notice if we simply stopped >> including the header. > > While I'm not aware of any subtleties it could be causing on other > platforms, it does seem fine to drop sys/param.h

Re: [PATCH 1/4] Support builds when sys/param.h is missing

2012-12-14 Thread David Michael
Hi, On Fri, Dec 14, 2012 at 6:41 PM, Junio C Hamano wrote: > I have this suspicion that nobody would notice if we simply stopped > including the header. While I'm not aware of any subtleties it could be causing on other platforms, it does seem fine to drop sys/param.h on my test GNU/Linux system

Re: [PATCH 1/4] Support builds when sys/param.h is missing

2012-12-14 Thread Junio C Hamano
David Michael writes: > An option is added to the Makefile to skip the inclusion of sys/param.h. > The only known platform with this condition thus far is the z/OS UNIX System > Services environment. > > Signed-off-by: David Michael > --- Hmm, makes me wonder if we can remove that inclusion eve

[PATCH 1/4] Support builds when sys/param.h is missing

2012-12-14 Thread David Michael
An option is added to the Makefile to skip the inclusion of sys/param.h. The only known platform with this condition thus far is the z/OS UNIX System Services environment. Signed-off-by: David Michael --- Makefile | 5 + configure.ac | 6 ++ git-compat-util.h | 2 ++ 3 file