-
-
Notifications
You must be signed in to change notification settings - Fork 244
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
Develop windows compiler fixes #912
Conversation
Restore MS C++ Default exception handling behavior #885 Remove clang toolchain as it skips compiler checks
@@ -631,6 +631,12 @@ | |||
PTHREAD_SCOPE_SYSTEM) call. */ | |||
#cmakedefine H5_SYSTEM_SCOPE_THREADS @H5_SYSTEM_SCOPE_THREADS@ | |||
|
|||
/* Define using v1.6 public API symbols by default */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix order
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -wd5105") | ||
endif() | ||
if (CMAKE_CXX_COMPILER_ID STREQUAL "MSVC" AND CMAKE_CXX_COMPILER_LOADED) | ||
set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /EHsc") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actual change - most other changes just whitespace
config/toolchain/clang.cmake
Outdated
@@ -1,15 +1,15 @@ | |||
# Uncomment the following to use cross-compiling | |||
#set(CMAKE_SYSTEM_NAME Linux) | |||
|
|||
set(CMAKE_COMPILER_VENDOR "clang") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This caused the compiler checks to be skipped. Left code in case a change is needed after extensive testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will revert this because the reason this change was based on was incorrect.
src/H5private.h
Outdated
@@ -44,7 +44,9 @@ | |||
#include <sys/time.h> | |||
#endif | |||
#ifdef H5_HAVE_UNISTD_H | |||
#ifdef H5_HAVE_GETPWUID |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mingw fix
No description provided.