#5 Support Windows-style Visual Studio builds

병합
necklace fanc999/msvc.build 에서 necklace/master 로 2 commits 를 머지했습니다 3 년 전
fanc999 코멘트됨, 3 년 전

Hi,

This attempts to improve the situation on building with Visual Studio for Windows, by:

  • Improving the search for libopenjpeg. The CMake build files for Visual Studio builds do not generate a pkg-config file for us but only generates a set of CMake config files, in which the package is not named as libopenjp2 but as OpenJPEG, so we should search for that as well on Visual Studio-style builds.

  • Mark fill_[vtable|info]() in io-jp2.c with G_MODULE_EXPORT so that these symbols are properly exported during the build on Visual Studio builds as well. Gdk-Pixbuf requires that these symbols are exported in order to load the loader.

The other thing that probably needs to be addressed but it not covered by this merge request is the default loader location in loaders.cache.in is hard-coded to a certain location, so it would be nice if that can be deduced at build time. By setting that path for myself manually, all the test programs pass successfully.

With blessings, thank you!

Hi, This attempts to improve the situation on building with Visual Studio for Windows, by: * Improving the search for libopenjpeg. The CMake build files for Visual Studio builds do not generate a pkg-config file for us but only generates a set of CMake config files, in which the package is not named as `libopenjp2` but as OpenJPEG, so we should search for that as well on Visual Studio-style builds. * Mark `fill_[vtable|info]()` in `io-jp2.c` with `G_MODULE_EXPORT` so that these symbols are properly exported during the build on Visual Studio builds as well. Gdk-Pixbuf requires that these symbols are exported in order to load the loader. The other thing that probably needs to be addressed but it not covered by this merge request is the default loader location in `loaders.cache.in` is hard-coded to a certain location, so it would be nice if that can be deduced at build time. By setting that path for myself manually, all the test programs pass successfully. With blessings, thank you!
Nichlas Severinsen 코멘트됨, 3 년 전
소유자

Greatly appreciated! I don't use Windows so I'm not able to test at the moment, but it doesn't break building on Linux so I'll happily accept your patch.

I'll see if I can figure out how to fix the hard coded loaders.cache.in location whenever I get some spare time.

Thanks!

Greatly appreciated! I don't use Windows so I'm not able to test at the moment, but it doesn't break building on Linux so I'll happily accept your patch. I'll see if I can figure out how to fix the hard coded loaders.cache.in location whenever I get some spare time. Thanks!
이 풀리퀘스트가 성공적으로 머지되었습니다!
로그인하여 이 대화에 참여
레이블 없음
마일스톤 없음
담당자 없음
참여자 2명
로딩중...
취소
저장
아직 콘텐츠가 없습니다.