Re: [PATCH 3/4] ruby: Add workarounds to use in-tree build not the installed one

diff --git a/83/2bc712dc6a661153f7446623342d32cd334b56 b/83/2bc712dc6a661153f7446623342d32cd334b56
new file mode 100644 (file)
index 0000000..43d74f7
--- /dev/null
+++ b/83/2bc712dc6a661153f7446623342d32cd334b56
@@ -0,0 +1,113 @@
+Return-Path: <felipe.contreras@gmail.com>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 06625431FCF\r
+       for <notmuch@notmuchmail.org>; Thu, 24 May 2012 01:57:01 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.799\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
+       tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
+       FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id jH6neqJPXVkf for <notmuch@notmuchmail.org>;\r
+       Thu, 24 May 2012 01:57:00 -0700 (PDT)\r
+Received: from mail-lpp01m010-f53.google.com (mail-lpp01m010-f53.google.com\r
+       [209.85.215.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id 1EA44431FC4\r
+       for <notmuch@notmuchmail.org>; Thu, 24 May 2012 01:56:59 -0700 (PDT)\r
+Received: by lagu2 with SMTP id u2so6662211lag.26\r
+       for <notmuch@notmuchmail.org>; Thu, 24 May 2012 01:56:58 -0700 (PDT)\r
+DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
+       h=mime-version:in-reply-to:references:date:message-id:subject:from:to\r
+       :cc:content-type;\r
+       bh=xaoNbORwsILS5aO/xxwsRrDLMGJZ1LEprtWP+cL9XO8=;\r
+       b=fYgrxjmxM4QYAtJcAdt7TGBOJPq1gnqpdurc77ATb5FXByKCWGj3f6mIN7EL4VsrYW\r
+       5of9URK8Pd4R5k6/FOo6qk56NAnxifkDbU1Fd7mz0SCUAEF7OriNdjtWP8Sd/2SAZaHW\r
+       ndeh0Gs64vZ0gikNjhLLjjttX1H24x9N6pE5QoO9wFxDaXOXQmiptt0VpfVXmUrVTSq5\r
+       79TN54YcocI4GPysZ3xdWPt83lJF6nc7tRgMuyuknZfEjxkhSiE0tgPoH2dkrmgBB19M\r
+       HLZFsSVC2VY5ESenXj2nGIIjmJjYnGOKrZ2Ng0/NusAGfteYLbleKKA4EaQvPIq2Fdrj\r
+       HanQ==\r
+MIME-Version: 1.0\r
+Received: by 10.152.108.38 with SMTP id hh6mr29784900lab.28.1337849818389;\r
+       Thu, 24 May 2012 01:56:58 -0700 (PDT)\r
+Received: by 10.112.107.65 with HTTP; Thu, 24 May 2012 01:56:58 -0700 (PDT)\r
+In-Reply-To: <878vgi1ign.fsf@zancas.localnet>\r
+References: <87txzsgs4g.fsf@zancas.localnet>\r
+       <cover.1336402558.git.alip@exherbo.org>\r
+       <1559639bc5e9e3253f2ee2220a8d8d00c39d9d43.1336402558.git.alip@exherbo.org>\r
+       <CAMP44s15Q6F9bA0MirS3zyBU3AqomH7wKBmLvqEtAnBnzHrw=g@mail.gmail.com>\r
+       <878vgi1ign.fsf@zancas.localnet>\r
+Date: Thu, 24 May 2012 10:56:58 +0200\r
+Message-ID:\r
+ <CAMP44s2ATDkix0OKEc6c9pZrudnYk=7tCme=d+VJcfeivFRhew@mail.gmail.com>\r
+Subject: Re: [PATCH 3/4] ruby: Add workarounds to use in-tree build not the\r
+       installed one\r
+From: Felipe Contreras <felipe.contreras@gmail.com>\r
+To: David Bremner <david@tethera.net>\r
+Content-Type: text/plain; charset=UTF-8\r
+Cc: Ali Polatel <alip@exherbo.org>, notmuch@notmuchmail.org\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Thu, 24 May 2012 08:57:01 -0000\r
+\r
+On Thu, May 24, 2012 at 3:22 AM, David Bremner <david@tethera.net> wrote:\r
+> Felipe Contreras <felipe.contreras@gmail.com> writes:\r
+>\r
+>\r
+>> I don't see how this patch could be fixed properly easily, and it was\r
+>> labeled as a hack, and I didn't like it in the first place anyway, so\r
+>> I'm going to revert it by tomorrow if I don't hear any good reason not\r
+>> to.\r
+>\r
+> I think this highlights the need for at least a minimal test suite for\r
+> the ruby\r
+\r
+Yeap, I thought the same.\r
+\r
+> I do plan on a bug fix release, to fix an annoying emacs interface bug\r
+> if nothing else. I'd rather see a fix/revert coordinated with Ali in his\r
+> role as ruby bindings maintainer.\r
+\r
+Well, sure, if possible that would be best. But I doubt the ruby\r
+bindings maintainer would complain about making the Ruby bindings work\r
+again :)\r
+\r
+Worst-case scenario we would end up with what we had before.\r
+\r
+> FWIW, the previous situation of linking with the installed version of\r
+> notmuch sounds somewhat broken as well, although obviously preferable to\r
+> not working at all.\r
+\r
+That would be only if there was installed version of notmuch, which\r
+some people don't have (e.g. me).\r
+\r
+Either way, to fix that particular problem we don't need to link to\r
+the static library, we can just the right CFLAGS and LDFLAGS, in fact,\r
+I wonder how is it that it's not working properly; AFAIK the linker\r
+and compiler would use first the headers and libraries found in these\r
+flags, and only as a fallback use the system ones, so if anybody has\r
+managed to compile before (I have) the headers and libraries would be\r
+in some -I/-L path.\r
+\r
+I could try to investigate, but at the moment I still see no reason\r
+not to revert.\r
+\r
+Cheers.\r
+\r
+-- \r
+Felipe Contreras\r