[PR #1367] cmake: fix man page install directories #1977

New Issue

claunia · 2026-01-29T20:56:45Z

claunia commented

2026-01-29 20:56:45 +00:00

Original Pull Request: https://github.com/google/brotli/pull/1367

State: closed
Merged: No

To not use a prefix-less absolute path with
cmake --install <d> --prefix <path>.

According to CMake docs about _FULL_ variables:
"These variables shouldn't be used in install() commands as they do
not work with the cmake --install command's --prefix option, or with
the cpack installer generators."

Ref: https://cmake.org/cmake/help/v4.2/module/GNUInstallDirs.html

Follow-up to cff5803216 #1084

**Original Pull Request:** https://github.com/google/brotli/pull/1367 **State:** closed **Merged:** No --- To not use a prefix-less absolute path with `cmake --install <d> --prefix <path>`. According to CMake docs about `_FULL_` variables: "These variables shouldn't be used in install() commands as they do not work with the cmake --install command's --prefix option, or with the cpack installer generators." Ref: https://cmake.org/cmake/help/v4.2/module/GNUInstallDirs.html Follow-up to cff58032160406c170a3dc319ad6e7b288cafcc0 #1084

claunia added the pull-request label 2026-01-29 20:56:45 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#1977