diff options
| author | Jonas Schnelli <[email protected]> | 2018-04-17 10:58:22 +0200 |
|---|---|---|
| committer | Jonas Schnelli <[email protected]> | 2018-04-17 10:58:42 +0200 |
| commit | 307699304886ba5ab67487780463b238edabd744 (patch) | |
| tree | 9133fc0657546c6a12927f6cd52fab9144812e35 /src/script/script.cpp | |
| parent | Merge #12972: Add python3 script shebang lint (diff) | |
| parent | macOS: Prevent Xcode 9.3 build warnings (diff) | |
| download | discoin-307699304886ba5ab67487780463b238edabd744.tar.xz discoin-307699304886ba5ab67487780463b238edabd744.zip | |
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/script/script.cpp')
0 files changed, 0 insertions, 0 deletions