Skip to content

Configure pkgconf with multiarch pkg-config and library search paths#115

Merged
kimago merged 1 commit into
mainfrom
fix-pkgconf-noble
Jun 8, 2026
Merged

Configure pkgconf with multiarch pkg-config and library search paths#115
kimago merged 1 commit into
mainfrom
fix-pkgconf-noble

Conversation

@abg

@abg abg commented Jun 8, 2026

Copy link
Copy Markdown
Member

The original pkg-config build included --with-pc-path to search /usr/lib/x86_64-linux-gnu/pkgconfig. When switching to pkgconf, the equivalent --with-pkg-config-dir flag was not included, so pkgconf omitted the multiarch path from its compiled-in defaults.

On ubuntu-noble, cmake 3.28's FindOpenSSL uses pkg-config to resolve the library directory hint. Without openssl.pc visible to pkgconf, cmake could not locate OPENSSL_CRYPTO_LIBRARY despite libssl-dev being installed, causing the percona-xtrabackup builds to fail at the libkmip cmake configure step.

This was a regression introduced by #107 and only visible under ubuntu-noble compilation which was not caught before the merge.

The original pkg-config build included --with-pc-path to search
/usr/lib/x86_64-linux-gnu/pkgconfig. When switching to pkgconf, the
equivalent --with-pkg-config-dir flag was not included, so pkgconf
omitted the multiarch path from its compiled-in defaults.

On ubuntu-noble, cmake 3.28's FindOpenSSL uses pkg-config to resolve
the library directory hint. Without openssl.pc visible to pkgconf,
cmake could not locate OPENSSL_CRYPTO_LIBRARY despite libssl-dev being
installed, causing the percona-xtrabackup builds to fail at the libkmip
cmake configure step.

@kimago kimago left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reasonable changes to package builds exclusively.

@github-project-automation github-project-automation Bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Jun 8, 2026
@kimago kimago merged commit c888336 into main Jun 8, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants