b9/19a0e80b8d086b36b9c90cbb6506f1e6617e0d

   1 Return-Path: <felipe.contreras@gmail.com>\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 9CA9E431FC1\r
   6         for <notmuch@notmuchmail.org>; Sat,  5 Jun 2010 04:05:50 -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: -1.999\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.999 tagged_above=-999 required=5\r
  12         tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1,\r
  13         DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001]\r
  14         autolearn=ham\r
  15 Received: from olra.theworths.org ([127.0.0.1])\r
  16         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  17         with ESMTP id b+z8dwVI-Coc for <notmuch@notmuchmail.org>;\r
  18         Sat,  5 Jun 2010 04:05:37 -0700 (PDT)\r
  19 Received: from mail-bw0-f53.google.com (mail-bw0-f53.google.com\r
  20         [209.85.214.53])\r
  21         by olra.theworths.org (Postfix) with ESMTP id 488554196F3\r
  22         for <notmuch@notmuchmail.org>; Sat,  5 Jun 2010 04:05:28 -0700 (PDT)\r
  23 Received: by mail-bw0-f53.google.com with SMTP id 12so685083bwz.26\r
  24         for <notmuch@notmuchmail.org>; Sat, 05 Jun 2010 04:05:28 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=domainkey-signature:received:received:from:to:cc:subject:date\r
  27         :message-id:x-mailer:in-reply-to:references;\r
  28         bh=fYtbZ4EKxYlcOEeDKzYFQFPakPI9Co4SGNTs0nUFdYo=;\r
  29         b=mo6mhReiCYr9/3b2Um1AK5VdXrH9hW9grQ8XkflJ4wPATmmapnr09O6AqZ2qbw8h6r\r
  30         hqZL/W3rTJY1dhWx1ji1Q8HR18sPq+38/DeTAVkv5Vaw4zqbYk5Xw4BkOosXj5UHvomA\r
  31         /M5DtKu+dsU/cnE70TbA7px0NYne5CvKH5PxA=\r
  32 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;\r
  33         h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references;\r
  34         b=X0HXPvyfZI7KLLVUJfY/6jtrzEThsv3Vvb1NPvx2lBk7p21HEqlQgzUgoOZERm1IYN\r
  35         grJhscRBpGp9EuKU0JltA2efhA1630qNqu5i3m7QIlBSJE07Y07eecEkxp6+OhvtNV8t\r
  36         +gTyBli5XcF6M2xWq/fFUrSlOou5iCgybopq4=\r
  37 Received: by 10.204.81.153 with SMTP id x25mr4817539bkk.36.1275735927975;\r
  38         Sat, 05 Jun 2010 04:05:27 -0700 (PDT)\r
  39 Received: from localhost (a91-153-253-80.elisa-laajakaista.fi [91.153.253.80])\r
  40         by mx.google.com with ESMTPS id v2sm9557898bkz.19.2010.06.05.04.05.27\r
  41         (version=TLSv1/SSLv3 cipher=RC4-MD5);\r
  42         Sat, 05 Jun 2010 04:05:27 -0700 (PDT)\r
  43 From: Felipe Contreras <felipe.contreras@gmail.com>\r
  44 To: notmuch@notmuchmail.org\r
  45 Subject: [PATCH 2/3] build: fix DSO dependencies\r
  46 Date: Sat,  5 Jun 2010 14:05:14 +0300\r
  47 Message-Id: <1275735915-22650-3-git-send-email-felipe.contreras@gmail.com>\r
  48 X-Mailer: git-send-email 1.7.1\r
  49 In-Reply-To: <1275735915-22650-1-git-send-email-felipe.contreras@gmail.com>\r
  50 References: <1275735915-22650-1-git-send-email-felipe.contreras@gmail.com>\r
  51 X-BeenThere: notmuch@notmuchmail.org\r
  52 X-Mailman-Version: 2.1.13\r
  53 Precedence: list\r
  54 List-Id: "Use and development of the notmuch mail system."\r
  55         <notmuch.notmuchmail.org>\r
  56 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  58 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  59 List-Post: <mailto:notmuch@notmuchmail.org>\r
  60 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  61 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  63 X-List-Received-Date: Sat, 05 Jun 2010 11:05:50 -0000\r
  64 \r
  65 At least on Fedora 13, this doesn't link; the linker finds the\r
  66 dependencies, and aborts saying we should include them.\r
  67 \r
  68 /usr/bin/ld: gmime-filter-reply.o: undefined reference to symbol 'g_mime_filter_set_size'\r
  69 /usr/bin/ld: note: 'g_mime_filter_set_size' is defined in DSO /usr/lib/libgmime-2.6.so.0 so try adding it to the linker command line\r
  70 /usr/lib/libgmime-2.6.so.0: could not read symbols: Invalid operation\r
  71 \r
  72 We do need to link at least to what we really use, the linker resolves\r
  73 the dependencies of our dependencies at loading time. So let's only\r
  74 specify what we use directly.\r
  75 \r
  76 For more information, see:\r
  77 https://fedoraproject.org/wiki/UnderstandingDSOLinkChange\r
  78 \r
  79 Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\r
  80 ---\r
  81 \r
  82 NOTE: I'm not sure about using $(CC) as a linker in !linux platforms, but if\r
  83 that doesn't work, there's already an 'ifeq' that checks for that so it can be\r
  84 moved there.\r
  85 \r
  86  Makefile.local |    3 ---\r
  87  configure      |   11 +++++------\r
  88  2 files changed, 5 insertions(+), 9 deletions(-)\r
  89 \r
  90 diff --git a/Makefile.local b/Makefile.local\r
  91 index bc61a3c..cc8b23b 100644\r
  92 --- a/Makefile.local\r
  93 +++ b/Makefile.local\r
  94 @@ -33,10 +33,7 @@ FINAL_CFLAGS = -DNOTMUCH_VERSION=$(VERSION) $(CFLAGS) $(WARN_CFLAGS) $(CONFIGURE\r
  95  FINAL_CXXFLAGS = $(CXXFLAGS) $(WARN_CXXFLAGS) $(CONFIGURE_CXXFLAGS) $(extra_cflags) $(extra_cxxflags)\r
  96  FINAL_NOTMUCH_LDFLAGS = $(LDFLAGS) -Llib -lnotmuch\r
  97  FINAL_NOTMUCH_LINKER = CC\r
  98 -ifneq ($(LINKER_RESOLVES_LIBRARY_DEPENDENCIES),1)\r
  99  FINAL_NOTMUCH_LDFLAGS += $(CONFIGURE_LDFLAGS)\r
 100 -FINAL_NOTMUCH_LINKER = CXX\r
 101 -endif\r
 102  ifeq ($(PLATFORM),LINUX)\r
 103  ifeq ($(LIBDIR_IN_LDCONFIG),0)\r
 104  FINAL_NOTMUCH_LDFLAGS += -Wl,--enable-new-dtags -Wl,-rpath,$(libdir)\r
 105 diff --git a/configure b/configure\r
 106 index 1eb4785..ff775f0 100755\r
 107 --- a/configure\r
 108 +++ b/configure\r
 109 @@ -255,7 +255,11 @@ for gmimepc in gmime-2.6 gmime-2.4; do\r
 110         printf "Yes ($gmimepc).\n"\r
 111         have_gmime=1\r
 112         gmime_cflags=$(pkg-config --cflags $gmimepc)\r
 113 -       gmime_ldflags=$(pkg-config --libs $gmimepc)\r
 114 +       if [ $linker_resolves_library_dependencies = "1" ]; then\r
 115 +               gmime_ldflags="-lgmime-2.6 -lgobject-2.0 -lglib-2.0"\r
 116 +       else\r
 117 +               gmime_ldflags=$(pkg-config --libs $gmimepc)\r
 118 +       fi\r
 119      fi\r
 120  done\r
 121  if [ "$have_gmime" = "0" ]; then\r
 122 @@ -481,11 +485,6 @@ HAVE_STRCASESTR = ${have_strcasestr}\r
 123  # Supported platforms (so far) are: LINUX, MACOSX, SOLARIS\r
 124  PLATFORM = ${platform}\r
 125  \r
 126 -# Whether the linker will automatically resolve the dependency of one\r
 127 -# library on another (if not, then linking a binary requires linking\r
 128 -# directly against both)\r
 129 -LINKER_RESOLVES_LIBRARY_DEPENDENCIES = ${linker_resolves_library_dependencies}\r
 130 -\r
 131  # Flags needed to compile and link against Xapian\r
 132  XAPIAN_CXXFLAGS = ${xapian_cxxflags}\r
 133  XAPIAN_LDFLAGS = ${xapian_ldflags}\r
 134 -- \r
 135 1.7.0.1\r
 136 \r