Skip to content

GH-49486: [CI][C++] Fix Meson build missing tensor extension sources#49487

Merged
kou merged 1 commit intoapache:mainfrom
WillAyd:worktree-fix-new-issue
Mar 10, 2026
Merged

GH-49486: [CI][C++] Fix Meson build missing tensor extension sources#49487
kou merged 1 commit intoapache:mainfrom
WillAyd:worktree-fix-new-issue

Conversation

@WillAyd
Copy link
Contributor

@WillAyd WillAyd commented Mar 10, 2026

Summary

  • Add extension/tensor_internal.cc and extension/variable_shape_tensor.cc to the arrow_json sources in meson.build
  • Add variable_shape_tensor.h to installed headers in extension/meson.build
  • These files were present in CMakeLists.txt but missing from the Meson build, causing undefined reference linker errors for variable_shape_tensor(), IsPermutationTrivial(), IsPermutationValid(), ComputeStrides(), and SliceTensorBuffer()

Test plan

  • Verify the Meson CI build passes without linker errors

🤖 Generated with Claude Code

…urces

Add tensor_internal.cc and variable_shape_tensor.cc to the arrow_json
sources in meson.build, and variable_shape_tensor.h to the installed
headers. These files were present in CMakeLists.txt but missing from
the Meson build, causing undefined reference linker errors.

* GitHub Issue: apache#49486

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

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

@WillAyd WillAyd added the CI: Extra: C++ Run extra C++ CI label Mar 10, 2026
Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Let's try this on main.

@kou kou merged commit ca68452 into apache:main Mar 10, 2026
75 of 80 checks passed
@kou kou removed the awaiting review Awaiting review label Mar 10, 2026
@github-actions github-actions bot added the awaiting merge Awaiting merge label Mar 10, 2026
@rok
Copy link
Member

rok commented Mar 10, 2026

Oh sorry about this one. I wasn't aware I had to update meson.build files. Will do next time.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants