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

bug: Shouldn't programs.swaylock.package allow null? #6090

Open
2 tasks done
s0racat opened this issue Nov 16, 2024 · 2 comments · May be fixed by #6095
Open
2 tasks done

bug: Shouldn't programs.swaylock.package allow null? #6090

s0racat opened this issue Nov 16, 2024 · 2 comments · May be fixed by #6095
Assignees
Labels
bug triage Issues or feature request that have not been triaged yet

Comments

@s0racat
Copy link

s0racat commented Nov 16, 2024

Are you following the right branch?

  • My Nixpkgs and Home Manager versions are in sync

Is there an existing issue for this?

  • I have searched the existing issues

Issue description

swaylock requires additional configuration for non NixOS and non-sway environments.
Shouldn't such things be left to NixOS and the system?

Maintainer CC

@rcerc

System information

- system: `"x86_64-linux"`
 - host os: `Linux 6.6.60, NixOS, 24.11 (Vicuna), 24.11.20241113.24f0d4a`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.24.10`
 - nixpkgs: `/nix/store/c05d1sqfhkl93p3j5ykic68mgg1gsrvb-source`
@s0racat s0racat added bug triage Issues or feature request that have not been triaged yet labels Nov 16, 2024
@rcerc rcerc linked a pull request Nov 17, 2024 that will close this issue
6 tasks
@rcerc
Copy link
Contributor

rcerc commented Nov 17, 2024

Thanks for reporting! I have filed #6095 to change this. In the meantime, you could set

package = pkgs.runCommandLocal "dummy" { } "mkdir $out";

@s0racat
Copy link
Author

s0racat commented Nov 17, 2024

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug triage Issues or feature request that have not been triaged yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants