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
openhantek6022: init at 3.1.1 #93613
Conversation
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.
Note that evaluation fails. Also added some comments.
pkgs/applications/science/electronics/openhantek6022/udev-rules.patch
Outdated
Show resolved
Hide resolved
pkgs/applications/science/electronics/openhantek6022/cmake-paths.patch
Outdated
Show resolved
Hide resolved
pkgs/applications/science/electronics/openhantek6022/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/science/electronics/openhantek6022/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/science/electronics/openhantek6022/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/science/electronics/openhantek6022/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/science/electronics/openhantek6022/default.nix
Outdated
Show resolved
Hide resolved
63c2d7c
to
ffdf30d
Compare
Not sure why the evaluation check fails, I can build and install it on my machine 🙈 |
e32258b
to
5556c9f
Compare
@danieldk thank you for the review and suggestions. I am still not sure why the checks fail. I have run the commands on mentioned on NixOS/ofborg and did not complain. Can you point me where to look for the problem? |
|
5556c9f
to
956d35b
Compare
I think I finally get it:
hides aliasses, which results in
|
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.
LGTM
Result of nixpkgs-review pr 93613
1
1 package built:
- openhantek6022
But it would be nice if someone else could also review this. I am currently on the go and cannot test GUI applications.
Motivation for this change
Adding this package for a cheap USB oscilloscope I bought recently.
On NixOS, the package can be added to
services.udev.packages
to set up udev rules.TAG+="uaccess", TAG+="udev-acl"
did not do the trick. I have addedGROUP="plugdev", MODE="0666"
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)