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
realsense + examples + udev rules #87503
Conversation
…nixos module providing udev rules
hardware.intel.realsense.enable = mkOption { | ||
type = types.boolean; | ||
default = false; | ||
description = '' Enable (intel) librealsense udev rules ''; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
description = '' Enable (intel) librealsense udev rules ''; | |
description = '' | |
Enable (intel) librealsense udev rules | |
''; |
chmod +x $udev_bins_path/* | ||
cp config/99-realsense-libusb.rules $out/lib/udev/rules.d/99-realsense-libusb.rules | ||
sed -i -e "s@/usr/local/bin/\\(usb-R200-in_udev\\|usb-R200-in\\)@$udev_bins_path/\\1@" -e "s@/bin/sh@$(type -p sh)@" $udev_bins_path/* $out/lib/udev/rules.d/99-realsense-libusb.rules | ||
''; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is good. Please open an issue upstream, fix indentation, and link in a comment here to the issue upstream.
}: | ||
|
||
# See ./nixos/modules/hardware/intel-realsense.nix's option to also get udev rules | ||
# TODO: turn buildExamples into multi output derivation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd advise to do this before we'd continue the review.
hardware.intel.realsense.package = mkOption { | ||
type = types.package; | ||
default = pkgs.librealsenseWithExamples; | ||
description = '' use pkgs.librealsense to avoid examples. The examples for instance contain realsense-viewer.''; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
description = '' use pkgs.librealsense to avoid examples. The examples for instance contain realsense-viewer.''; | |
description = '' | |
The librealsense package to use. | |
''; | |
example = pkgs.librealsenseWithCuda; |
|
||
hardware.intel.realsense.package = mkOption { | ||
type = types.package; | ||
default = pkgs.librealsenseWithExamples; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
default = pkgs.librealsenseWithExamples; | |
default = pkgs.librealsense; |
Note the merge conflicts. Also please check whether the examples by any chance retain references to |
I marked this as stale due to inactivity. → More info |
No response to reviews and no compliance to https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md, closing. |
realsense-viewer still complains about udev rules being outdated cause it seems to byte compare with the patched version. But it seems to work.