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.

Sign up for GitHub

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

Jump to bottom

fix multiple DIRECTORY keywords #49

Merged
merged 1 commit into from
Aug 6, 2021
Merged

fix multiple DIRECTORY keywords #49

merged 1 commit into from
Aug 6, 2021

Conversation

dusty-nv
Copy link
Contributor

@dusty-nv dusty-nv commented Aug 5, 2021

To build some other packages, I had to upgrade CMake (CMake 3.21.1)

I then started getting build errors from zed2-ros-wrapper:

--- stderr: zed_components
CMake Error at cmake_install.cmake:69 (file):
  file INSTALL cannot find
  "/opt/zed/zed-ros2-wrapper/zed_components/DIRECTORY": No such file or
  directory.

It turns out it did not like multiple DIRECTORY statements in the one install() call in CMakeLists.txt
https://cmake.org/cmake/help/latest/command/install.html#directory

@Myzhar Myzhar merged commit 99ed321 into stereolabs:master Aug 6, 2021
peredwardsson added a commit to peredwardsson/zed-ros2-wrapper that referenced this pull request Aug 26, 2021
@peredwardsson
Fixed CMake DIRECTORY error
25d9e6e 
The proposed change makes the eloquent branch look like the master branch on these lines. Looks like this has been fixed previously, but only on master. (stereolabs#49)
@peredwardsson peredwardsson mentioned this pull request Aug 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dusty-nv @Myzhar