Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • CMake CMake
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 4,161
    • Issues 4,161
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 15
    • Merge requests 15
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • External wiki
    • External wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • CMakeCMake
  • CMakeCMake
  • Merge requests
  • !7090

define_property(): Clean up docs and implementation related to INITIALIZE_FROM_VARIABLE

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Craig Scott requested to merge craig.scott/cmake:cleanup-define_property into master Mar 19, 2022
  • Overview 4
  • Commits 3
  • Pipelines 1
  • Changes 2

INITIALIZE_FROM_VARIABLE is new in CMake 3.23, but the versionadded note was missing in the original commit. The docs also failed to mention that the new option only applies to target properties. The docs for define_property() also no longer reflected what the command was mostly useful for, so they have been updated in line with the comment in #20698 (comment 1109450). A minor code tweak has also been made to avoid an unnecessary argument check when INITIALIZE_FROM_VAFRIABLE is not given.

Backport: release

Edited Mar 19, 2022 by Craig Scott
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: cleanup-define_property