Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

common/options: validate see-also #42845

Merged
merged 2 commits into from Aug 24, 2021
Merged

common/options: validate see-also #42845

merged 2 commits into from Aug 24, 2021

Commits on Aug 24, 2021

  1. common/options: validate see-also 

    y2c.py is like a compiler which translates .yaml to .cc and .h files,
it does not have access to all .yaml files. to validate the dangling
see-also issue, we need to do this with a "linker".

in this change, validate-options.py is introduced to check if any of
option name included by the see-also property is valid.

Fixes: https://tracker.ceph.com/issues/51483
Signed-off-by: Kefu Chai <kchai@redhat.com>
    @tchaikov
    tchaikov committed Aug 24, 2021
    Configuration menu
    Copy the full SHA
    6cfdd40 View commit details
    Browse the repository at this point in the history

  2. cmake: s/Python_EXECUTABLE/Python3_EXECUTABLE/ 

    as FindPython3.cmake only sets Python3_EXECUTABLE for us, we should
stick with Python3_EXECUTABLE instead of Python_EXECUTABLE.

Signed-off-by: Kefu Chai <kchai@redhat.com>
    @tchaikov
    tchaikov committed Aug 24, 2021
    Configuration menu
    Copy the full SHA
    a66daec View commit details
    Browse the repository at this point in the history