Skip to content

Conversation

@WillAyd
Copy link
Contributor

@WillAyd WillAyd commented Oct 23, 2025

Rationale for this change

This retains parity with the CMake configuration

What changes are included in this PR?

C Data Interface library is created as a standalone

Are these changes tested?

No (not possible with current CI)

Are there any user-facing changes?

No

@WillAyd WillAyd added the CI: Extra Run extra CI label Oct 23, 2025
@github-actions
Copy link

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

@WillAyd
Copy link
Contributor Author

WillAyd commented Oct 23, 2025

This is a follow on to #47888 to fix the Meson configuration. I don't believe its possible to actually test this with our current CI setup

@github-actions github-actions bot added the awaiting review Awaiting review label Oct 23, 2025
@WillAyd
Copy link
Contributor Author

WillAyd commented Oct 27, 2025

@kou @pitrou any objections to merging this one? Should get the Meson build back to green

Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

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

No objection from me :)

@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Oct 27, 2025
arrow_c_data_integration_lib = library(
'arrow_c_data_integration',
sources: ['c_data_integration_internal.cc'],
dependencies: [arrow_test_dep],
Copy link
Member

Choose a reason for hiding this comment

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

Can we use arrow_test_dep_no_main?

See also: #47888 (comment)

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Oct 28, 2025
@WillAyd WillAyd force-pushed the meson-extension-type-change branch from 2746ccf to afde612 Compare October 28, 2025 13:22
@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Oct 28, 2025
@WillAyd
Copy link
Contributor Author

WillAyd commented Oct 28, 2025

@kou it looks like both #47968 and this are required to get CI green. Do you want me to combine both together into one PR, or should we merge them in separately knowing the first will fail Meson CI on main?

@kou
Copy link
Member

kou commented Oct 29, 2025

Let's merge in separately.

@WillAyd WillAyd force-pushed the meson-extension-type-change branch from afde612 to 24443b4 Compare October 29, 2025 18:58
@WillAyd
Copy link
Contributor Author

WillAyd commented Oct 29, 2025

Alrighty - looks like there was another PR that needed addressing as well. I've added as a second commit here to get us green - if you need me to split off into another PR let me know

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

@kou kou merged commit 430ad81 into apache:main Oct 30, 2025
66 of 68 checks passed
@kou kou removed the awaiting change review Awaiting change review label Oct 30, 2025
@github-actions github-actions bot added the awaiting merge Awaiting merge label Oct 30, 2025
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