David Michael <fedora.dm0@xxxxxxxxx> writes: > On Fri, Dec 14, 2012 at 6:41 PM, Junio C Hamano <gitster@xxxxxxxxx> 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 > systems. > > I can resend the series and just remove the include, if preferred. I am sure the patch as posted is much safer, but I was hoping that folks on non-Linux platforms may say "I tried to compile with the include removed, and I get identical binaries from before". Until that happens, I would prefer queuing your patch as-is. Thanks. -- 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