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

compilation issues with Mingw64 /gcc 10.2 #516

Open
sancelot opened this issue Dec 17, 2020 · 4 comments · May be fixed by #461
Open

compilation issues with Mingw64 /gcc 10.2 #516

sancelot opened this issue Dec 17, 2020 · 4 comments · May be fixed by #461

Comments

@sancelot
Copy link

It looks like there are some dllimport and functions may be declared as inline that conflicts

leading to many compiler (gcc10.2 ) messages as this one:

mingw-w64-libfcl/fcl-0.6.1/include/fcl/math/motion/taylor_model/interval-inl.h:417:13: warning: 'fcl::Interval fcl::bound(const fcl::Interval &, S) redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]

@sherm1
Copy link
Member

sherm1 commented Dec 17, 2020

We would welcome a PR to fix this!

@sancelot
Copy link
Author

I agree, I understand there are some dll imports definitions in headers, that should be defineedd in inl.h files or inlined.... I am not really sure how to solve it at the moment

@sancelot
Copy link
Author

Sorry, but your help is needed,you must keep in mind that this code is not very easy to understand

duplicate of #474

@jamiesnape
Copy link
Contributor

It should not really be a duplicate of #474. CMake should be handling the right syntax cross-platform. Assuming it is, if this compiler is complaining in this respect, it means a mistake was made by developers for all platforms, but it is less of an issue with those platforms. That being said the CMake is very messy at the moment, so potentially there is an issue, but I think this one may be down to the C++ code.

@SpaceIm SpaceIm linked a pull request Jan 29, 2022 that will close this issue
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 a pull request may close this issue.

3 participants