-
Notifications
You must be signed in to change notification settings - Fork 7.6k
[libgeotiff] Update to 1.7.0, revise features and config #21253
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
Merged
Merged
Changes from 12 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
acda456
Update to 1.7.0
dg0yt 07e7dac
Refresh patches, drop fix-proj4.patch
dg0yt 0c5617e
Restore GeoTIFF cmake package name (reverts #15750)
dg0yt ae5d02e
Add usage (reason: #15723)
dg0yt c21ac2a
Remove zlib dependency, transitive usage only
dg0yt ea21a1d
Move tools to optional feature
dg0yt 197c69b
Modernize portfile, fix copyright
dg0yt dc9240a
Don't install doc and man files
dg0yt 5f172fa
Trim main patch
dg0yt 4fb7969
Allow reduced tiff dependencies
dg0yt 7c10b7b
Update versions
dg0yt dff258e
Remove libgeotiff uwp failure from CI baseline
dg0yt 78200b3
Update GeoTIFF in gdal wrapper
dg0yt fdb8faf
Update versions
dg0yt b35add0
Update GeoTIFF in pdal config
dg0yt 527dd76
Update versions
dg0yt 8fc8414
Switch to 'version' field
dg0yt f44534b
Update versions
dg0yt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,19 +1,16 @@ | ||
| diff --git a/libgeotiff/CMakeLists.txt b/libgeotiff/CMakeLists.txt | ||
| index 80fe121..5c1430d 100644 | ||
| index b4a3cee..fd9f0a5 100644 | ||
| --- a/libgeotiff/CMakeLists.txt | ||
| +++ b/libgeotiff/CMakeLists.txt | ||
| @@ -80,8 +80,12 @@ SET(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/lib) | ||
| @@ -80,7 +80,11 @@ SET(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/lib) | ||
|
|
||
| IF(WIN32) | ||
| IF(MSVC) | ||
| - ADD_DEFINITIONS(-DBUILD_AS_DLL=1) | ||
| - ADD_DEFINITIONS(/DW4) | ||
| + IF(BUILD_SHARED_LIBS) | ||
| + ADD_DEFINITIONS(-DBUILD_AS_DLL=1) | ||
| ADD_DEFINITIONS(-DBUILD_AS_DLL=1) | ||
| + ELSE() | ||
| + ADD_DEFINITIONS(-DBUILD_AS_DLL=0) | ||
| + ENDIF() | ||
| + ADD_DEFINITIONS(/DW4) | ||
| + ADD_DEFINITIONS(-DBUILD_AS_DLL=0) | ||
| + ENDIF() | ||
| ADD_DEFINITIONS(/DW4) | ||
| if (NOT (MSVC_VERSION VERSION_LESS 1400)) | ||
| ADD_DEFINITIONS(-D_CRT_SECURE_NO_DEPRECATE) | ||
| ADD_DEFINITIONS(-D_CRT_SECURE_NO_WARNINGS) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,17 +1,14 @@ | ||
| diff --git a/cmake/project-config.cmake.in b/cmake/project-config.cmake.in | ||
| index 0dd376aca..0615081bd 100644 | ||
| --- a/libgeotiff/cmake/project-config.cmake.in | ||
| diff --git a/libgeotiff/cmake/project-config.cmake.in b/libgeotiff/cmake/project-config.cmake.in | ||
| index dc5b999..2ce07fd 100644 | ||
| --- a/libgeotiff/cmake/project-config.cmake.in | ||
| +++ b/libgeotiff/cmake/project-config.cmake.in | ||
| @@ -23,6 +23,12 @@ message (STATUS "Reading ${CMAKE_CURRENT_LIST_FILE}") | ||
| message (STATUS | ||
| "@PROJECT_NAME@ configuration, version ${@PROJECT_NAME@_VERSION}") | ||
|
|
||
| +#Find dependencies | ||
| +if(@WITH_PROJ4@) | ||
| + include(CMakeFindDependencyMacro) | ||
| + find_dependency(PROJ4) | ||
| +endif() | ||
| + | ||
| # Tell the user project where to find our headers and libraries | ||
| get_filename_component (_DIR ${CMAKE_CURRENT_LIST_FILE} PATH) | ||
| get_filename_component (_ROOT "${_DIR}/@PROJECT_ROOT_DIR@" ABSOLUTE) | ||
| @@ -23,6 +23,9 @@ message (STATUS "Reading ${CMAKE_CURRENT_LIST_FILE}") | ||
| message (STATUS | ||
| "@PROJECT_NAME@ configuration, version ${@PROJECT_NAME@_VERSION}") | ||
|
|
||
| +include(CMakeFindDependencyMacro) | ||
| +find_dependency(PROJ) | ||
| + | ||
| # Tell the user project where to find our headers and libraries | ||
| get_filename_component (_DIR ${CMAKE_CURRENT_LIST_FILE} PATH) | ||
| get_filename_component (_ROOT "${_DIR}/@PROJECT_ROOT_DIR@" ABSOLUTE) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,49 +1,48 @@ | ||
| vcpkg_from_github( | ||
| OUT_SOURCE_PATH SOURCE_PATH | ||
| REPO OSGeo/libgeotiff | ||
| REF 8b1a8f52bc909f86e04ceadd699db102208074a2 #v1.6.0 | ||
| SHA512 41715d6a416307a93b2f95874c00ed27c3a0450d70311e77ed45f7ff477bd85f4a69b549bde01dfb9412a62a482467222fc8ed398478e2829e4d112012aab852 | ||
| REF 7da5bacae7814c65ebb78f0b64e1141fbcb3de1e #v1.7.0 | ||
| SHA512 36047778fbbb4a533a7b65e7b32ab8c0955f59b95417b68b68e7ddd398191445e730e00271756213bf657cbf7cd5eb028b25d4b0741e5b309c78c207b4ec01c6 | ||
| HEAD_REF master | ||
| PATCHES | ||
| cmakelists.patch | ||
| geotiff-config.patch | ||
| fix-proj4.patch | ||
| fix-staticbuild.patch | ||
| fix-config-version.patch | ||
| skip-doc-install.patch | ||
| ) | ||
|
|
||
| set(SOURCE_PATH ${SOURCE_PATH}/libgeotiff) | ||
|
|
||
| # Delete FindPROJ4.cmake | ||
| file(REMOVE ${SOURCE_PATH}/cmake/FindPROJ4.cmake) | ||
| vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS | ||
| FEATURES | ||
| tools WITH_JPEG | ||
| tools WITH_UTILITIES | ||
| ) | ||
|
|
||
| vcpkg_configure_cmake( | ||
| SOURCE_PATH ${SOURCE_PATH} | ||
| PREFER_NINJA | ||
| vcpkg_cmake_configure( | ||
| SOURCE_PATH "${SOURCE_PATH}/libgeotiff" | ||
| OPTIONS | ||
| -DGEOTIFF_BIN_SUBDIR=bin | ||
| -DGEOTIFF_DATA_SUBDIR=share | ||
| -DWITH_TIFF=1 | ||
| -DWITH_PROJ4=1 | ||
| -DWITH_ZLIB=1 | ||
| -DWITH_JPEG=1 | ||
| -DWITH_UTILITIES=1 | ||
| -DHAVE_TIFFOPEN=1 | ||
| -DHAVE_TIFFMERGEFIELDINFO=1 | ||
| -DCMAKE_MACOSX_BUNDLE=0 | ||
| ${FEATURE_OPTIONS} | ||
| ) | ||
|
|
||
| vcpkg_install_cmake() | ||
| vcpkg_cmake_install() | ||
|
|
||
| vcpkg_copy_tools(TOOL_NAMES applygeo geotifcp listgeo makegeo AUTO_CLEAN) | ||
| if(WITH_UTILITIES) | ||
| vcpkg_copy_tools(TOOL_NAMES applygeo geotifcp listgeo makegeo AUTO_CLEAN) | ||
| endif() | ||
|
|
||
| vcpkg_copy_pdbs() | ||
| vcpkg_fixup_cmake_targets(CONFIG_PATH share/geotiff TARGET_PATH share/geotiff) | ||
| vcpkg_cmake_config_fixup(PACKAGE_NAME GeoTIFF) | ||
|
|
||
| if(VCPKG_LIBRARY_LINKAGE STREQUAL "static") | ||
| vcpkg_replace_string(${CURRENT_PACKAGES_DIR}/share/geotiff/geotiff-config.cmake "if (GeoTIFF_USE_STATIC_LIBS)" "if (1)") | ||
| file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/debug/bin ${CURRENT_PACKAGES_DIR}/bin) | ||
| vcpkg_replace_string("${CURRENT_PACKAGES_DIR}/share/GeoTIFF/geotiff-config.cmake" "if (GeoTIFF_USE_STATIC_LIBS)" "if (1)") | ||
| file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/bin" "${CURRENT_PACKAGES_DIR}/bin") | ||
| endif() | ||
|
|
||
| file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/debug/include ${CURRENT_PACKAGES_DIR}/debug/doc ${CURRENT_PACKAGES_DIR}/debug/share) | ||
|
|
||
| file(INSTALL ${SOURCE_PATH}/COPYING DESTINATION ${CURRENT_PACKAGES_DIR}/share/${PORT} RENAME copyright) | ||
| file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/include" "${CURRENT_PACKAGES_DIR}/debug/share") | ||
|
|
||
| file(INSTALL "${SOURCE_PATH}/libgeotiff/LICENSE" DESTINATION "${CURRENT_PACKAGES_DIR}/share/${PORT}" RENAME copyright) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| diff --git a/libgeotiff/CMakeLists.txt b/libgeotiff/CMakeLists.txt | ||
| index b4a3cee..f838a36 100644 | ||
| --- a/libgeotiff/CMakeLists.txt | ||
| +++ b/libgeotiff/CMakeLists.txt | ||
| @@ -286,6 +286,7 @@ SET(GEOTIFF_LIB_DIR ${GEOTIFF_LIB_SUBDIR}) | ||
| SET(GEOTIFF_INCLUDE_DIR ${GEOTIFF_INCLUDE_SUBDIR}) | ||
| SET(GEOTIFF_DATA_DIR ${GEOTIFF_DATA_SUBDIR}) | ||
|
|
||
| +if(0) | ||
| # Install doc files | ||
| INSTALL(FILES | ||
| AUTHORS ChangeLog COPYING INSTALL LICENSE README README_BIN README.WIN | ||
| @@ -294,6 +295,7 @@ INSTALL(FILES | ||
|
|
||
| # Install man pages | ||
| INSTALL(FILES ${GEOTIFF_MAN_PAGES} DESTINATION share/man/man1) | ||
| +endif() | ||
|
|
||
|
|
||
| # Install header files for development distribution |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| The package libgeotiff provides a CMake config file: | ||
|
|
||
| find_package(GeoTIFF CONFIG REQUIRED) | ||
| target_include_directories(main PRIVATE ${GEOTIFF_INCLUDE_DIR}) | ||
| target_link_libraries(main PRIVATE ${GEOTIFF_LIBRARIES}) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,29 @@ | ||
| { | ||
| "name": "libgeotiff", | ||
| "version-string": "1.6.0", | ||
| "port-version": 4, | ||
| "description": "Libgeotiff is an open source library normally hosted on top of libtiff for reading, and writing GeoTIFF information tags.", | ||
| "version-string": "1.7.0", | ||
| "description": "Libgeotiff is an open source library on top of libtiff for reading and writing GeoTIFF information tags.", | ||
| "homepage": "https://github.com/OSGeo/libgeotiff", | ||
| "dependencies": [ | ||
| "libjpeg-turbo", | ||
| "proj4", | ||
| "tiff", | ||
| "zlib" | ||
| ] | ||
| { | ||
| "name": "tiff", | ||
| "default-features": false | ||
| }, | ||
| { | ||
| "name": "vcpkg-cmake", | ||
| "host": true | ||
| }, | ||
| { | ||
| "name": "vcpkg-cmake-config", | ||
| "host": true | ||
| } | ||
| ], | ||
| "features": { | ||
| "tools": { | ||
| "description": "Install libgeotiff utility programs", | ||
| "dependencies": [ | ||
| "libjpeg-turbo" | ||
| ] | ||
| } | ||
| } | ||
| } | ||
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.