Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: configure script warnings on macOS #896

Closed
hebasto opened this issue Feb 17, 2021 · 6 comments
Closed

build: configure script warnings on macOS #896

hebasto opened this issue Feb 17, 2021 · 6 comments

Comments

@hebasto
Copy link
Member

hebasto commented Feb 17, 2021

Configuring the recent master branch (24d1656) 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 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
...
@real-or-random
Copy link
Contributor

real-or-random commented Feb 17, 2021

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
Copy link
Member Author

hebasto commented Feb 17, 2021

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.

@real-or-random
Copy link
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
Copy link
Member Author

hebasto commented Feb 17, 2021

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
Copy link
Member Author

hebasto commented Feb 17, 2021

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

@hebasto
Copy link
Member Author

hebasto commented Feb 17, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants