Re: [PATCH] configure: only install bash completion if supported

Subject: Re: [PATCH] configure: only install bash completion if supported

Date: Sun, 02 Feb 2014 19:18:31 -0400

To: Jani Nikula, Mark Walters, notmuch@notmuchmail.org

Cc:

From: David Bremner


Jani Nikula <jani@nikula.org> writes:

> On Sun, 02 Feb 2014, David Bremner <david@tethera.net> wrote:
>> Mark Walters <markwalters1009@gmail.com> writes:
>>
>>> This LGTM (untested)
>>
>> I did test it, at least completely removing bash completion works as
>> expected.  
>>
>> Unfortunately --with-bash-completion does not override this test because
>> the order things are processed. Do you think this is a bug?  I wondered
>> if users that "know what they are doing" (TM) might want to force
>> installation even though the pkg-config test fails.
>
> I thought it was a feature, not a bug, but I'm fine either way.

I can live with the current patch. As you pointed out on IRC, this is
the usual way missing dependencies work.  And I think we should avoid
extra complications in the configure script when we can.

Thread: