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

cork library doesn't build with clang-5 #706

Closed
yurivict opened this issue Apr 13, 2018 · 5 comments
Closed

cork library doesn't build with clang-5 #706

yurivict opened this issue Apr 13, 2018 · 5 comments

Comments

@yurivict
Copy link

CloudCompare/cork fails with errors like these:

In file included from src/cork.cpp:28:
In file included from src/mesh/mesh.h:48:
In file included from src/mesh/mesh.remesh.tpp:35:
src/mesh/mesh.topoCache.tpp:436:13: error: no matching conversion for functional-style cast from 'const char [14]' to 'std::exception'
            ENSURE(vaddr.count(v) > 0);
            ^~~~~~~~~~~~~~~~~~~~~~~~~~

In file included from src/cork.cpp:28:
In file included from src/mesh/mesh.h:48:
In file included from src/mesh/mesh.remesh.tpp:35:
src/mesh/mesh.topoCache.tpp:437:13: error: no matching conversion for functional-style cast from 'const char [14]' to 'std::exception'
            ENSURE(count(v->tris, t) == 1);
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/util/prelude.h:63:9: note: expanded from macro 'ENSURE'
                throw std::exception("ENSURE FAILED"); \
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@dgirardeau
Copy link
Member

Shouldn't this issue be posted on the Cork repo page directly? (https://github.com/gilbo/cork) Or at least in our dedicated fork?

Anyway this library is not maintained. Therefore I fear that apart if you feel motivated to fix this issue, it will stay as is. It doesn't seem to be very challenging, but as I am not a clang user I won't be able to help.

@yurivict
Copy link
Author

Your fork doesn't have the 'Issues' page.
Your plugin seems to depend on cork, and cork can't be built, therefore your plugin can't be built.

@dgirardeau
Copy link
Member

Most of CC's plugins are "3rd-party" by definition. Push too far in this direction and I'll simply remove the plugin from this repo (anyway it's the fate of this plugin as, again, Cork is not maintained).

@dgirardeau
Copy link
Member

Moreover your 'nice' demonstration eludes the fact that Cork's github repo has an Issue page (which was my first suggestion).

@asmaloney
Copy link
Member

I see it's now been reported in the proper place: gilbo/cork#39

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

No branches or pull requests

3 participants