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

Linker error: multiple definition of `typeinfo name for g2o::BaseVertex<7, g2o::Sim3>'; #721

Closed
dkogan opened this issue Oct 3, 2023 · 3 comments · Fixed by #727
Closed

Comments

@dkogan
Copy link

dkogan commented Oct 3, 2023

Hello. This is the inverse of #444. I just pushed g2o to Debian. On armhf there's a build error:

https://buildd.debian.org/status/fetch.php?pkg=g2o&arch=armhf&ver=0%7E20230806-1&stamp=1696268512&raw=0

Look at the end. Removing the fix for #444 makes the build complete, at least on armhf. Do you have a clear understanding of #444, and what this failure means? Should the template be explicitly instantiated, or not?

Thanks

@wangzheqie
Copy link

wangzheqie commented Oct 3, 2023 via email

@RainerKuemmerle
Copy link
Owner

@dkogan Do you know if removing this line would work on other architectures?

Back then I thought it passes in general but avoids an issue on MSVC. But MSVC is somewhat not so clear to me and we already have a few #ifdef MSC_VER in the code.

@dkogan
Copy link
Author

dkogan commented Oct 20, 2023

I currently have a distro-patch to remove that line, and here's what the build results are: https://buildd.debian.org/status/package.php?p=g2o

The i386 and arch=all failures are unrelated. So I would say that as far as Debian is concerned, removing that line is the right thing to do.

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