Skip to content

nixos/hardware.dslogic: init - #453877

Open
antono wants to merge 1 commit into
NixOS:masterfrom
antono:hardware-dslogic
Open

nixos/hardware.dslogic: init#453877
antono wants to merge 1 commit into
NixOS:masterfrom
antono:hardware-dslogic

Conversation

@antono

@antono antono commented Oct 20, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

This PR adds support for DreamSourceLab DSLogic devices by creating a new NixOS hardware module that provides udev rules.

  • Adds a new hardware module for DSLogic device support
  • Includes udev rules for USB device access permissions

Things done

Add a 👍 reaction to pull requests you find important.

Copilot AI review requested due to automatic review settings October 20, 2025 14:04

This comment was marked as spam.

Sign in to view

@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` 6.topic: hardware Drivers, Firmware and Kernels labels Oct 20, 2025
@antono

antono commented Oct 20, 2025

Copy link
Copy Markdown
Member Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 453877
Commit: 63549eb9e09f493b4bc7113df483d98c14f0de07

x86_64-linux

⏩ 2 packages blacklisted:
  • nixos-install-tools
  • tests.nixos-functions.nixos-test

@nixpkgs-ci nixpkgs-ci Bot added the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Oct 20, 2025
@antono antono changed the title Udev rules and software for Dream DreamSourceLab DSLogic devices Udev rules and software for DreamSourceLab DSLogic devices Oct 20, 2025
@antono antono changed the title Udev rules and software for DreamSourceLab DSLogic devices Udev rules for DreamSourceLab DSLogic devices Oct 21, 2025
@antono antono changed the title Udev rules for DreamSourceLab DSLogic devices Udev rules for DSLogic devices Oct 21, 2025
@h7x4 h7x4 added the 8.has: module (new) This PR adds a module in `nixos/` label Oct 23, 2025
das-g
das-g suggested changes Oct 28, 2025
View reviewed changes

@das-g das-g left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please squash your commits and give the resulting commit a commit message conformant to Commit conventions for changes to nixos.

See also https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions

Comment thread nixos/modules/hardware/dslogic.nix
@nixpkgs-ci nixpkgs-ci Bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Oct 28, 2025
@antono

antono commented Oct 30, 2025 •
edited
Loading

Copy link
Copy Markdown
Member Author

Please squash your commits and give the resulting commit a commit message conformant to Commit conventions for changes to nixos.

See also https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions

✔️ Done

Estis interese ekkoni de via profilo, ke vi partoprenas Esperanto agado :) Saluton do!

das-g
das-g approved these changes Oct 30, 2025
View reviewed changes

@das-g das-g left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. (I didn't test it, though, and I know near-nothing about udev rules.)

Tiu ŝanĝo ŝajnas bona al mi. (Sed mi ne provis ĝin, kaj mi tute ne spertas pri udev-reguloj.)

@nixpkgs-ci nixpkgs-ci Bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Oct 30, 2025
@wolfgangwalther

wolfgangwalther commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

@antono, I see you are maintaining a few packages in Nixpkgs, but are not part of the @NixOS/nixpkgs-maintainers team. You likely missed the invite when the team was created. Without membership in this team, you can't be requested for review for your packages properly. Please see https://github.com/NixOS/nixpkgs/tree/master/maintainers#tools-for-maintainers on how to request another invite to join the team.

@wolfgangwalther wolfgangwalther changed the title Udev rules for DSLogic devices nixos/hardware.dslogic: init Nov 6, 2025
@antono

antono commented Nov 29, 2025

Copy link
Copy Markdown
Member Author

@antono, I see you are maintaining a few packages in Nixpkgs, but are not part of the @NixOS/nixpkgs-maintainers team. You likely missed the invite when the team was created. Without membership in this team, you can't be requested for review for your packages properly. Please see https://github.com/NixOS/nixpkgs/tree/master/maintainers#tools-for-maintainers on how to request another invite to join the team.

Thanks. Applied and joined the team.

@nixpkgs-ci nixpkgs-ci Bot added 8.has: changelog This PR adds or changes release notes 8.has: documentation This PR adds or changes documentation labels Mar 11, 2026
@wolfgangwalther
wolfgangwalther removed their request for review March 11, 2026 17:50
@antono
antono force-pushed the hardware-dslogic branch from 71ee3d1 to bcee73e Compare March 12, 2026 08:30
@nixpkgs-ci nixpkgs-ci Bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Jun 5, 2026
@nixpkgs-ci nixpkgs-ci Bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Aug 4, 2026
@antono
nixos/hardware.dslogic: init
aaa96a0 
This commit adds support for DreamSourceLab DSLogic devices by creating
a new NixOS hardware module that provides udev rules.
@antono
antono force-pushed the hardware-dslogic branch from 6fbacfd to aaa96a0 Compare August 6, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wolfgangwalther wolfgangwalther wolfgangwalther left review comments
Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
+1 more reviewer
@das-g das-g das-g approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

6.topic: hardware Drivers, Firmware and Kernels 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: changelog This PR adds or changes release notes 8.has: documentation This PR adds or changes documentation 8.has: module (new) This PR adds a module in `nixos/` 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

Nix documentation
Status: No status

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@antono @wolfgangwalther @das-g @h7x4