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

Adding roslint dependency to fuse_viz #231

Merged
merged 2 commits into from Jul 20, 2021
Merged

Adding roslint dependency to fuse_viz #231

merged 2 commits into from Jul 20, 2021

Conversation

ayrton04
Copy link
Contributor

https://build.ros.org/job/Nbin_ufv8_uFv8__fuse_viz__ubuntu_focal_arm64__binary/2/consoleFull#console-section-15

07:23:47 CMake Error at CMakeLists.txt:116 (find_package):
07:23:47   By not providing "Findroslint.cmake" in CMAKE_MODULE_PATH this project has
07:23:47   asked CMake to find a package configuration file provided by "roslint", but
07:23:47   CMake did not find one.
07:23:47 
07:23:47   Could not find a package configuration file provided by "roslint" with any
07:23:47   of the following names:
07:23:47 
07:23:47     roslintConfig.cmake
07:23:47     roslint-config.cmake
07:23:47 
07:23:47   Add the installation prefix of "roslint" to CMAKE_PREFIX_PATH or set
07:23:47   "roslint_DIR" to a directory containing one of the above files.  If
07:23:47   "roslint" provides a separate development package or SDK, be sure it has
07:23:47   been installed.

Strangely, this passed the prerelease tests.

@ayrton04 ayrton04 requested a review from efernandez July 20, 2021 08:26
fuse_viz/package.xml Outdated Show resolved Hide resolved
@efernandez
Copy link
Collaborator

At least 1 approving review is required by reviewers with write access.

It looks like you'd have to add @svwilliams to the reviewers in order to merge this.

@ayrton04 ayrton04 requested a review from svwilliams July 20, 2021 10:21
@ayrton04 ayrton04 merged commit da8a140 into devel Jul 20, 2021
@ayrton04 ayrton04 deleted the viz-dependency branch July 20, 2021 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants