Skip to content

write_language: Disables hook when CMP0220 is set to NEW - #1072

Open
arhag23 wants to merge 5 commits into
rapidsai:mainfrom
arhag23:write-language-policy
Open

write_language: Disables hook when CMP0220 is set to NEW#1072
arhag23 wants to merge 5 commits into
rapidsai:mainfrom
arhag23:write-language-policy

Conversation

@arhag23

@arhag23 arhag23 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Description

Now that enable_language support in subdirectories has been merged to CMake upstream, rapids_export_write_language does not need to use its hook logic when the CMake policy CMP0220 is set to NEW. Now simply calling enable_language is sufficient to get the behavior of write_language.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The cmake-format.json is up to date with these changes.
  • I have added new files under rapids-cmake/
    • I have added include guards (include_guard(GLOBAL))
    • I have added the associated docs/ rst file and update the api.rst

@arhag23
arhag23 requested a review from a team as a code owner August 13, 2026 19:59
@robertmaynard robertmaynard added feature request New feature or request non-breaking Introduces a non-breaking change labels Aug 13, 2026
In CMake 4.5 and above, the policy CMP0220 controls whether
enable_language will propagate into the current subdirectory/scope.
We need to update the logic so that if this policy exists and is set
that write_language does nothing since enable_language takes care of
everything for us.
Please enter the commit message for your changes. Lines starting
@arhag23
arhag23 force-pushed the write-language-policy branch from 11b166d to bfde838 Compare August 14, 2026 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature request New feature or request non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants