Skip to content

Conversation

@stephanlachnit
Copy link
Contributor

What is the purpose of the change

This adds a pkg-config file for the C++ version to make it easier to use it with non-CMake build systems. Requires #3478.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? not applicable

@github-actions github-actions bot added the C++ Pull Requests for C++ binding label Sep 3, 2025
@stephanlachnit stephanlachnit marked this pull request as draft September 3, 2025 18:42
@stephanlachnit
Copy link
Contributor Author

To be synced with the name to be decided in #3477.

Copy link
Member

@wgtmac wgtmac left a comment

Choose a reason for hiding this comment

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

As #3477 is merged, could you please rebase on it and switch the PR state?

@stephanlachnit
Copy link
Contributor Author

Rebased this PR and #3478

@stephanlachnit stephanlachnit changed the title [C++] Add pkg-config file AVRO-4180: [C++] Add pkg-config file Sep 5, 2025
@martin-g
Copy link
Member

@stephanlachnit There is a new merge conflict ... Please also mark is as ready for review (i.e. non-draft).

@stephanlachnit stephanlachnit marked this pull request as ready for review November 3, 2025 08:53
@stephanlachnit
Copy link
Contributor Author

@martin-g sorry for the delay, resolved and marked ready now!

@martin-g martin-g merged commit 4ea7592 into apache:main Nov 4, 2025
4 checks passed
@martin-g
Copy link
Member

martin-g commented Nov 4, 2025

Thank you, @stephanlachnit !

@stephanlachnit stephanlachnit deleted the p-cpp-pkg-config branch November 4, 2025 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C++ Pull Requests for C++ binding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants