Skip to content
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

Building OpenDDS and ACE/TAO at the same time with CMake Fails on Windows with Release Config #4531

Closed
iguessthislldo opened this issue Mar 21, 2024 Discussed in #4529 · 0 comments · Fixed by #4535
Closed

Comments

@iguessthislldo
Copy link
Member

Discussed in #4529

Originally posted by nsarzyns March 20, 2024
I don't know if I'm just being dumb here but following the build instructions for cmake, successfully creates the sln, but if I change the build type to 'Release' it fails because for some reason the linker is still trying to use debug libraries in release mode. For example, the linker for the 'inspect' sub-project tried to link libs like ACEsd.lib, TAOsd.lib...basically all the libs have 'd' appended. I've even tried forcing the CMAKE_CONFIGURATION_TYPES to Release in the root cmakelists without success.

On the other hand, building using the configure file, works perfectly fine. The same sub project links the correct libs for the selected configuration of release.

Both instances are using ACE/TAO 7.1.3 and building on windows.

iguessthislldo added a commit to iguessthislldo/OpenDDS that referenced this issue Mar 25, 2024
Should fix OpenDDS#4531

I think this used to work somewhat, but I think was more by accident.
@iguessthislldo iguessthislldo changed the title Building OpenDDS with CMake Fails on Windows with Release Config Building OpenDDS and ACE/TAO at the same time with CMake Fails on Windows with Release Config Mar 25, 2024
iguessthislldo added a commit to iguessthislldo/OpenDDS that referenced this issue Mar 25, 2024
Should fix OpenDDS#4531

I think this used to work somewhat, but I think was more by accident.
Also separated CMake builds from `build_and_test.yml`.
iguessthislldo added a commit to iguessthislldo/OpenDDS that referenced this issue Mar 25, 2024
Should fix OpenDDS#4531

I think this used to work somewhat, but I think was more by accident.
Also separated CMake builds from `build_and_test.yml`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
1 participant