From: Ali Polatel Date: Thu, 16 May 2013 14:49:05 +0000 (+0300) Subject: Re: [PATCH v2] debian: package ruby bindings X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=41a08b41cc4cecfe85f873466496586017f52b13;p=notmuch-archives.git Re: [PATCH v2] debian: package ruby bindings --- diff --git a/d3/f4f1cdd28a072b6902966b68a4d02cd0ba8a3d b/d3/f4f1cdd28a072b6902966b68a4d02cd0ba8a3d new file mode 100644 index 000000000..f8b253413 --- /dev/null +++ b/d3/f4f1cdd28a072b6902966b68a4d02cd0ba8a3d @@ -0,0 +1,99 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by olra.theworths.org (Postfix) with ESMTP id 1D23D431FD5 + for ; Thu, 16 May 2013 07:49:34 -0700 (PDT) +X-Virus-Scanned: Debian amavisd-new at olra.theworths.org +X-Spam-Flag: NO +X-Spam-Score: -0.799 +X-Spam-Level: +X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5 + tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, + FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled +Received: from olra.theworths.org ([127.0.0.1]) + by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id iFKagUwJ+Krd for ; + Thu, 16 May 2013 07:49:28 -0700 (PDT) +Received: from mail-lb0-f171.google.com (mail-lb0-f171.google.com + [209.85.217.171]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) + (No client certificate requested) + by olra.theworths.org (Postfix) with ESMTPS id A44F4431FC3 + for ; Thu, 16 May 2013 07:49:27 -0700 (PDT) +Received: by mail-lb0-f171.google.com with SMTP id v20so3234281lbc.16 + for ; Thu, 16 May 2013 07:49:26 -0700 (PDT) +DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; + h=x-received:mime-version:in-reply-to:references:from:date:message-id + :subject:to:cc:content-type; + bh=nNbCk3pwlopnjjvWIUHfcAoyy3Je+LUe0FukVLFDgAA=; + b=gyVMVJcRHHl1NAMqT6x0AgrZLaRq+NmxmNEeQTtyvW2l9944fvHBaspWpSigUPOKf8 + 3MS/MO2VsHECP6CJxFCBZ+nes2jPMZF8qnDW7F1VDjmCnFXDf/snlFhOQnuC8/s2lnIr + osABBFXCL1aAfu0VjC9KnXtw9khqL5ZK1iltfQwgG1htnsj+NHzs/JDwV8diGQSkAeOT + yGZPKcPQqYPH2y/iPUydwl1xcYqf5yo0G8A6Bvy4UMaTSRIY6baPUvbgY/Q1OvSUNYYG + jbOFKNMqqnDiHwrr29H2D4nWSl0odaD2nsKYauG3iY7/CcU4XpUurZCX8P3RUApAr+kA + KtLQ== +X-Received: by 10.112.89.168 with SMTP id bp8mr3408728lbb.50.1368715766084; + Thu, 16 May 2013 07:49:26 -0700 (PDT) +MIME-Version: 1.0 +Received: by 10.112.130.39 with HTTP; Thu, 16 May 2013 07:49:05 -0700 (PDT) +In-Reply-To: <878v3fc7xn.fsf@convex-new.cs.unb.ca> +References: <1366139319-6985-1-git-send-email-felipe.contreras@gmail.com> + <87ehd7c9sk.fsf@convex-new.cs.unb.ca> + + <878v3fc7xn.fsf@convex-new.cs.unb.ca> +From: Ali Polatel +Date: Thu, 16 May 2013 17:49:05 +0300 +Message-ID: + +Subject: Re: [PATCH v2] debian: package ruby bindings +To: David Bremner +Content-Type: text/plain; charset=ISO-8859-1 +Cc: notmuch@notmuchmail.org +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.13 +Precedence: list +List-Id: "Use and development of the notmuch mail system." + +List-Unsubscribe: , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: , + +X-List-Received-Date: Thu, 16 May 2013 14:49:34 -0000 + +2013/5/16 David Bremner : +> Felipe Contreras writes: +>> +>> Yes. Do you have an old version of notmuch installed? +>> +> +> Yes I do. Hmm. It still seems like a bug to me; shouldn't the include +> paths be ordered to find the right include files first? +> +> d + +Well I must accept I haven't spent much time on thinking how to make +extconf.rb perfect. +Excerpt from extconf.rb from bindings/ruby: + +find_header('notmuch.h', '../../lib') +find_library('notmuch', 'notmuch_database_create', '../../lib') + +Two problems here: +1. The library must be built before the bindings are built otherwise +the installed version is used. + However the installed version should never be used (or must be +made a non-default option somehow) +2. The library must be built in the source tree. Out-of-source tree +builds will cause extconf.rb to look for the installed version too. + Not sure whether this is a major concern because I don't know +whether out-of-tree builds are supported by the build system. + +Having said that, I will try to have a look at solving these issues +this weekend. +Unfortunately I can't make promises because real-life is killing all +my time lately. + + -alip