place default keystore to install space #103
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR implements the keystore location in install space, based on discussion from #101 (comment)
The original implementation aimed at creating the default keystore in the
sros2_cmake
workspace install space under aros2_security
directory.https://github.com/ross-desmond/sros2/blob/51fd900a1665bd6a4d760aa501488401f32946f3/sros2_cmake/sros2_cmakeConfig.cmake.in#L3-L5
In the case of installing from packages, this logic results in creating the keystore at
/opt/ros/crystal/share/sros2_cmake/cmake/../../../ros2_security
(which requires root access).Changes from original implementation:
sros2_cmake
's workspace (aka results in the same directory whether the underlay was built in isolated install or merged install layout)keys
is renamedkeystore
@ross-desmond
Does this match the expected behavior?
What files do we expect to be placed in the
ros2_security
folders other than the keystore? (user policy files maybe?). If none maybe 1 layer of subdirectory would be enough