gpu-screen-recorder-ui: init at 1.1.7

[js6pak]

https://git.dec05eba.com/gpu-screen-recorder-ui/about/
Fullscreen overlay UI for GPU Screen Recorder in the style of ShadowPlay

Closes #369558
Depends on #367552

Usage

1. Enable the module to add the setcap wrappers and systemd service

programs.gpu-screen-recorder-ui.enable = true;

2. Start the program using one of the following ways:
   1. Run gsr-ui
   2. Enable the service imperatively systemctl enable --now --user gpu-screen-recorder-ui
   3. Enable the service declaratively

      systemd.user.services.gpu-screen-recorder-ui.wantedBy = [ "default.target" ];

      Note: This will break the "Startup" setting in-app
      Question: Should this be a nixos module option?
3. Press Alt+Z (key binds are not configurable yet)

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[lucasew]

Please introduce the packages in pkgs/by-name instead

EDIT: it will be easier to maintain because of bots

[js6pak]

Please introduce the packages in pkgs/by-name instead

I want to move gpu-screen-recorder{,-gtk} first, not sure whether in another PR or this yet.

[keenanweaver]

Nice work! Here are my overall observations:

• I think the ui module should be consolidated into the existing gpu-screen-recorder module instead of being a separate one. For instance programs.gpu-screen-recorder.ui.enable = true;
• The service should be started by default with the ui.enable option set to true, but allow to be disabled.
• Maybe allow for the notification to be disabled. Not sure why people would do that, but you never know.
• Agreed that the package should be moved to by-name.

[lucasew]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 369574

---

x86_64-linux

⏩ 1 package blacklisted:

• nixos-install-tools

✅ 2 packages built:

• gpu-screen-recorder-notification
• gpu-screen-recorder-ui

[js6pak]

@keenanweaver

• The service should be started by default with the ui.enable option set to true, but allow to be disabled.

That will break this setting in the app (it just won't have any effect), not sure whether we want to do it by default

• Maybe allow for the notification to be disabled. Not sure why people would do that, but you never know.

You can disable it in-app

[Atemu]

I want to move gpu-screen-recorder{,-gtk} first, not sure whether in another PR or this yet.

I don't see a any dependency on that.

You must init this in by-name by the way; it's policy now.

[keenanweaver]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 369574

---

x86_64-linux

✅ 2 packages built:

• gpu-screen-recorder-notification
• gpu-screen-recorder-ui

[keenanweaver]

All functionality is seemingly working as intended. My recordings look weird but I probably have to set the appropriate options first.