Skip to content

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Oct 20, 2025

To not have to guess. Also to sync with autotools, which already uses
this wording.

Also:

  • replace the stray term 'folder' with 'directory' for consistency.
  • store help text in a temp variable to avoid overly long strings
    (mandatory in CMake <4.2.0 and can't be trivially split), also
    to avoid repeating this string 4 times.

Ref: https://cmake.org/cmake/help/v4.2/command/set.html


This will be a tedious rebase in #16973.

The ([A-Za-z0-9-]+) include directory.
Absolute path to \1 include directory.

@github-actions github-actions bot added the build label Oct 20, 2025
@vszakats vszakats marked this pull request as draft October 20, 2025 22:07
@testclutch

This comment was marked as resolved.

@vszakats vszakats marked this pull request as ready for review October 20, 2025 23:54
@vszakats vszakats changed the title cmake: say "absolute path" in setting descriptions and docs cmake: say 'absolute path' in setting descriptions and docs Oct 21, 2025
@vszakats vszakats changed the title cmake: say 'absolute path' in setting descriptions and docs cmake: say 'absolute path' in option descriptions and docs Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants