-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
add effcee recipe #6133
add effcee recipe #6133
Conversation
This comment has been minimized.
This comment has been minimized.
6951227
to
15a40d8
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Do someone have a clue, why mac-clang complains about the test package? the enum is declared like: namespace effcee { Test code: auto result = effcee::Match(input_stream.str(), checks_stream.str(), // Successful match result converts to true. /Users/jenkins/w/BuildSingleReference@2/conan-center-index/recipes/effcee/all/test_package/main.cc:54:12: error: expression is not an integral constant expression |
This comment has been minimized.
This comment has been minimized.
What is the best way to force compiler to use at least cpp11 (the higher, the better)? |
conan-io/conan#8002 is my go to |
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Why windows runtime for test package is not taken from conan?... |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
If I try to reproduce the problem from the CI, I am fetting a different error... does anyone have a clue? |
Ah, I see, I have compiler version 16, not 14. |
@AndreyMlashkin The install is incorrect https://github.com/google/effcee/blob/2ec8f8738118cc483b67c04a759fee53496c5659/effcee/CMakeLists.txt#L16 It's missing runtime for the DLL/shared |
Not sure, what should I change there |
This comment has been minimized.
This comment has been minimized.
Fix conan build Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com> Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Export all symbols on windows Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
8995438
to
e28e98f
Compare
I don't know, what happened to PR, I needed to rebase and force push. |
All green in build 22 (
|
Specify library name and version: effcee/v2019.1@
this lib is needed for creation a recipe for https://github.com/microsoft/DirectXShaderCompiler
closes #6128
conan-center hook activated.