Skip to content

Commit

Permalink
[openexr] fixup pkgconfig (#18123)
Browse files Browse the repository at this point in the history
* [openexr] fixup pkgconfig

* [openexr] x-add-version

* [openexr] drop _d library suffix for debug builds (ensures correct .pc file)

* [openexr] x-add-version

* [openexr] rename patches in order, add pkg-config debug suffix patch

* [openexr] x-add-version

* [openexr] update patch to match upstream PR

* [openexr] x-add-version
  • Loading branch information
mcmtroffaes authored Jun 11, 2021
1 parent 02fa0eb commit 1d81331
Show file tree
Hide file tree
Showing 8 changed files with 66 additions and 5 deletions.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
From 6cd6b32af052563e4cd5114a1d279736d1d8653a Mon Sep 17 00:00:00 2001
From: "Matthias C. M. Troffaes" <matthias.troffaes@gmail.com>
Date: Tue, 1 Jun 2021 08:06:45 +0100
Subject: [PATCH] Use CMAKE_<CONFIG>_POSTFIX for .pc file lib suffix.

Signed-off-by: Matthias C. M. Troffaes <matthias.troffaes@gmail.com>
---
IlmBase/config/CMakeLists.txt | 3 ++-
OpenEXR/config/CMakeLists.txt | 3 ++-
PyIlmBase/config/CMakeLists.txt | 3 ++-
3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/IlmBase/config/CMakeLists.txt b/IlmBase/config/CMakeLists.txt
index d9c5ae449..dfebb43bd 100644
--- a/IlmBase/config/CMakeLists.txt
+++ b/IlmBase/config/CMakeLists.txt
@@ -78,7 +78,8 @@ if(ILMBASE_INSTALL_PKG_CONFIG)
set(exec_prefix "\${prefix}")
set(libdir "\${exec_prefix}/${CMAKE_INSTALL_LIBDIR}")
set(includedir "\${prefix}/${CMAKE_INSTALL_INCLUDEDIR}")
- set(LIB_SUFFIX_DASH ${ILMBASE_LIB_SUFFIX})
+ string(TOUPPER "${CMAKE_BUILD_TYPE}" uppercase_CMAKE_BUILD_TYPE)
+ set(LIB_SUFFIX_DASH ${ILMBASE_LIB_SUFFIX}${CMAKE_${uppercase_CMAKE_BUILD_TYPE}_POSTFIX})
if(TARGET Threads::Threads)
# hrm, can't use properties as they end up as generator expressions
# which don't seem to evaluate
diff --git a/OpenEXR/config/CMakeLists.txt b/OpenEXR/config/CMakeLists.txt
index 8386e3a0e..039190cd9 100644
--- a/OpenEXR/config/CMakeLists.txt
+++ b/OpenEXR/config/CMakeLists.txt
@@ -77,7 +77,8 @@ if(OPENEXR_INSTALL_PKG_CONFIG)
set(exec_prefix "\${prefix}")
set(libdir "\${exec_prefix}/${CMAKE_INSTALL_LIBDIR}")
set(includedir "\${prefix}/${CMAKE_INSTALL_INCLUDEDIR}")
- set(LIB_SUFFIX_DASH ${OPENEXR_LIB_SUFFIX})
+ string(TOUPPER "${CMAKE_BUILD_TYPE}" uppercase_CMAKE_BUILD_TYPE)
+ set(LIB_SUFFIX_DASH ${OPENEXR_LIB_SUFFIX}${CMAKE_${uppercase_CMAKE_BUILD_TYPE}_POSTFIX})
if(TARGET Threads::Threads)
# hrm, can't use properties as they end up as generator expressions
# which don't seem to evaluate
diff --git a/PyIlmBase/config/CMakeLists.txt b/PyIlmBase/config/CMakeLists.txt
index 1872c89e7..a4139bcfd 100644
--- a/PyIlmBase/config/CMakeLists.txt
+++ b/PyIlmBase/config/CMakeLists.txt
@@ -18,7 +18,8 @@ if(PYILMBASE_INSTALL_PKG_CONFIG)
set(exec_prefix ${CMAKE_INSTALL_BINDIR})
set(libdir ${CMAKE_INSTALL_LIBDIR})
set(includedir ${CMAKE_INSTALL_INCLUDEDIR})
- set(LIB_SUFFIX_DASH ${OPENEXR_LIB_SUFFIX})
+ string(TOUPPER "${CMAKE_BUILD_TYPE}" uppercase_CMAKE_BUILD_TYPE)
+ set(LIB_SUFFIX_DASH ${OPENEXR_LIB_SUFFIX}${CMAKE_${uppercase_CMAKE_BUILD_TYPE}_POSTFIX})
string(REPLACE ".in" "" pcout ${pcinfile})
configure_file(${pcinfile} ${CMAKE_CURRENT_BINARY_DIR}/${pcout} @ONLY)
install(
8 changes: 5 additions & 3 deletions ports/openexr/portfile.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,10 @@ vcpkg_from_github(
SHA512 549d37ed1ef4d1ff7e732d583f7213ee15c7f92625aea9fd65345e4c5b854902c02e5940d0692b1af5ae0a02abf46aaefea2662db2389d1b2fb4264a373baac2
HEAD_REF master
PATCHES
remove_find_package_macro.patch
fixup_cmake_exports_path.patch
remove_symlinks.patch
0001-remove_find_package_macro.patch
0002-fixup_cmake_exports_path.patch
0003-remove_symlinks.patch
0004-Fix-pkg-config-lib-suffix-for-cmake-debug-builds.patch # https://github.com/AcademySoftwareFoundation/openexr/pull/1032
)

vcpkg_configure_cmake(SOURCE_PATH ${SOURCE_PATH}
Expand All @@ -25,6 +26,7 @@ vcpkg_install_cmake()

vcpkg_fixup_cmake_targets(CONFIG_PATH share/ilmbase TARGET_PATH share/ilmbase)
vcpkg_fixup_cmake_targets()
vcpkg_fixup_pkgconfig()

file(REMOVE ${CURRENT_PACKAGES_DIR}/debug/bin/exrenvmap${VCPKG_HOST_EXECUTABLE_SUFFIX})
file(REMOVE ${CURRENT_PACKAGES_DIR}/debug/bin/exrheader${VCPKG_HOST_EXECUTABLE_SUFFIX})
Expand Down
2 changes: 1 addition & 1 deletion ports/openexr/vcpkg.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "openexr",
"version-string": "2.5.0",
"port-version": 2,
"port-version": 3,
"description": "OpenEXR is a high dynamic-range (HDR) image file format developed by Industrial Light & Magic for use in computer imaging applications",
"homepage": "https://www.openexr.com/",
"supports": "!uwp",
Expand Down
2 changes: 1 addition & 1 deletion versions/baseline.json
Original file line number Diff line number Diff line change
Expand Up @@ -4538,7 +4538,7 @@
},
"openexr": {
"baseline": "2.5.0",
"port-version": 2
"port-version": 3
},
"opengl": {
"baseline": "0.0",
Expand Down
5 changes: 5 additions & 0 deletions versions/o-/openexr.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
{
"versions": [
{
"git-tree": "7589d3f1641e6e63450303e424d966221375109f",
"version-string": "2.5.0",
"port-version": 3
},
{
"git-tree": "42bec4d0daed751ad449a504b7fa43d57eec9000",
"version-string": "2.5.0",
Expand Down

0 comments on commit 1d81331

Please sign in to comment.