[PATCH 1/2] build: generate sh.config for feeding configure results to shell scripts

W. Trevor King wking at tremily.us
Fri May 30 08:42:21 PDT 2014


On Fri, May 30, 2014 at 10:43:04AM +0300, Jani Nikula wrote:
> Only include the relevant information.

I prefer this approach too.

> +# Whether the Xapian version in use supports compaction
> +_HAVE_XAPIAN_COMPACT=${have_xapian_compact}

Why the leading underscore?  Are we worried about colliding with
existing HAVE_XAPIAN_COMPACT variables?  Python conventionally uses
the leading underscore for private variables, but these are certainly
meant to be used by the sourcing script.

Cheers,
Trevor

-- 
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20140530/d9474e84/attachment.pgp>


More information about the notmuch mailing list