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

Fix compilation with ikos-scan-cc (space-ros/docker#138). #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xfiderek
Copy link

@xfiderek xfiderek commented Feb 20, 2024

This commit fixes compilation with ikos-scan-cc compiler, by modifying linker flags. Refer to problem B described in space-ros/docker#138 for more details. I have verified that compilation works fine with gcc and clang after adding these flags.

Of course, I can create a separate issue for this in ament cobra repo if needed. I am also open to alternative solutions (e.g. modifying cobra makefile).

This commit fixes compilation with ikos-scan, by modifying linker flags
@xfiderek xfiderek changed the title Fix compilation with ikos-scan-cc (space-ros/docker#132). Fix compilation with ikos-scan-cc (space-ros/docker#138). Feb 20, 2024
@ivanperez-keera
Copy link
Collaborator

ivanperez-keera commented Aug 22, 2024

Can you modify the commit message to close issue #17? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants