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

Swaylock wrapper #8

Closed
danth opened this issue Jul 24, 2022 · 12 comments
Closed

Swaylock wrapper #8

danth opened this issue Jul 24, 2022 · 12 comments
Assignees
Labels
feature A new feature or a feature request

Comments

@danth
Copy link
Owner

danth commented Jul 24, 2022

Description

Within my NixOS configuration I use a wrapper script for Swaylock to set the colors.

This should be merged into Stylix.

Use case

Theming Swaylock :)

@danth danth added the feature A new feature or a feature request label Jul 24, 2022
@danth danth self-assigned this Jul 24, 2022
@danth danth closed this as completed in d528e85 Jul 24, 2022
@Lyndeno
Copy link
Contributor

Lyndeno commented Jul 25, 2022

I am having trouble getting this to apply. I should be able to just run swaylock in the terminal and have this theme correct?

@danth
Copy link
Owner Author

danth commented Jul 25, 2022

Yes. It's supposed to replace swaylock with a shell script.

less "$(which swaylock)" will display the script if it's installed, and a binary file otherwise.

@Lyndeno
Copy link
Contributor

Lyndeno commented Jul 25, 2022

Interesting, seems to still be a binary

@danth
Copy link
Owner Author

danth commented Jul 25, 2022

Perhaps you have another overlay which adds a swaylock package? That could overwrite our script, if the other overlay is applied later.

@Lyndeno
Copy link
Contributor

Lyndeno commented Jul 25, 2022

As far as I know the term "overlay" does not occur in my nixos config. I will dig deeper and see if any other input could be affecting it.

@danth
Copy link
Owner Author

danth commented Jul 25, 2022

I just checked, and Home Manager has an option for Swaylock now. Perhaps using that rather than a wrapper script would help.

@Lyndeno
Copy link
Contributor

Lyndeno commented Jul 25, 2022

Worth a shot. It is worth noting that module only exists in home-manager master, and not release-22.05 version. Not sure if that presents a problem

@danth
Copy link
Owner Author

danth commented Jul 25, 2022

I think it's possible to check existence of the option and disable our module if it's not present.

danth added a commit that referenced this issue Jul 25, 2022
@danth
Copy link
Owner Author

danth commented Jul 25, 2022

@Lyndeno I've edited the module. Can you try again?

@Lyndeno
Copy link
Contributor

Lyndeno commented Jul 26, 2022

It works for me now, thanks!

@0x715C
Copy link

0x715C commented Oct 15, 2022

Using home-manager/release-22.05 i get, error: The option `home-manager.users.${USER}.programs.swaylock' does not exist.

@danth
Copy link
Owner Author

danth commented Oct 16, 2022

@0x715C Should be working now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature or a feature request
Projects
None yet
Development

No branches or pull requests

3 participants