From: Kim Minh Kaplan <kimminh.kaplan+nomuch@afnic.fr>
To: Tomi Ollila <tomi.ollila@iki.fi>
Subject: Re: [PATCH 1/3] Add notmuch_database_close_compact
References: <1345476704-17091-1-git-send-email-bgamari.foss@gmail.com>
	<1345476704-17091-2-git-send-email-bgamari.foss@gmail.com>
	<m27gssitot.fsf@guru.guru-group.fi> <87y5l8z4dw.fsf@gmail.com>
	<m2txvv794d.fsf@guru.guru-group.fi>
Date: Thu, 23 Aug 2012 05:56:57 +0000
In-Reply-To: <m2txvv794d.fsf@guru.guru-group.fi> (Tomi Ollila's message of
	"Wed, 22 Aug 2012 09:07:14 +0300")
Message-ID: <87haruxiae.fsf@kaki.tech.prive.nic.fr>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Cc: notmuch <notmuch@notmuchmail.org>
Precedence: list

Tomi Ollila=C2=A0:

> On Tue, Aug 21 2012, Ben Gamari wrote:
>
>> Eh? 1.2.6 is the first Xapian release to have Compactor exposed in the
>> public API.
>
> Presuming that those variables are always numeric the comparison could be:
>
> if [ ${xapian_major_version} -gt 1 ] ||=20
>    [ ${xapian_major_version} -eq 1 -a ${xapian_minor_version} -gt 2 ] ||=
=20
>    [ ${xapian_major_version} -eq 1 -a ${xapian_minor_version} -eq 2 -a \
>      ${xapian_subminor_version} -ge 6 ]; then
>
> (I could not figure out anything shorter and/or cleaner)

Try:

    case "${xapian_version}" in
         0.*|1.[01].*|1.2.[0-5]) ;;
         *) ... ;;
    esac

Kim Minh.