Skip to content

[24095] Build: use GNUInstallDirs CMake module to create directory paths#300

Merged
richiware merged 2 commits intoeProsima:masterfrom
fedepell:build_use_GNUInstallDirs
Jan 19, 2026
Merged

[24095] Build: use GNUInstallDirs CMake module to create directory paths#300
richiware merged 2 commits intoeProsima:masterfrom
fedepell:build_use_GNUInstallDirs

Conversation

@fedepell
Copy link
Contributor

@fedepell fedepell commented Dec 30, 2025

Description

Instead of defining manually certain install paths, use the standard (since 3.0) CMake module GNUInstallDirs to derive them. This is cross-platform and offers an automatic integration on distributions customizations (ie. when certain distribution system-wide define a specific place where to install stuff).

See also: https://cmake.org/cmake/help/v3.0/module/GNUInstallDirs.html

The goal is to make the build more modern and software easy to deploy in any distribution.

Contributor Checklist

  • Commit messages follow the project guidelines.
  • The code follows the style guidelines of this project.
  • Changes are backport compatible: they do NOT break ABI nor change library core behavior.
  • Changes are API compatible.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • Check CI results: changes do not issue any warning.
  • Check CI results: CI pass and failing tests are unrelated with the changes.

Instead of defining manually certain install paths, use the standard (since 3.0)
CMake module GNUInstallDirs to derive them. This is cross-platform and offers
an automatic integration on distributions customizations (ie. when certain
distribution system-wide define a specific place where to install stuff).

See also: https://cmake.org/cmake/help/v3.0/module/GNUInstallDirs.html

The goal is to make the build more modern and software easy to deploy in
any distribution.

Signed-off-by: Federico Pellegrin <fede@evolware.org>
Co-authored-by: Ricardo González <correoricky@gmail.com>
Signed-off-by: Federico Pellegrin <fede@evolware.org>
@fedepell
Copy link
Contributor Author

@richiware thanks for the review! I merged in the changes, look good to me! I've tried before not to change too much the original CMake script except what was directly related to the proposed change, but now it is even cleaner ;-)

@MiguelCompany MiguelCompany requested review from richiprosima and removed request for richiprosima January 19, 2026 09:13
@MiguelCompany MiguelCompany modified the milestones: v2.4.0, v2.3.5 Jan 19, 2026
@richiware richiware merged commit 1ca99ea into eProsima:master Jan 19, 2026
12 checks passed
@richiware richiware changed the title Build: use GNUInstallDirs CMake module to create directory paths [24095] Build: use GNUInstallDirs CMake module to create directory paths Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants