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.
After installing
librealsense
with Conda we found that some of the utility tools, such asrealsense-viewer
are not included. That makes perfect sense, because those tools are optional (they are even behind aBUILD_GRAPHICAL_EXAMPLES
CMake option) and pull in a lot more dependencies. Whenlibrealsense
is installed throughapt
using Intel's PPA, you have to install a separate package to get those tools (librealsense2-utils
).This is just a Draft PR showing how the viewer can be build, but as you can see it's certainly not optimal and pulls in quite a lot of extra dependencies. What do you think is the best way to tackle this? As I see it, I could:
realsense-viewer
actually works with the real hardware), but it means that withlibrealsense
, users will also pull in a lot of extra dependencieslibrealsense
'core' library, vs. depending on it somehow)conda build
can do, but we would need a bit of help...)Looking forward to your view on this!
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)