diff options
| author | Jonas Schnelli <dev@jonasschnelli.ch> | 2018-04-17 10:58:22 +0200 |
|---|---|---|
| committer | Jonas Schnelli <dev@jonasschnelli.ch> | 2018-04-17 10:58:42 +0200 |
| commit | 307699304886ba5ab67487780463b238edabd744 (patch) | |
| tree | 9133fc0657546c6a12927f6cd52fab9144812e35 /src | |
| parent | 07825088f9cfd8abece774b9d978c36ab90ce3d1 (diff) | |
| parent | 2eb5036c330fcc4fc115fe378b8211fd14711195 (diff) | |
| download | bitcoin-307699304886ba5ab67487780463b238edabd744.tar.xz | |
Merge #12899: macOS: Prevent Xcode 9.3 build warnings
2eb5036c3 macOS: Prevent Xcode 9.3 build warnings (Akio Nakamura)
Pull request description:
This PR intends to solve #12867
1. clang (Apple LLVM version 9.1.0 (clang-902.0.39.1)) warns unused argument '-pie' during compilation. This parameter should pass to the linker, so use '-Wl,-pie' style in configure.ac.
2. The other hand, when linking libbitcoinconsensus.la with passing '-Wl,-pie', ld warns that '-pie' being ignored because not link a main executable. So remove it from $libbitcoinconsensus_la_LDFLAGS in src/Makefile.am.
- In order to apply this change to Makefile, you need to run autogen.sh and ./configure.
- I think no need to add if-clause by target-os because also g++ can recognize/handle '-Wl,-pie'.
- I confirmed that the build was successful on macos10.13.4/XCode9.3 and ubuntu 17.10/g++7.2.0 and that warning was not issued.
Tree-SHA512: 45b7c3881f3ad92172eb2ff6fa90c4dc70a42037450ea4b6fd87613b10c0aa90ebcffd54e7c50b70ba876b7c2e356825950fbf5a7a4e8e25118688d98d7b6ee0
Diffstat (limited to 'src')
0 files changed, 0 insertions, 0 deletions