Re: [PATCH] fix build without sfsexp

Subject: Re: [PATCH] fix build without sfsexp

Date: Fri, 15 Apr 2022 14:31:46 -0300


Cc: Michael J Gruber

From: David Bremner writes:

> From: Michael J Gruber <>
> a1d139de ("lib: add sexp: prefix to Xapian (infix) query parser.",
> 2022-04-09) introduced sfsexp infix queries. This requires the infix
> preprocessor to be built in in a way which does not require sfsexp when
> notmuch is built without it.
> Make the preprocessor throw a Xapian error in this case (and fix the
> build).
> Signed-off-by: Michael J Gruber <>
> ---
> Maybe something like this - I have not test *with* sfsexp, though.

Either your patch or mine is OK with me. Yours might be slightly nicer
since it gives an actual error message, rather than falling back to the
corresponding phrase (generic Xapian query parser behaviour with unknown

I did test both with and without libsfsexp-dev installed.

notmuch mailing list --
To unsubscribe send an email to