Re: [notmuch] Notmuch shared library
authorCarl Worth <cworth@cworth.org>
Thu, 1 Apr 2010 07:54:16 +0000 (00:54 +1700)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:36:27 +0000 (09:36 -0800)
0e/1e2eb5e0b92eff397cf1a80450b49a5dd3e48c [new file with mode: 0644]

diff --git a/0e/1e2eb5e0b92eff397cf1a80450b49a5dd3e48c b/0e/1e2eb5e0b92eff397cf1a80450b49a5dd3e48c
new file mode 100644 (file)
index 0000000..3adf3db
--- /dev/null
@@ -0,0 +1,113 @@
+Return-Path: <cworth@cworth.org>\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 2EFCD4196F0\r
+       for <notmuch@notmuchmail.org>; Thu,  1 Apr 2010 00:54:23 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -1.49\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-1.49 tagged_above=-999 required=5\r
+       tests=[ALL_TRUSTED=-1, BAYES_05=-0.5, T_MIME_NO_TEXT=0.01]\r
+       autolearn=ham\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 bumISTLIK8Dr; Thu,  1 Apr 2010 00:54:22 -0700 (PDT)\r
+Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 644D5431FC1;\r
+       Thu,  1 Apr 2010 00:54:22 -0700 (PDT)\r
+Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
+       id 1CBBE25400C; Thu,  1 Apr 2010 00:54:22 -0700 (PDT)\r
+From: Carl Worth <cworth@cworth.org>\r
+To: Ingmar Vanhassel <ingmar@exherbo.org>, notmuch@notmuchmail.org\r
+In-Reply-To: <1268401656-12827-1-git-send-email-ingmar@exherbo.org>\r
+References: <1268347072-2050-1-git-send-email-bgamari.foss@gmail.com>\r
+       <1268401656-12827-1-git-send-email-ingmar@exherbo.org>\r
+Date: Thu, 01 Apr 2010 00:54:16 -0700\r
+Message-ID: <8739zfip5j.fsf@yoom.home.cworth.org>\r
+MIME-Version: 1.0\r
+Content-Type: multipart/signed; boundary="=-=-=";\r
+       micalg=pgp-sha1; protocol="application/pgp-signature"\r
+Subject: Re: [notmuch] Notmuch shared library\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, 01 Apr 2010 07:54:23 -0000\r
+\r
+--=-=-=\r
+\r
+On Fri, 12 Mar 2010 14:47:33 +0100, Ingmar Vanhassel <ingmar@exherbo.org> wrote:\r
+> I'd very much like to see this upstream.\r
+\r
+Me too. ;-) Sorry for the long delay.\r
+\r
+> I amended your first patch to make it install the notmuch.h header\r
+> too.\r
+\r
+A very nice addition.\r
+\r
+> The second patch fixed some parallel make issue I had while testing\r
+> your series.\r
+\r
+As is this. I ran into this problem immediately when testing the first\r
+patch.\r
+\r
+> The third patch adds a configure switch to configure where to install\r
+> libraries. It has one issue I'm aware off. Setting --prefix=/usr\r
+> without setting --libdir does the wrong thing. This should probably be\r
+> fixed, but I don't have any immediate ideas how to do this.\r
+\r
+This configure option is nice too, but the problem you pointed out was\r
+serious, (so, thanks for noting it). I went ahead and fixed this.\r
+\r
+I also spent some time making sure I (think I) understand how library\r
+sonames and library versions work. I did several investigations with\r
+libtool that convinced me it adds problems without any tangible benefit,\r
+(at least for the target platforms I'm aware of people using with\r
+notmuch).\r
+\r
+I finished this by establishing a 3-part library interface version,\r
+(initially 1.0.0), along with careful documentation in the Makefile on\r
+how to increment it. And of course, I ended up touching up about a\r
+half-dozen issues in the Makefiles along the way.\r
+\r
+We'll still want to add functions/macros to the library to make the\r
+library version available to the user. And I do want to still add some\r
+symbol hiding and potentially some symbol versioning to the library. But\r
+I'm not making the patch series block on that.\r
+\r
+Finally, I'm a tiny bit annoyed that now after a fresh checkout of\r
+notmuch and "make" that one can't easily run ./notmuch without either\r
+installing the library (or fiddling with LD_LIBRARY_PATH). I've got some\r
+ideas on how to simplify that, but I'm not sure if any are good or worth\r
+it.\r
+\r
+Anyway, this patch series is now pushed with my changes on top. Thanks,\r
+Ben and Ingmar!\r
+\r
+-Carl\r
+\r
+\r
+\r
+\r
+--=-=-=\r
+Content-Type: application/pgp-signature\r
+\r
+-----BEGIN PGP SIGNATURE-----\r
+Version: GnuPG v1.4.10 (GNU/Linux)\r
+\r
+iD8DBQFLtFEo6JDdNq8qSWgRAr6lAJ99xHAJ4x0kg8MFSy1TvqMaKSwgyACfUQkA\r
+EedHujwOW7z83zcE0njRJZk=\r
+=rfhL\r
+-----END PGP SIGNATURE-----\r
+--=-=-=--\r