-
Notifications
You must be signed in to change notification settings - Fork 9
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
*.tpp files throw syntax error #62
Comments
Hi @Chasikanaft, thanks for opening the issue. Could you please post here the commands you run with |
Looking at the output of the tests, I see the error
This may mean that the compilation was not completed successfully. Could you please paste here the |
That ALL BUILD output when compile Release x64 on VS2017 CE
|
The tests are failing because the compilation of the test is failing, so the original error are the compilation errors C2061, C2143, etc etc. In the CI we never test VS2017, only VS2019, so it is possible that that files does not support VS2017 at all. If you could use VS2019, I guess everything should work smooth. Anyhow, if the only objective is using other functionalities of the library that work fine under VS2017, probably you could just set BUILD_TESTING to OFF? |
Build was successful but I can't use the library..
|
Ah ok, so you basically have the same error of the test but in your code? Then I guess the only option is to update to VS2019. |
I think it could also be related to the version of |
Its no like test error, its different error like that:
|
all build succesfully without tests, but syntax error throw just into |
I would separate two problems at the moment:
Problem 1Looking at the errors, it fails on this line
In this case array is supposed to be the name of the field whose type is array_3f . The fact that it throws an error there, it could be related to the version of visit_struct and the availability of the macro VISITABLE_DIRECT_INIT . Please, let us know if the tests also compile fine when unticking USE_SYSTEM_visit_struct
Problem 2In order to use the library, you don't need to include the folders manually. You would need to use First, you need to install Then, you need to add the following lines to your find_package(matioCpp REQUIRED)
# definition of yourTarget ...
target_link_libraries(yourTarget PRIVATE matioCpp::matioCpp) |
Regarding problem 1, I have tried on my system (running Ubuntu), with the master version of
|
This may be a good lead indeed. vcpkg has a version of visit_struct that does not contain the features necessary by matio-cpp (see cbeck88/visit_struct#25 and conda-forge/visit_struct-feedstock#1). |
Hi @Chasikanaft, is the problem solved? |
Hello, thanks for reply :) I will try to return to this task soon with an updated VS from 2017 to 2022. Really not sure if that would work, that be weird. |
That would not be enough unfortuantly, if I remember correctly matio-cpp uses some features that are not releases in 1.0.0 . |
Hello there
What I got when I tried to load this library into vcpkg:
I tried compiling the dll and lib via just cmake on VS2017 and got the same results...
What am I doing wrong?)
The text was updated successfully, but these errors were encountered: