Perlmint / glew-cmake

GLEW(https://github.com/nigels-com/glew, source updated nightly) with Cmake and pre-generated sources
Other
234 stars 95 forks source link

CMake - Fix shared runtimes install path #15

Closed GekkieHenkie closed 5 years ago

GekkieHenkie commented 5 years ago

Currently (well, at least on Windows), the created DLL files aren't installed to the correct directory after a build. The dll files are installed to the 'lib' directory. Which actually is the place for the static library or the shared library's symbols file (.lib). The shared library runtimes should be installed to the 'bin' output directory, as proposed in this PR.

See the CONFIGURATIONS option in Install command introduction in the CMake manual for an example.

It's considered best practice and follows convention as used in other notable libraries as libpng, libogg/vorbis, freetype, zlib, tinyxml, etc.

GekkieHenkie commented 5 years ago

Maybe I've been to hasty. I'll have to test some more.