-
Notifications
You must be signed in to change notification settings - Fork 69
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
Drop pkg-config for static libs #68
Comments
I'm on board with reducing dependence on pkg-config wherever possible (#12). But GLFW is certainly a case we should keep pkg-config around for. |
An example of a package that seems broken is curl and openssl:
You'll see that the |
Closing. The motivation for this ticket was solved by using |
In particular, this is solved by including a |
For all the statically linked libs we should stop using
pkg-config
becuase most libraries do not provide a valid.pc
file for static linking.See http://stackoverflow.com/questions/27586503/how-to-use-pkg-config-to-link-a-library-statically
Currently I am seeing link commnads with a mix of static and dynamic link flags for the same library. This is just asking for trouble, no to mention a missing symbol will result in a run-time error instead of a compile time error - I have hit this many times on Android.
@kkaefer Said
pkg-config
is needed for GLFW, so assuming they maintain a valid.pc
we can leave that lib alone.The text was updated successfully, but these errors were encountered: