71/7bd6548ef1ba8e1125eb3a7e366413bdf87ef2

   1 Return-Path: <cworth@cworth.org>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id CDE784196F2\r
   6         for <notmuch@notmuchmail.org>; Wed, 14 Apr 2010 11:01:59 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id 9TazNFOkHey8; Wed, 14 Apr 2010 11:01:59 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 04D7A431FC1;\r
  19         Wed, 14 Apr 2010 11:01:59 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id B6DCE568DE1; Wed, 14 Apr 2010 11:01:58 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: Aaron Ecay <aaronecay@gmail.com>, notmuch@notmuchmail.org\r
  24 Subject: Re: [PATCH 3/4] Add infrastructure for building shared library on OS\r
  25         X.\r
  26 In-Reply-To: <1271029494-89014-3-git-send-email-aaronecay@gmail.com>\r
  27 References: <4bc25ea0.86c3f10a.45a3.ffff80d3@mx.google.com>\r
  28         <1271029494-89014-3-git-send-email-aaronecay@gmail.com>\r
  29 Date: Wed, 14 Apr 2010 11:01:58 -0700\r
  30 Message-ID: <87sk6xgbft.fsf@yoom.home.cworth.org>\r
  31 MIME-Version: 1.0\r
  32 Content-Type: multipart/signed; boundary="=-=-=";\r
  33         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.13\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38         <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Wed, 14 Apr 2010 18:02:00 -0000\r
  47 \r
  48 --=-=-=\r
  49 \r
  50 On Sun, 11 Apr 2010 19:44:53 -0400, Aaron Ecay <aaronecay@gmail.com> wrote:\r
  51 > This patch adds a configure check for OS X (actually Darwin),\r
  52 > and sets up the Makefiles to build a proper shared library on\r
  53 > that platform.\r
  54 ...\r
  55 > -include $(subdirs:%=%/Makefile.local) Makefile.local\r
  56 > +include Makefile.config $(subdirs:%=%/Makefile.local) Makefile.local\r
  57 \r
  58 This first hunk looks unrelated to what's described in the commit\r
  59 message.\r
  60 \r
  61 It also results in Makefile.config being included from both the toplevel\r
  62 Makefile and the toplevel Makefile.local, so that seems wrong.\r
  63 \r
  64 I had wanted to keep the top-level Makefile totally generic, (so that if\r
  65 a project wanted to imitate the notmuch flat-Makefile build system that\r
  66 would be easy). And perhaps including Makefile.config here violates\r
  67 that.\r
  68 \r
  69 But I'd be willing to accept that if necessary---should just remove the\r
  70 include of Makefile.config from Makefile.local then I think.\r
  71 \r
  72 > +printf "Checking for Mac OS X (for shared library)... "\r
  73 > +if [ `uname` = "Darwin" ] ; then\r
  74 > +    printf "Yes.\n"\r
  75 > +    mac_os_x=1\r
  76 > +else\r
  77 > +    printf "No.\n"\r
  78 > +    mac_os_x=0\r
  79 > +fi\r
  80 > +\r
  81 \r
  82 Instead of inventing a new mac_os_x variable, we should follow the GNU\r
  83 configure conventions of build_cpu, build_vendor, build_os\r
  84 variables. We're already allowing the user to assign to these by passing\r
  85 a --build option to configure (though not yet doing anything with the\r
  86 values).\r
  87 \r
  88 But now that you've got something you actually do want to do with the\r
  89 values, we should use those same variables. It might not even be crazy\r
  90 to copy in config.guess (or pieces of it). Though, frankly, it's not\r
  91 doing anything for Darwin unlike you're doing above, so you might as\r
  92 well just use your own code as you have.\r
  93 \r
  94 >  libnotmuch_c_srcs =          \\r
  95 > +     $(notmuch_compat_srcs)  \\r
  96 >       $(dir)/libsha1.c        \\r
  97 >       $(dir)/message-file.c   \\r
  98 >       $(dir)/messages.c       \\r
  99 \r
 100 This again looks like an independent fix that should be in a separate\r
 101 commit.\r
 102 \r
 103 -Carl\r
 104 \r
 105 --=-=-=\r
 106 Content-Type: application/pgp-signature\r
 107 \r
 108 -----BEGIN PGP SIGNATURE-----\r
 109 Version: GnuPG v1.4.10 (GNU/Linux)\r
 110 \r
 111 iD8DBQFLxgMW6JDdNq8qSWgRAmONAKCmj4z2S7S8fAUxDGUUg4FT0B3k+ACgkLEY\r
 112 4do3So28H0s+ueVR7DuRkQE=\r
 113 =nVeg\r
 114 -----END PGP SIGNATURE-----\r
 115 --=-=-=--\r