Skip to content

gcs: Bump to 0.0.4#1341

Open
russellhancox wants to merge 4 commits intoduckdb:mainfrom
russellhancox:main
Open

gcs: Bump to 0.0.4#1341
russellhancox wants to merge 4 commits intoduckdb:mainfrom
russellhancox:main

Conversation

@russellhancox
Copy link
Contributor

Also: re-enable Windows builds

@taniabogatsch
Copy link
Contributor

Hi - I restarted the Windows run now since it seems spurious. But if it fails again it might need more thorough investigation. 👍

@taniabogatsch taniabogatsch added the unrelated CI failure CI failure appears unrelated and might be solved by re-running CI after fixing elsewhere label Feb 19, 2026
@russellhancox
Copy link
Contributor Author

This still appears to be failing but I'm a little at a loss as to why, the same build worked fine in our extension repo: https://github.com/northpolesec/duckdb-gcs/actions/runs/22139297199/job/64015425496. A cursory search suggests the error is related to input paths being too long.

@brokenjacobs
Copy link

I don't know if this helps but I looked through the build logs and there is a slight version mismatch, this failing PR says:

Visual Studio 2022 Developer Command Prompt v17.14.26

but the working PR from the extension repo says:

Visual Studio 2022 Developer Command Prompt v17.14.25

on the build step.

@carlopi
Copy link
Collaborator

carlopi commented Mar 7, 2026

There was some instability on CI, that I think it's fixed in main.

I took the liberty of rebasing the PR onto duckdb/community-extensions main branch, let's see what CI says.

@fb64
Copy link

fb64 commented Mar 10, 2026

Thanks @carlopi, could you try using the latest commit 9d2cb749affdebef8f4c0c78e21683bc9f5e2c0f with the updated submodule (1.4.4). I hope this resolves ix the build issue 🙏

@fb64
Copy link

fb64 commented Mar 11, 2026

The build has used the latest version (1.5.0)...According to the build script, to use the v1.4 duckdb_version property should be set to v1.4.4 instead of v1.4.2 in the MainDistributionPipeline
(If I understood correctly 🤔)

@fb64
Copy link

fb64 commented Mar 13, 2026

🤞

@fb64
Copy link

fb64 commented Mar 14, 2026

The upgrade to DuckDB v1.5.0 has been done and the build was successfull for the on both my fork branch and on the main branch of the extension repository 🤔
Any help would be appreciate to fix this build issue @carlopi @taniabogatsch 🙏

@carlopi
Copy link
Collaborator

carlopi commented Mar 14, 2026

Restarted Windows CI, let's see.

@fb64
Copy link

fb64 commented Mar 14, 2026

Thanks, @carlopi but still fail ... It's possible to get the following log file from the build : D:\a\community-extensions\community-extensions\vcpkg\buildtrees\google-cloud-cpp\install-x64-windows-static-release-rel-out.log or re-run windows build in debug mode ?
It could be a cache issue or something else ? Are the build tools the same as extension-ci-tools ?

@fb64
Copy link

fb64 commented Mar 16, 2026

After reviewing logs, It seems that in the community-extensions build an external ninja version 1.12.1 instead of 1.13.2 is used to install vcpkg package : -- Found external ninja('1.12.1'). and this log is not present in the main branch build of the extension repository 🤔
The rest seems to match : vcpkg packages version, MSVC compiler versions (19.44.35223.0) etc ...
I'm no expert in C++ compilation, and even less so on the Windows platform, but this is the only clue I've found.

@carlopi
Copy link
Collaborator

carlopi commented Mar 16, 2026

I am trying to debug the issue, opened a PR at #1536 to double check the results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

unrelated CI failure CI failure appears unrelated and might be solved by re-running CI after fixing elsewhere

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants