Skip to content

GH-55316: [CI][FlightRPC][ODBC] Apply ports.patch in vcpkg setup - #55317

Draft
amoeba wants to merge 1 commit into
apache:mainfrom
amoeba:GH-55316-odbc-vcpkg-ports-patch
Draft

amoeba wants to merge 1 commit into
apache:mainfrom
amoeba:GH-55316-odbc-vcpkg-ports-patch

Conversation

@amoeba

@amoeba amoeba commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

WIP

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

No.

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

…DBC Windows builds

The ODBC Windows action checked out microsoft/vcpkg at HEAD and never
applied ci/vcpkg/ports.patch. Since apacheGH-55165 bumped vcpkg, protobuf
v36.2 requires our fix-msvc-global-empty-string.patch to link with
C++20 code on MSVC.

Use ci/scripts/install_vcpkg.sh so the vcpkg version from .env is used
and the patches are applied. Because we use manifest mode, also register
the patched ports as overlay ports so vcpkg actually uses them.
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #55316 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions Bot added the awaiting review Awaiting review label Oct 29, 2026
@amoeba

amoeba commented Oct 29, 2026

Copy link
Copy Markdown
Member Author

I think the "AMD64 Debian SIMD Level None" failure is a real issue but unrelated to this PR. Maybe related to #54905.

@alinaliBQ
alinaliBQ self-requested a review October 29, 2026 17:23
@tadeja

tadeja commented Oct 29, 2026

Copy link
Copy Markdown
Member

I think the "AMD64 Debian SIMD Level None" failure is a real issue but unrelated to this PR. Maybe related to #54905.

Right! The scheduled C++ Extra on main started failing for AMD64 Debian SIMD Level NONE 2 days ago. All C++ Extra jobs were green 3 days ago.

(Here's a hypothetical draft PR #55319 as exploratory static linking fix post-#54905. fyi @raulcd)

But failing JNI ubuntu-24.04-arm arm64/v8 is new!

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

Labels

awaiting review Awaiting review CI: Extra: C++ Run extra C++ CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants