Hi Brandon,

On Tue, Jun 01, 2010 at 08:55:36PM -0500, Brandon Casey wrote:
quoted text
> From: Gary V. Vaughan <git@mlists.thewrittenword.com>
> 
> 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@nrlssc.navy.mil>

Signed-off-by: Gary V. Vaughan <gary@thewrittenword.com>

quoted text
> 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! ;)

quoted text
> 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@thewrittenword.com)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html