git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tor Arntsen <tor@spacetec.no>
To: Brandon Casey <casey@nrlssc.navy.mil>
Cc: gitster@pobox.com, git@vger.kernel.org, drafnel@gmail.com,
	"Gary V. Vaughan" <git@mlists.thewrittenword.com>,
	"Gary V. Vaughan" <gary@thewrittenword.com>
Subject: Re: [PATCH] git-compat-util.h: use apparently more common __sgi macro  to detect SGI IRIX
Date: Wed, 2 Jun 2010 11:56:56 +0200	[thread overview]
Message-ID: <AANLkTinvmNM_nKQnZNgQMWstrevG4RfcjW7RdCILVLYF@mail.gmail.com> (raw)
In-Reply-To: <4gJWLBIdRVtK7gq22o-LadCixyvOA-9FlEvk-ZrCHyUsGwArvR9oPkV-qlez-T-Nsn0v6UO633Y@cipher.nrlssc.navy.mil>

On Wed, Jun 2, 2010 at 03:55, Brandon Casey <casey@nrlssc.navy.mil> wrote:
> 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.

Yes, __sgi is the one to use. It's been there on all the SGI systems
I've used, at least back to IRIX 5.3 (I don't recall for sure about
4.0.5 but my guess is yes). 'sgi' is often there too, also on IRIX 6.2
(with the old MIPS compiler), but on both 6.2 and 6.5 it goes away if
you use certain other options, e.g. -ansi. __sgi, on the other hand,
is always there - it doesn't depend on any other compiler options.

-Tor

  parent reply	other threads:[~2010-06-02  9:57 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-14  9:31 [PATCH v5 00/18] Portability patches for git-1.7.1 Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 01/18] Makefile: pass CPPFLAGS through to fllow customization Gary V. Vaughan
2010-05-14  9:53   ` Robin H. Johnson
2010-05-14 10:58     ` Gary V. Vaughan
2010-05-14 11:04       ` Robin H. Johnson
2010-05-14 12:01         ` Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 02/18] Rewrite dynamic structure initializations to runtime assignment Gary V. Vaughan
2010-06-02  4:39   ` Junio C Hamano
2010-05-14  9:31 ` [PATCH v5 03/18] Makefile: -lpthread may still be necessary when libc has only pthread stubs Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 04/18] enums: omit trailing comma for portability Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 05/18] Do not use "diff" found on PATH while building and installing Gary V. Vaughan
2010-06-02  4:39   ` Junio C Hamano
2010-05-14  9:31 ` [PATCH v5 06/18] tests: use "test_cmp", not "diff", when verifying the result Gary V. Vaughan
2010-06-02  4:39   ` Junio C Hamano
2010-05-14  9:31 ` [PATCH v5 07/18] test_cmp: do not use "diff -u" on platforms that lack one Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 08/18] git-compat-util.h: some platforms with mmap() lack MAP_FAILED definition Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 09/18] Makefile: some platforms do not have hstrerror anywhere Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 10/18] Make NO_{INET_NTOP,INET_PTON} configured independently Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 11/18] Some platforms lack socklen_t type Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 12/18] Allow disabling "inline" Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 13/18] inline declaration does not work on AIX Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 14/18] Makefile: SunOS 5.6 portability fix Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 15/18] git-compat-util.h: Irix 6.5 defines sgi but not __sgi Gary V. Vaughan
2010-06-02  1:55   ` [PATCH] git-compat-util.h: use apparently more common __sgi macro to detect SGI IRIX Brandon Casey
2010-06-02  8:43     ` Gary V. Vaughan
2010-06-02  9:56     ` Tor Arntsen [this message]
2010-05-14  9:31 ` [PATCH v5 16/18] Makefile: HPUX11 portability fixes Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 17/18] Makefile: HP-UX 10.20 " Gary V. Vaughan
2010-05-14  9:31 ` [PATCH v5 18/18] Makefile: Tru64 portability fix Gary V. Vaughan
2010-05-26  5:56 ` [PATCH v5 00/18] Portability patches for git-1.7.1 Gary V. Vaughan
2010-06-07 15:45 ` Gary V. Vaughan
2010-06-07 18:07   ` Junio C Hamano
2010-06-09  9:37     ` Tor Arntsen
2010-06-11  4:30       ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=AANLkTinvmNM_nKQnZNgQMWstrevG4RfcjW7RdCILVLYF@mail.gmail.com \
    --to=tor@spacetec.no \
    --cc=casey@nrlssc.navy.mil \
    --cc=drafnel@gmail.com \
    --cc=gary@thewrittenword.com \
    --cc=git@mlists.thewrittenword.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).