Skip to content

[bbalouki-itch] Fix package integrations - #4

Closed
vicroms wants to merge 1 commit into
pr-52908-bbalouki-itch-basefrom
pr-52908-bbalouki-itch-fixes
Closed

[bbalouki-itch] Fix package integrations#4
vicroms wants to merge 1 commit into
pr-52908-bbalouki-itch-basefrom
pr-52908-bbalouki-itch-fixes

Conversation

@vicroms

@vicroms vicroms commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Applies the validated review patch on top of microsoft#52908.

Changes:

  • Fixes the upstream header install layout so consumers can include <itch/parser.hpp> from the triplet root include directory.
  • Adds Arrow/Parquet find_dependency() calls to the exported CMake config when the arrow feature is enabled.
  • Removes the currently failing python feature and forces ITCH_BUILD_PYTHON=OFF.
  • Updates the version database entry for bbalouki-itch.

Validated during review with patched core and arrow installs plus Debug/Release find_package(itch CONFIG REQUIRED) and direct include/link consumer checks.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vicroms
vicroms deleted the branch pr-52908-bbalouki-itch-base July 15, 2026 22:47
@vicroms vicroms closed this Jul 15, 2026
@vicroms
vicroms deleted the pr-52908-bbalouki-itch-fixes branch July 21, 2026 23:31
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