88/4c7de7c89dc4299a5dfb57a0fc8bf3e74b3bc2

   1 Return-Path: <jani@nikula.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 69760431FAF\r
   6         for <notmuch@notmuchmail.org>; Mon, 20 Oct 2014 08:03:34 -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: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id j3F1wU7dWvQC for <notmuch@notmuchmail.org>;\r
  16         Mon, 20 Oct 2014 08:03:29 -0700 (PDT)\r
  17 Received: from mail-wg0-f49.google.com (mail-wg0-f49.google.com\r
  18  [74.125.82.49])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
  19  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
  20  B890F431FAE    for <notmuch@notmuchmail.org>; Mon, 20 Oct 2014 08:03:28 -0700\r
  21  (PDT)\r
  22 Received: by mail-wg0-f49.google.com with SMTP id x12so5658857wgg.8\r
  23         for <notmuch@notmuchmail.org>; Mon, 20 Oct 2014 08:03:26 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:from:to:cc:subject:date:message-id;\r
  27         bh=xpj5MRhotBAUTILo8BGntCIp+Fn5pXuVjjeN6PxJ9bI=;\r
  28         b=V5nKEtI1tqY1DNWKf/y07ZZP5aw8w16an2W+X6QxWG7o5VLkDVDvboLy2w7OB/hrAQ\r
  29         HddYSf4w7wLBsjGtcIkXo8FTLux9gbl9sQ1NbqfGJNqJ8Uzmr3ESpaeKD6F5yMfThQtk\r
  30         g6BQRksmVr16qOkN0PJY0S52qqUWdKqZYpgJSY5rzYo0V9z/HGfKInO73OM08TLQyE1V\r
  31         6IIAeC1Q1Jv3eKcNBHuHN6yU/NuFq2vpMtepDYJzNaGkN0cZ9olpAK8cFlUGqUf7rT/E\r
  32         fKi2BUz8e6O0rwP0JMJze+ZKFWgUXdCpvaEIatIYzKfCKCnp7PNe5Dh1Mc7ZgFTHYH0c\r
  33         f/lQ==\r
  34 X-Gm-Message-State:\r
  35  ALoCoQnsdqC+ni6ude1CG59F/YtJUQ1DjO5o9016WEkEJmvE2T0BfwpFS8sG4RpDq+TFxE7C40NP\r
  36 X-Received: by 10.181.27.229 with SMTP id jj5mr21091026wid.64.1413817405174;\r
  37         Mon, 20 Oct 2014 08:03:25 -0700 (PDT)\r
  38 Received: from localhost (mobile-internet-5d6ad2-138.dhcp.inet.fi.\r
  39         [93.106.210.138]) by mx.google.com with ESMTPSA id\r
  40         ws2sm12166358wjc.32.2014.10.20.08.03.23 for <multiple recipients>\r
  41         (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\r
  42         Mon, 20 Oct 2014 08:03:24 -0700 (PDT)\r
  43 From: Jani Nikula <jani@nikula.org>\r
  44 To: notmuch@notmuchmail.org\r
  45 Subject: [PATCH] test: use LDFLAGS in test/Makefile.local\r
  46 Date: Mon, 20 Oct 2014 18:03:21 +0300\r
  47 Message-Id: <1413817401-6058-1-git-send-email-jani@nikula.org>\r
  48 X-Mailer: git-send-email 2.1.1\r
  49 X-BeenThere: notmuch@notmuchmail.org\r
  50 X-Mailman-Version: 2.1.13\r
  51 Precedence: list\r
  52 List-Id: "Use and development of the notmuch mail system."\r
  53         <notmuch.notmuchmail.org>\r
  54 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  56 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  57 List-Post: <mailto:notmuch@notmuchmail.org>\r
  58 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  59 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  61 X-List-Received-Date: Mon, 20 Oct 2014 15:03:34 -0000\r
  62 \r
  63 Apparently the test binaries are built with minimal LDFLAGS, only\r
  64 adding dependency specific LDFLAGS as needed. However because some of\r
  65 the test binaries incorporate notmuch object files, it is necessary to\r
  66 use the same link flags as notmuch. For example user provided\r
  67 CFLAGS/CXXFLAGS/LDFLAGS with -fsanitize=undefined fails to build the\r
  68 test binaries if the flags differ.\r
  69 \r
  70 ---\r
  71 \r
  72 With this I could do\r
  73 \r
  74 CFLAGS="-fsanitize=undefined -g" CXXFLAGS="-fsanitize=undefined -g" LDFLAGS="-fsanitize=undefined -g" ./configure\r
  75 \r
  76 to try ubsan with gcc 4.9. No problems found while running make test\r
  77 (and it did find an UB I planted to ensure I had it all set up right).\r
  78 ---\r
  79  test/Makefile.local | 14 +++++++-------\r
  80  1 file changed, 7 insertions(+), 7 deletions(-)\r
  81 \r
  82 diff --git a/test/Makefile.local b/test/Makefile.local\r
  83 index a2d58fc1a1b0..2331ceb15e33 100644\r
  84 --- a/test/Makefile.local\r
  85 +++ b/test/Makefile.local\r
  86 @@ -13,10 +13,10 @@ smtp_dummy_srcs =           \\r
  87  smtp_dummy_modules = $(smtp_dummy_srcs:.c=.o)\r
  88  \r
  89  $(dir)/arg-test: $(dir)/arg-test.o command-line-arguments.o util/libutil.a\r
  90 -       $(call quiet,CC) $^ -o $@\r
  91 +       $(call quiet,CC) $^ -o $@ $(LDFLAGS)\r
  92  \r
  93  $(dir)/hex-xcode: $(dir)/hex-xcode.o command-line-arguments.o util/libutil.a\r
  94 -       $(call quiet,CC) $^ $(TALLOC_LDFLAGS) -o $@\r
  95 +       $(call quiet,CC) $^ -o $@ $(LDFLAGS) $(TALLOC_LDFLAGS)\r
  96  \r
  97  random_corpus_deps =  $(dir)/random-corpus.o  $(dir)/database-test.o \\r
  98                         notmuch-config.o command-line-arguments.o \\r
  99 @@ -24,19 +24,19 @@ random_corpus_deps =  $(dir)/random-corpus.o  $(dir)/database-test.o \\r
 100                         parse-time-string/libparse-time-string.a\r
 101  \r
 102  $(dir)/random-corpus: $(random_corpus_deps)\r
 103 -       $(call quiet,CXX) $(CFLAGS_FINAL) $^ -o $@ $(CONFIGURE_LDFLAGS)\r
 104 +       $(call quiet,CXX) $^ -o $@ $(LDFLAGS) $(CONFIGURE_LDFLAGS)\r
 105  \r
 106  $(dir)/smtp-dummy: $(smtp_dummy_modules)\r
 107 -       $(call quiet,CC) $^ -o $@\r
 108 +       $(call quiet,CC) $^ -o $@ $(LDFLAGS)\r
 109  \r
 110  $(dir)/symbol-test: $(dir)/symbol-test.o lib/$(LINKER_NAME)\r
 111 -       $(call quiet,CXX) $^ -o $@ -Llib -lnotmuch $(XAPIAN_LDFLAGS)\r
 112 +       $(call quiet,CXX) $^ -o $@ $(LDFLAGS) -Llib -lnotmuch $(XAPIAN_LDFLAGS)\r
 113  \r
 114  $(dir)/parse-time: $(dir)/parse-time.o parse-time-string/parse-time-string.o\r
 115 -       $(call quiet,CC) $^ -o $@\r
 116 +       $(call quiet,CC) $^ -o $@ $(LDFLAGS)\r
 117  \r
 118  $(dir)/make-db-version: $(dir)/make-db-version.o\r
 119 -       $(call quiet,CXX) $^ -o $@ $(XAPIAN_LDFLAGS)\r
 120 +       $(call quiet,CXX) $^ -o $@ $(LDFLAGS) $(XAPIAN_LDFLAGS)\r
 121  \r
 122  .PHONY: test check\r
 123  \r
 124 -- \r
 125 2.1.1\r
 126 \r