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
Precompile windows gcc 8 #1039
Precompile windows gcc 8 #1039
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -108,7 +108,11 @@ endif | |
STAN_FLAGS=$(STAN_FLAG_THREADS)$(STAN_FLAG_MPI)$(STAN_FLAG_OPENCL) | ||
|
||
ifeq ($(OS),Windows_NT) | ||
ifeq ($(shell expr $(CXX_MAJOR) \>= 8), 1) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we also be checking they are using gcc and not clang or would it be fine in that case as well? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Clang on windows with cmdstan does not work at all (#736) so that didnt cross my mind. Will add a note there if we ever figure that out. Other option would be to keep the precompiled headers off in the off chance someone tries this with clang on Windows? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Changed the logic to also handle clang on Windows. As we have no idea if it works, I opted to disabling it by default. |
||
PRECOMPILED_HEADERS ?= true | ||
else | ||
PRECOMPILED_HEADERS ?= false | ||
endif | ||
else | ||
PRECOMPILED_HEADERS ?= true | ||
endif | ||
|
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 is just a test that shows that its still disabled in the Github Action test with Rtools 3.5.