Skip to content
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

openrgb: init at unstable-2020-03-03 #82269

Closed
wants to merge 2 commits into from
Closed

openrgb: init at unstable-2020-03-03 #82269

wants to merge 2 commits into from

Conversation

@evanjs
Copy link
Member

evanjs commented Mar 10, 2020

Motivation for this change

https://gitlab.com/CalcProgrammer1/OpenRGB/-/wikis/home

Currently, there are no GUI-based tools to manage e.g. Razer devices on NixOS.
This utility provides a way to manage RGB functionality for many different types of devices.

Notes

This application will currently segfault if you try to detect or dump devices without access to i2c devices (or rather, it won't perform any length checks, and will try to get the current index of an empty list of SMBus devices)
Alternatively, you can just run the application as root.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

cc @adisbladis


date = "2020-03-03";

src = fetchFromGitHub {

This comment has been minimized.

Copy link
@evanjs

evanjs Mar 10, 2020

Author Member

I used GitHub because we need submodules, and the source seems to be equivalent with what's available on GitLab


buildInputs = [ libusb1 hidapi wrapQtAppsHook ];

propagatedBuildInputs = [ hicolor-icon-theme openrazer-daemon ];

This comment has been minimized.

Copy link
@evanjs

evanjs Mar 10, 2020

Author Member

We probably don't actually need hicolor-icon theme; this was probably added when I was trying to figure out why the application was segfaulting

@evanjs
Copy link
Member Author

evanjs commented Mar 10, 2020

I’m also currently unable to access/list Razer devices from this application, even when running as root.

Didn’t see any explicit deps in the project for openrazer on Linux, so I’m not sure why this is the case or where to look.

@evanjs evanjs force-pushed the evanjs:openrgb branch from 770f5ea to 8f16471 Mar 29, 2020
@evanjs
Copy link
Member Author

evanjs commented Mar 29, 2020

I still get some pretty nasty crashes when attempting to detect devices and etc, but at the very least, my Razer devices now appear and I am able to set various effects.

@evanjs
Copy link
Member Author

evanjs commented Mar 29, 2020

Opened an issue related to the crashes I'm experiencing on the main repository.


Update: Fixed via https://gitlab.com/CalcProgrammer1/OpenRGB/-/commit/eba9fc0213b1ffe77b5bff4daf9f5bdeef3a6d16

@JJJollyjim
Copy link
Contributor

JJJollyjim commented Jun 19, 2020

Is there a reason for this to remain a draft? Version 0.2 is out now, btw.

@JJJollyjim
Copy link
Contributor

JJJollyjim commented Jun 20, 2020

Superseded by #90691, which has been merged.

@evanjs evanjs closed this Jun 20, 2020
@evanjs
Copy link
Member Author

evanjs commented Jun 20, 2020

Closed by #90691

@jonringer
Copy link
Contributor

jonringer commented Jun 20, 2020

ah, sorry I didn't see this.

I also used the process to make this "how to add a package to nixpkgs video" https://www.youtube.com/watch?v=fvj8H5yUKu8

@evanjs
Copy link
Member Author

evanjs commented Jun 20, 2020

ah, sorry I didn't see this.

I also used the process to make this "how to add a package to nixpkgs video" https://www.youtube.com/watch?v=fvj8H5yUKu8

All good! Thanks for getting it merged! One more TODO off my list! 😀

Also: did not realize you made that video, thanks!
This should be helpful for getting the new guys into NixOS/nixpkgs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.