-
Notifications
You must be signed in to change notification settings - Fork 97
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 on Windows (VS2015 Win64) - Linker Errors #11
Comments
Could you try this cmakelists? https://github.com/Yixin-Hu/TetWild/blob/cbd8713cb5ee37c8fe475a87f5c61c4829b89dd1/CMakeLists.txt |
Just did, still the same result
Also the complete build output: |
Could you try using older version of geogram, like version ~1.5.5? |
Tried with 1.5.5 and 1.5.2, unfortuneately, exactly the same result for both version. Maybe I am using the wrong, cmake generator/VS version, which did you use to test it under Windows? |
@herzig you may have a look here https://github.com/trelau/TetWild/commits/reorg For Windows, nothing was being exported to the shared library so it didn't generate TetWild.lib. I made some changes as outlined in the commit history that includes:
Still some things that need cleaned up but I think I'm on the right track... |
@trelau Could you get it working with your version? I have the same issue as before. geogram.lib is created fine, but then linking tetwild_lib fails. I tried to use the dumpbin utility to list all symbols defined in geogram.lib, and the supposedly missing symbol Full output of |
@herzig in my branch, when i turn |
@trelau Sorry I mixed up my working copies, embarassing... I can build your branch fine (with BUILD_WITH_DEBUG OFF). i generates an exe, and I just successfully meshed a geometry that all other simple meshing tools failed to mesh. Awesome project! |
@herzig Great! I just took a step back and pushed a branch that builds the basic exe https://github.com/trelau/TetWild/commit/78aa7fc42750e6dce1278890c97c5428af46b47c I had to comment out some code to resolve the external, so looking for the root cause now. This should now represent the initial state of @Yixin-Hu 's code. I'm thinking about refactoring the API a little in the tetwild.h file (use a single class to define parameters and load/retrieve mesh) to make it easier to optionally build as a shared lib. If you're looking to contribute the next step will be to optimize for speed! |
I could resolve the linker issue by moving the definition for the static function MeshCellsStore::cell_type_to_cell_descriptor in geogram mesh.h to the mesh.cpp file. mesh.h now looks like this:
and this added to mesh.cpp
I can build tetwild.exe now without any problems and using background meshes (the part that was causing linker problems) now works as well. |
Just FYI, the linking issue with Geogram should be fixed with the latest version (1.6.6). |
I have problems building TetWild on Windows. CGAL 4.12 is installed and working (examples work).
cmake can generate build files (VS Projects & sln)
geogram.lib seems to be compiling fine (using VS2015)
then building lib_tetwild fails with the following linker error:
geogram.lib seems to be fine, and is found by the linker as only this single symbol is causing a problem.
Thanks for you help!
The text was updated successfully, but these errors were encountered: