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.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants