Re: [PATCH 2/5] configure, test: Added variables for paths to true and false.

Subject: Re: [PATCH 2/5] configure, test: Added variables for paths to true and false.

Date: Thu, 08 May 2014 21:32:39 +0900

To: Charles Celerier, notmuch@notmuchmail.org

Cc:

From: David Bremner

Charles Celerier <cceleri@cs.stanford.edu> writes:

> The path to true may not be the same on all platforms (e.g. on Mac OS X
> it is /usr/bin/true), so the hard-coded path of /bin/true is not
> portable. This is resolved by adding a step to the configure script to
> locate the path of true and to use the TRUE variable wherever /bin/true
> was needed. The same was done for false.

Does this really need a configure script variable? It seems like
overkill.

d

• Previous message (by thread): Re: [PATCH v2 4/5] T360-symbol-hiding: Added code to support testing on Mac OS X.

Thread:

• Charles Celerier—[PATCH 0/5] Improving portability to Mac OS X [inbox, unread]
  • Charles Celerier—[PATCH 1/5] test/Makefile.local: Added configured TALLOC_LDFLAGS. [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
    • Charles Celerier—[PATCH 2/5] configure, test: Added variables for paths to true and false. [inbox, notmuch::patch, notmuch::test, notmuch::wontfix, unread]
      • Charles Celerier—[PATCH 3/5] atomicity.gdb: Allow breakpoint symbols to be resolved later. [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
        • Charles Celerier—[PATCH 4/5] T360-symbol-hiding: Added code to support testing on Mac OS X. [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
          • Charles Celerier—[PATCH 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
          • Tomi Ollila—Re: [PATCH 4/5] T360-symbol-hiding: Added code to support testing on Mac OS X. [inbox, notmuch::review, unread]
            • Charles Celerier—Re: [PATCH 4/5] T360-symbol-hiding: Added code to support testing on Mac OS X. [inbox, signed, unread]
              • Tomi Ollila—Re: [PATCH 4/5] T360-symbol-hiding: Added code to support testing on Mac OS X. [inbox, unread]
        • Tomi Ollila—Re: [PATCH 3/5] atomicity.gdb: Allow breakpoint symbols to be resolved later. [inbox, unread]
      • David Bremner—Re: [PATCH 2/5] configure, test: Added variables for paths to true and false. [inbox, unread]
        • Charles Celerier—Re: [PATCH 2/5] configure, test: Added variables for paths to true and false. [inbox, signed, unread]
          • David Bremner—Re: [PATCH 2/5] configure, test: Added variables for paths to true and false. [inbox, unread]
    • David Bremner—Re: [PATCH 1/5] test/Makefile.local: Added configured TALLOC_LDFLAGS. [inbox, unread]
  • Charles Celerier—[PATCH v2 4/5] T360-symbol-hiding: Added code to support testing on Mac OS X. [inbox, notmuch::moreinfo, notmuch::patch, notmuch::test, unread]
    • Charles Celerier—[PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, notmuch::patch, notmuch::test, notmuch::wontfix, unread]
      • David Bremner—Re: [PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, unread]
        • Charles Celerier—Re: [PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, signed, unread]
          • David Bremner—Re: [PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, unread]
            • Charles Celerier—Re: [PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, signed, unread]
              • David Bremner—Re: [PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, unread]
                • Charles Celerier—Re: [PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, signed, unread]
                  • David Bremner—Re: [PATCH v2 5/5] T360-symbol-hiding: Use nm instead of objdump. [inbox, unread]
    • David Bremner—Re: [PATCH v2 4/5] T360-symbol-hiding: Added code to support testing on Mac OS X. [inbox, unread]