Skip to content

cmake: improve install prefix handling and clarity#232

Open
venkata-p2498 wants to merge 2 commits intoOpenMoonRay:mainfrom
venkata-p2498:fix/cmake-install-prefix-cleanup
Open

cmake: improve install prefix handling and clarity#232
venkata-p2498 wants to merge 2 commits intoOpenMoonRay:mainfrom
venkata-p2498:fix/cmake-install-prefix-cleanup

Conversation

@venkata-p2498
Copy link
Copy Markdown

Summary

Minor improvement to install prefix handling in root CMake configuration.

Changes

  • Replaced CMAKE_SOURCE_DIR with CMAKE_CURRENT_SOURCE_DIR for correctness
  • Added explanatory comment to clarify default install behavior

Motivation

Improves correctness of path handling and makes intent clearer for maintainers.

Testing

  • Build configured successfully using CMake
  • No functional changes to build output observed

…alization (Issue OpenMoonRay#227)

Signed-off-by: PrashanthBhasutkar <81845583+venkata-p2498@users.noreply.github.com>
@venkata-p2498 venkata-p2498 force-pushed the fix/cmake-install-prefix-cleanup branch 4 times, most recently from 1aefe94 to 94602d3 Compare May 5, 2026 15:20
Signed-off-by: Prashanth B <venkata.p2498@gmail.com>
@venkata-p2498 venkata-p2498 force-pushed the fix/cmake-install-prefix-cleanup branch from 94602d3 to dd3c5f0 Compare May 5, 2026 15:24
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.

1 participant