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 an option for INSTALL_PREFIX #1

Open
wants to merge 3 commits into
base: feat/protocol
Choose a base branch
from

Conversation

YuanYuYuan
Copy link

Sometimes, we want to install zenoh-pico locally without the need for root privilege. For example, running zenoh-pico-perf. 😁

With this PR, one can achieve it by

INSTALL_PREFIX=./install make
make install

to have zenoh-pico installed under a local directory named "install".

The changes include

  • Modify the CMakeLists.txt to enable the control of CMAKE_INSTALL_PREFIX. This lets users change the prefix via cmake.
  • Add an option for INSTALL_PREFIX. This lets users change the prefix via make.
  • Upgrade codacy/codacy-analysis-cli-action since it causes some outdated errors in CI.

@YuanYuYuan YuanYuYuan requested a review from p-avital August 31, 2023 06:57
zettascale-bot pushed a commit that referenced this pull request Sep 3, 2024
…ake_package_config

keep feature defines in PackageConfig.cmake.in
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