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
usb-modeswitch: 2.2.1 -> 2.5.0 and nixos module #22264
Conversation
We were carrying v2.2.1 and not v2.3.0 due to the update only touching the version and not the sha256 sum.
Plus some minor cleanups.
57f82d2
to
73c42eb
Compare
@MarcWeber, any comments? |
nixos/modules/hardware/usb_wwan.nix
Outdated
|
||
options = { | ||
|
||
hardware.usb_wwan = { |
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.
The consensus is to use camelCase for nixos options instead of separators.
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.
Changed. camelCase abbreviations just look really ugly though...
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.
In this case, I agree. :)
73c42eb
to
e82b059
Compare
e82b059
to
0789a2a
Compare
Motivation for this change
We were carrying v2.2.1 and not v2.3.0 due to the update only touching the version and not the sha256 sum.
Additionally, we add a nixos module for setting up udev to use the rules.
Unfortunately, I no longer have the hardware that made me write the module, so I cannot actually test if it works. I thought it would be better to get things into nixpkgs instead of lingering around here...
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)