Hi Brandon, On Tue, Jun 01, 2010 at 08:55:36PM -0500, Brandon Casey wrote: > From: Gary V. Vaughan <git@xxxxxxxxxxxxxxxxxxxxxxxxx> > > IRIX 6.5.26m does not define the 'sgi' macro, but it does define an '__sgi' > macro. Since later IRIX versions (6.5.29m) define both macros, and since > an underscore prefixed macro is preferred anyway, use '__sgi' to detect > compilation on SGI IRIX. > > Signed-off-by: Brandon Casey <casey@xxxxxxxxxxxxxxx> Signed-off-by: Gary V. Vaughan <gary@xxxxxxxxxxxxxxxxxx> > Here's a resubmit of your patch with an updated commit message. Since the > __sgi macro seems to be more common than the sgi macro, I modified your > patch to check for only the __sgi macro. I know Junio suggested checking > for both macros, but I'm the one that Junio was talking about when he made > his comments, and __sgi works for me. Sure, that's fine. I only need the __sgi part for git to compile correctly on my IRIX machines, so if you feel that the non-underscore flavour is superfluous then I won't miss it! ;) > It's such a simple little patch, but still I removed your signed-off-by line > since I modified the patch, and I am not you. Please do reply to this message > with a signed-off-by if you sign off on the patch. Junio will add it to the > patch when he applies the it. Done. Cheers, -- Gary V. Vaughan (gary@xxxxxxxxxxxxxxxxxx) -- 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