Skip to content

MDEV-40936 Install test binaries to libexecdir instead of bindir - #5608

Open
dhandzus wants to merge 1 commit into
MariaDB:mainfrom
dhandzus:test-to-libexecdir
Open

MDEV-40936 Install test binaries to libexecdir instead of bindir#5608
dhandzus wants to merge 1 commit into
MariaDB:mainfrom
dhandzus:test-to-libexecdir

Conversation

@dhandzus

Copy link
Copy Markdown

Test-component binaries (mariadb-test, mariadb-client-test, mariadb-test-embedded, mariadb-client-test-embedded, test-connect-t) are not intended for direct invocation by users and are not meant to appear in $PATH, therefore install them to libexecdir instead of bindir.

Changes:

  • Add INSTALL_LIBEXECDIR to all cmake install layouts (STANDALONE, RPM, DEB, SVR4)
  • Set DESTINATION ${INSTALL_LIBEXECDIR} for mariadb-test, mariadb-client-test, mariadb-test-embedded, mariadb-client-test-embedded, and test-connect-t
  • Update mariadb-test-run.pl to find test binaries under libexec/ for installed-layout runs
  • Update debian/mariadb-test.install paths accordingly

The test suite was executed successfully in debug mode.

Test-component binaries (mariadb-test, mariadb-client-test,
mariadb-test-embedded, mariadb-client-test-embedded, test-connect-t)
are not intended for direct invocation by users and are not meant to
appear in $PATH, therefore install them to libexecdir instead of bindir.

Changes:
-  Add INSTALL_LIBEXECDIR to all cmake install layouts
   (STANDALONE, RPM, DEB, SVR4)
-  Set DESTINATION ${INSTALL_LIBEXECDIR} for mariadb-test,
   mariadb-client-test, mariadb-test-embedded,
   mariadb-client-test-embedded, and test-connect-t
-  Update mariadb-test-run.pl to find test binaries under
   libexec/ for installed-layout runs
-  Update debian/mariadb-test.install paths accordingly
@CLAassistant

CLAassistant commented Aug 27, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@gkodinov gkodinov added the External Contribution All PRs from entities outside of MariaDB Foundation, Corporation, Codership agreements. label Aug 28, 2026

@gkodinov gkodinov 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.

Thank you for your contribution! This is a preliminary review (https://github.com/MariaDB/server/blob/main/COMMUNITY_CONTRIBUTIONS.md#preliminary-review).

LGTM. Please stand by for the final review.

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

Labels

External Contribution All PRs from entities outside of MariaDB Foundation, Corporation, Codership agreements.

Development

Successfully merging this pull request may close these issues.

5 participants