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

Compiler warnings about the visibility attribute from mingw #352

Closed
nemequ opened this issue May 24, 2016 · 6 comments
Closed

Compiler warnings about the visibility attribute from mingw #352

nemequ opened this issue May 24, 2016 · 6 comments

Comments

@nemequ
Copy link
Contributor

nemequ commented May 24, 2016

When attempting to cross-compile with mingw, I see a lot of "error: visibility attribute not supported in this configuration; ignored [-Werror=attributes]" messages (normally they're just warnings, but I use -Werror for CI builds). See https://travis-ci.org/quixdb/squash/jobs/132697469#L1320 for an example. I can reproduce on Fedora with Squash with something like ming64-cmake . && make -j.

If you want, you can adapt https://github.com/quixdb/squash/blob/26aa037b618c1a37d984099b8dd5a3e04d3058c3/squash/squash.h#L115

@eustas
Copy link
Collaborator

eustas commented Jun 2, 2016

Thanks. We're going to adapt it in few steps: first resolve the compiler warnings and later, when we are ready for windows, grab __declspec part.

@eustas
Copy link
Collaborator

eustas commented Jun 9, 2016

Should be fixed.

@eustas eustas closed this as completed Jun 9, 2016
@nemequ
Copy link
Contributor Author

nemequ commented Jun 9, 2016

Looks like it's still an issue: https://travis-ci.org/quixdb/squash/jobs/136512277

That's with 09543b5 (you can verify that at https://travis-ci.org/quixdb/squash/jobs/136512277#L198).

On a somewhat related note (an easy way to switch compilers), if you are interested I'd be willing to put together a CMake-based build system for you; it could be useful for Squash, too, since we could share the build system…

@eustas eustas reopened this Jun 9, 2016
@eustas
Copy link
Collaborator

eustas commented Jun 9, 2016

Ooops, thought it was about unused function...
Visibility is fixed in v0.4, going to back-port fix soon.

@eustas
Copy link
Collaborator

eustas commented Jun 10, 2016

Should be fixed. Déjà vu?

@nemequ
Copy link
Contributor Author

nemequ commented Jun 12, 2016

Thanks, it works :)

https://travis-ci.org/quixdb/squash/builds/137077283

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

2 participants