-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
Unittests don't compile #45
Comments
Edit: I managed to get them to compile by disabling cpputest's memoryleak detection, dockerfile:
And in the unittests cmakelists.txt enforce the use of C++20 standard:
If you like, I can make a PR out of this with the fixes. If this doesn't make any sense, please let me know as well! |
Sorry for a long reply. It is better to use cmake build system. It has no any issues with building unit tests. Regards |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks for the extensive protocol you've made! I'm trying it out, and wanted to run the unit tests, but I'm running into compile issues. I'm building with a container, but the same issue also arises bare metal.
Reproducing the issue
If you're interested in reproducing the issue, this is my setup
Dockerfile:
build it:
And run it (from this repo root):
From the docker:
Compilation problems
The build fails with the following output:
This seems similar to this issue in cpputest, but their they proposed the a fix of moving the system includes before the cputest include headers. However, the first file to fails the compilation (
fd_multidrop_tests.cpp
) is already failing on the first include. This include is a system include rather than a cpputest include, so there is nothing that could be moved further up here. In other words, I don't see how the proposed fix of that issue would apply here.I have no experience with cpputest, would you have any idea what's going on here? Are the unit tests still compiling on your system?
The text was updated successfully, but these errors were encountered: