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.

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

ensure correct cmake script argument #169

Conversation

fransklaver
Copy link
Contributor

When running a build in a scenario with multiple paths in CMAKE_MODULE_PATH, we aren't actually running the git-version script. This happens because we are APPENDing our own module path to the existing setting.

Instead of relying on the contents of CMAKE_MODULE_PATH, create a variable that is dedicated to our own location, and use that to specify the git-version script.

When running a build in a scenario with multiple paths in
CMAKE_MODULE_PATH, we aren't actually running the git-version script.
This happens because we are APPENDing our own module path to the
existing setting.

Instead of relying on the contents of CMAKE_MODULE_PATH, create a
variable that is dedicated to our own location, and use that to specify
the git-version script.

Signed-off-by: Frans Klaver <[email protected]>
Copy link
Contributor

@CoelacanthusHex CoelacanthusHex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@estkme estkme merged commit 1f94717 into estkme-group:main Nov 20, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants