Subject: Re: bug#43754: [PATCH] configure help message of --with-json

From: Eli Zaretskii <[email-masked]>
Date: Sat, 03 Oct 2020 20:59:19 +0300

> From: Lars Ingebrigtsen <[email-masked]>
> Cc: [email-masked],  [email-masked]
> Date: Sat, 03 Oct 2020 19:49:33 +0200
> 
> Oh, hey, whaddayaknow, it was a patch I applied a couple months ago:
> 
> commit 43091e6c5069797ba17d2c7429e0122d3a5337d9
> Author:     Noam Postavsky <[email-masked]>
> AuthorDate: Fri Aug 14 19:31:16 2020 +0200
> Commit:     Lars Ingebrigtsen <[email-masked]>
> CommitDate: Fri Aug 14 19:31:16 2020 +0200
> 
>     Make configure say so if we have "--with-json" but no jansson support
>     
>     * configure.ac (OPTION_DEFAULT_IFAVAILABLE): New macro.  Use it to
>     define the --with-json option.  Add with_json and HAVE_JSON to the
>     'MISSING' checks (bug#39953).
> 
> bug#39953 has the rationale behind the change.

What I said there notwithstanding, it looks like --with-json is the
odd one out nowadays, as we handle all the other optional libraries
differently.  So it seems to me that we should switch to DEFAULT_ON
for this library.

Obviously, it is not a catastrophe to leave it as it is now...