Skip to content

Conversation

@geoyee
Copy link
Contributor

@geoyee geoyee commented Jan 13, 2026

When building using CMake according to CI commands, the only difference is that I used VS's sln instead of Ninja (cmake -G "Visual Studio 16 2019" ...). I received the following error (#738):

D:\code\duckdb-spatial\./vcpkg_ports\gdal: info: installing overlay port from here
-- Using cached OSGeo-gdal-v3.8.5.tar.gz
-- Extracting source D:/code/vcpkg/downloads/OSGeo-gdal-v3.8.5.tar.gz
-- Applying patch find-link-libraries.patch
-- Applying patch fix-gdal-target-interfaces.patch
-- Applying patch libkml.patch
-- Applying patch target-is-valid.patch
-- Applying patch duckdb_gdal_json.patch
CMake Error at scripts/cmake/z_vcpkg_apply_patches.cmake:34 (message):
  Applying patch failed: error: corrupt patch at line 19
ilures?WT.mc_id=vcpkg_inproduct_cli for more information.
Elapsed time to handle gdal:x64-windows-static-md-release-vs2019comp: 12 s
Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+gdal
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?title=[gdal]+Build+error+on+x64-windows-static-md-release-vs2019comp&body=Copy%20issue%20body%20from%20D%3A%2Fcode%2Fduckdb-spatial%2Fbuild%2Frelease%2Fvcpkg_installed%2Fvcpkg%2Fissue_body.md

Completed submission of json-c:[email protected] to 1 binary cache(s) in 202 ms
-- Running vcpkg install - failed
CMake Error at D:/code/vcpkg/scripts/buildsystems/vcpkg.cmake:953 (message):
  vcpkg install failed.  See logs for more information:
  D:\code\duckdb-spatial\build\release\vcpkg-manifest-install.log
Call Stack (most recent call first):
  D:/CMake/share/cmake-3.31/Modules/CMakeDetermineSystem.cmake:146 (include)
  CMakeLists.txt:23 (project)


-- Configuring incomplete, errors occurred!

I think there is an issue with the format of duckdb_gdal_json.patch. I created a clean file, and now I can generate and compile normally.

@Maxxen
Copy link
Member

Maxxen commented Jan 17, 2026

Thanks!

@Maxxen Maxxen merged commit 46b4ecc into duckdb:v1.4-andium Jan 17, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants