build: configure script warnings on macOS #896

issue hebasto opened this issue on February 17, 2021
  1. hebasto commented at 8:26 AM on February 17, 2021: member

    Configuring the recent master branch (24d1656c328c932cd533f6f4bf80b345b6591f0d) on macOS Big Sur 11.2.1 (20D74), and receiving 4 messages rm: conftest.dSYM: is a directory:

    % ./configure -q
    rm: conftest.dSYM: is a directory
    rm: conftest.dSYM: is a directory
    rm: conftest.dSYM: is a directory
    rm: conftest.dSYM: is a directory
    
    Build Options:
      with ecmult precomp     = yes
      with external callbacks = no
      with benchmarks         = yes
      with tests              = yes
      with openssl tests      = yes
      with coverage           = no
      module ecdh             = no
      module recovery         = no
      module extrakeys        = no
      module schnorrsig       = no
    
      asm                     = x86_64
      bignum                  = no
      ecmult window size      = 15
      ecmult gen prec. bits   = 4
    
      valgrind                = no
      CC                      = gcc
      CFLAGS                  = -O2 -fvisibility=hidden -std=c89 -pedantic -Wall -Wextra -Wcast-align -Wnested-externs -Wshadow -Wstrict-prototypes -Wundef -Wno-unused-function -Wno-long-long -Wno-overlength-strings -W -g
      CPPFLAGS                = -I/usr/local/Cellar/valgrind/3.16.1/include 
      LDFLAGS                 = 
    
      CC_FOR_BUILD            = gcc
      CFLAGS_FOR_BUILD        = -O2 -fvisibility=hidden -std=c89 -pedantic -Wall -Wextra -Wcast-align -Wnested-externs -Wshadow -Wstrict-prototypes -Wundef -Wno-unused-function -Wno-long-long -Wno-overlength-strings -W -g
      CPPFLAGS_FOR_BUILD      = -I/usr/local/Cellar/valgrind/3.16.1/include 
      LDFLAGS_FOR_BUILD       =
    

    Detailed:

    % ./configure
    ...
    checking whether gcc understands -c and -o together... rm: conftest.dSYM: is a directory
    yes
    ...
    checking the name lister (/usr/bin/nm -B) interface... rm: conftest.dSYM: is a directory
    BSD nm
    ...
    checking for archiver [@FILE](/bitcoin-core-secp256k1/contributor/file/) support... rm: conftest.dSYM: is a directory
    no
    ...
    checking command to parse /usr/bin/nm -B output from gcc object... rm: conftest.dSYM: is a directory
    ok
    ...
    
  2. real-or-random commented at 11:14 AM on February 17, 2021: contributor

    Did you run autogen.sh again just to be sure?

    So I guess you may have figured that out already but that's not a serious issue, it just means that some conftest.dSYM directory will be left in your tree. See https://lists.gnu.org/archive/html/bug-autoconf/2007-11/msg00017.html ...

    What's the output of glibtool --version and autoconf --version? edit: and autoreconf --version?

  3. hebasto commented at 12:48 PM on February 17, 2021: member

    Did you run autogen.sh again just to be sure?

    Yess :)

    So I guess you may have figured that out already but that's not a serious issue, it just means that some conftest.dSYM directory will be left in your tree. See https://lists.gnu.org/archive/html/bug-autoconf/2007-11/msg00017.html ...

    Yeah, I've read some mls, including that you point at, and I agree that's not a serious issue.

    What's the output of glibtool --version and autoconf --version? edit: and autoreconf --version?

    % glibtool --version
    glibtool (GNU libtool) 2.4.6
    Written by Gordon Matzigkeit, 1996
    
    Copyright (C) 2014 Free Software Foundation, Inc.
    This is free software; see the source for copying conditions.  There is NO
    warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
    
    % autoconf --version
    autoconf (GNU Autoconf) 2.69
    Copyright (C) 2012 Free Software Foundation, Inc.
    License GPLv3+/Autoconf: GNU GPL version 3 or later
    <http://gnu.org/licenses/gpl.html>, <http://gnu.org/licenses/exceptions.html>
    This is free software: you are free to change and redistribute it.
    There is NO WARRANTY, to the extent permitted by law.
    
    Written by David J. MacKenzie and Akim Demaille.
    
    % autoreconf --version
    autoreconf (GNU Autoconf) 2.69
    Copyright (C) 2012 Free Software Foundation, Inc.
    License GPLv3+/Autoconf: GNU GPL version 3 or later
    <http://gnu.org/licenses/gpl.html>, <http://gnu.org/licenses/exceptions.html>
    This is free software: you are free to change and redistribute it.
    There is NO WARRANTY, to the extent permitted by law.
    
    Written by David J. MacKenzie and Akim Demaille.
    
  4. real-or-random commented at 4:54 PM on February 17, 2021: contributor

    Hm, these are not the most recent versions but they should be recent enough then... Cirrus CI shows the same issue: https://cirrus-ci.com/task/5325071847784448?command=test#L51

    Can you try removing the directory and see if the error persists?

    Looking at the the (old) Travis logs, I believe this was introduced by #862. Can you confirm this?

    before: https://travis-ci.org/github/bitcoin-core/secp256k1/builds/754093307 after: https://travis-ci.org/github/bitcoin-core/secp256k1/builds/754127317 Make sure to look at "gcc" build (which is Apple's fake gcc).

  5. hebasto commented at 5:47 PM on February 17, 2021: member

    Hm, these are not the most recent versions but they should be recent enough then...

    Both packages are installed via Homebrew, btw:

    Can you try removing the directory and see if the error persists?

    find . -type d does not list such directories...

  6. hebasto commented at 5:55 PM on February 17, 2021: member

    Looking at the the (old) Travis logs, I believe this was introduced by #862. Can you confirm this?

    # just cloned repo
    % git checkout 328aaef22a4378817dbc5b7f27cd6351858c0b6c
    % ./autogen.sh
    % ./configure -q
    rm: conftest.dSYM: is a directory
    rm: conftest.dSYM: is a directory
    rm: conftest.dSYM: is a directory
    rm: conftest.dSYM: is a directory
    ...
    
  7. hebasto commented at 6:11 PM on February 17, 2021: member

    I think 83fb1bcef49b1c12ef349f62d90bfcc83f0f7398 (#700) is the cause of warnings.

  8. hebasto cross-referenced this on Feb 17, 2021 from issue Allow overriding default flags by jonasnick
  9. real-or-random referenced this in commit 595efa0104 on May 13, 2021
  10. real-or-random cross-referenced this on May 13, 2021 from issue Various improvements related to CFLAGS by real-or-random
  11. real-or-random referenced this in commit 301d835275 on May 25, 2021
  12. real-or-random referenced this in commit 80dfdba19c on Jun 10, 2021
  13. real-or-random referenced this in commit b924e1e605 on Jul 1, 2021
  14. jonasnick closed this on Jul 1, 2021

  15. hebasto cross-referenced this on Mar 17, 2023 from issue build: Ensure no optimization when building for coverage analysis by hebasto
  16. dderjoel referenced this in commit 702d91f7cc on May 23, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-20 20:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me