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

Add Include Guard in Sample Modules #22

Merged
merged 2 commits into from
Jan 11, 2024
Merged

Add Include Guard in Sample Modules #22

merged 2 commits into from
Jan 11, 2024

Conversation

threeal
Copy link
Owner

@threeal threeal commented Jan 11, 2024

This pull request resolves #21 by introducing the following changes:

  • Adds the include_guard function on the top of the sample MkdirRecursive.cmake module to prevent the function from being included twice.
  • Modifies the sample MyMkdirConfig.cmake config file to append the CMAKE_CURRENT_LIST_DIR variable to the CMAKE_MODULE_PATH variable only if the CMAKE_CURRENT_LIST_DIR variable is not already present in the CMAKE_MODULE_PATH variable.

@threeal threeal self-assigned this Jan 11, 2024
@threeal threeal added the feat label Jan 11, 2024
@threeal threeal added this to the Version 1.1.0 milestone Jan 11, 2024
@threeal threeal merged commit 5369e47 into main Jan 11, 2024
2 checks passed
@threeal threeal deleted the add-include-guard branch January 11, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Include Guard in Sample Modules
1 participant