MIME-Version: 1.0
In-Reply-To: <87txgpv16s.fsf@zancas.localnet>
References: <1380558523-6913-1-git-send-email-felipe.contreras@gmail.com>
	<87txgpv16s.fsf@zancas.localnet>
Date: Tue, 15 Oct 2013 00:25:11 -0500
Message-ID:
 <CAMP44s390OhBrqwoKh9F5v5+6+r+jWBpvHZ2vS9mznjjW=dMFw@mail.gmail.com>
Subject: Re: [PATCH 0/3] General fixes
From: Felipe Contreras <felipe.contreras@gmail.com>
To: David Bremner <david@tethera.net>
Content-Type: text/plain; charset=UTF-8
Cc: "notmuch@notmuchmail.org" <notmuch@notmuchmail.org>
Precedence: list

On Thu, Oct 10, 2013 at 6:02 AM, David Bremner <david@tethera.net> wrote:
> Felipe Contreras <felipe.contreras@gmail.com> writes:
>
>> Felipe Contreras (3):
>>   query: bind queries to database objects
>>   ruby: allow build with RUNPATH
>>   ruby: bind database close()/destroy() properly
>
> Hi Felipe;
>
> I agree with the discussion on IRC that the change in the first patch
> makes sense.
>
> It would be nice to have a bit more explanation in the commit message of
> the second message; at least to point out that (iiuc) this is not a
> really a ruby specific thing, just a standard ld.so feature.

Yes, that is true, and an option libnotmuch.so already uses, so I
thought it made sense that Ruby's notmuch.so used it as well.

-- 
Felipe Contreras