build: configure script warnings on macOS

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:

 0% ./configure -q
 1rm: conftest.dSYM: is a directory
 2rm: conftest.dSYM: is a directory
 3rm: conftest.dSYM: is a directory
 4rm: conftest.dSYM: is a directory
 5
 6Build Options:
 7  with ecmult precomp     = yes
 8  with external callbacks = no
 9  with benchmarks         = yes
10  with tests              = yes
11  with openssl tests      = yes
12  with coverage           = no
13  module ecdh             = no
14  module recovery         = no
15  module extrakeys        = no
16  module schnorrsig       = no
17
18  asm                     = x86_64
19  bignum                  = no
20  ecmult window size      = 15
21  ecmult gen prec. bits   = 4
22
23  valgrind                = no
24  CC                      = gcc
25  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
26  CPPFLAGS                = -I/usr/local/Cellar/valgrind/3.16.1/include 
27  LDFLAGS                 = 
28
29  CC_FOR_BUILD            = gcc
30  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
31  CPPFLAGS_FOR_BUILD      = -I/usr/local/Cellar/valgrind/3.16.1/include 
32  LDFLAGS_FOR_BUILD       =

Detailed:

 0% ./configure
 1...
 2checking whether gcc understands -c and -o together... rm: conftest.dSYM: is a directory
 3yes
 4...
 5checking the name lister (/usr/bin/nm -B) interface... rm: conftest.dSYM: is a directory
 6BSD nm
 7...
 8checking for archiver [@FILE](/bitcoin-core-secp256k1/contributor/file/) support... rm: conftest.dSYM: is a directory
 9no
10...
11checking command to parse /usr/bin/nm -B output from gcc object... rm: conftest.dSYM: is a directory
12ok
13...

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?

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?

0% glibtool --version
1glibtool (GNU libtool) 2.4.6
2Written by Gordon Matzigkeit, 1996
3
4Copyright (C) 2014 Free Software Foundation, Inc.
5This is free software; see the source for copying conditions.  There is NO
6warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

0% autoconf --version
1autoconf (GNU Autoconf) 2.69
2Copyright (C) 2012 Free Software Foundation, Inc.
3License GPLv3+/Autoconf: GNU GPL version 3 or later
4<http://gnu.org/licenses/gpl.html>, <http://gnu.org/licenses/exceptions.html>
5This is free software: you are free to change and redistribute it.
6There is NO WARRANTY, to the extent permitted by law.
7
8Written by David J. MacKenzie and Akim Demaille.

0% autoreconf --version
1autoreconf (GNU Autoconf) 2.69
2Copyright (C) 2012 Free Software Foundation, Inc.
3License GPLv3+/Autoconf: GNU GPL version 3 or later
4<http://gnu.org/licenses/gpl.html>, <http://gnu.org/licenses/exceptions.html>
5This is free software: you are free to change and redistribute it.
6There is NO WARRANTY, to the extent permitted by law.
7
8Written by David J. MacKenzie and Akim Demaille.

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).

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…

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?

0# just cloned repo
1% git checkout 328aaef22a4378817dbc5b7f27cd6351858c0b6c
2% ./autogen.sh
3% ./configure -q
4rm: conftest.dSYM: is a directory
5rm: conftest.dSYM: is a directory
6rm: conftest.dSYM: is a directory
7rm: conftest.dSYM: is a directory
8...

hebasto commented at 6:11 pm on February 17, 2021: member

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

hebasto cross-referenced this on Feb 17, 2021 from issue Allow overriding default flags by jonasnick

real-or-random referenced this in commit 595efa0104 on May 13, 2021

real-or-random cross-referenced this on May 13, 2021 from issue Various improvements related to CFLAGS by real-or-random

real-or-random referenced this in commit 301d835275 on May 25, 2021

real-or-random referenced this in commit 80dfdba19c on Jun 10, 2021

real-or-random referenced this in commit b924e1e605 on Jul 1, 2021

jonasnick closed this on Jul 1, 2021

hebasto cross-referenced this on Mar 17, 2023 from issue build: Ensure no optimization when building for coverage analysis by hebasto

dderjoel referenced this in commit 702d91f7cc on May 23, 2023

build: configure script warnings on macOS #896