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

cmake: Report more compiler details. #189

Merged
merged 1 commit into from
May 6, 2024
Merged

Conversation

hebasto
Copy link
Owner

@hebasto hebasto commented May 4, 2024

Report more compiler details. Useful when compiler is being detected with its general symlink like /usr/bin/cc or /usr/bin/c++.

Pulled from #93.

Report more compiler details. Useful when compiler is being detected
with its general symlink like `/usr/bin/cc` or `/usr/bin/c++`.
@hebasto
Copy link
Owner Author

hebasto commented May 4, 2024

Friendly ping @m3dwards @fanquake @TheCharlatan :)

@m3dwards
Copy link

m3dwards commented May 6, 2024

ACK 5a68532

C compiler ............................ AppleClang 15.0.0.15000040, /Library/Developer/CommandLineTools/usr/bin/cc
CFLAGS ................................
C++ compiler .......................... AppleClang 15.0.0.15000040, /Library/Developer/CommandLineTools/usr/bin/c++
C compiler ............................ GNU 12.2.0, /usr/bin/cc
CFLAGS ................................
C++ compiler .......................... GNU 12.2.0, /usr/bin/c++
C compiler ............................ MSVC 19.39.33519.0, C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.39.33519/bin/Hostx64/x64/cl.exe
CFLAGS ................................ /DWIN32 /D_WINDOWS
C++ compiler .......................... MSVC 19.39.33519.0, C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.39.33519/bin/Hostx64/x64/cl.exe

@hebasto hebasto merged commit 8e3ff8d into cmake-staging May 6, 2024
28 checks passed
real-or-random added a commit to bitcoin-core/secp256k1 that referenced this pull request Jun 10, 2024
f55703b autotools: Delete unneeded compiler test (Hennadii Stepanov)
396e885 autotools: Align MSan checking code with CMake's implementation (Hennadii Stepanov)
abde59f cmake: Report more compiler details in summary (Hennadii Stepanov)
7abf979 cmake: Disable `ctime_tests` if build with `-fsanitize=memory` (Hennadii Stepanov)

Pull request description:

  Same as #1517, but for the CMakle build system.

  The second commit improves the configure summary (similar to hebasto/bitcoin#189.

ACKs for top commit:
  real-or-random:
    ACK f55703b

Tree-SHA512: 18190c062ae6e27d0ecbe7460cc22c960b25c0d35aa4b94f151d4b1c48f16e99fd5ecdfcb359784f95995292633d30d3d23b75a12be3aca5afffcc1e7e7daf31
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

Successfully merging this pull request may close these issues.

2 participants