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

installer: fix missing path to cat #27

Merged
merged 1 commit into from
Jan 23, 2024
Merged

installer: fix missing path to cat #27

merged 1 commit into from
Jan 23, 2024

Conversation

Lillecarl
Copy link
Contributor

Fixes cat not being accessible by installer script

  • Allow edits by maintainers

@gmodena
Copy link
Owner

gmodena commented Jan 23, 2024

Hey @Lillecarl ; Thanks for this!

Would you mind sharing some details of the system where this bug popped up? Was it on nixos or another distro?

I did not catch it with integration tests and could not reproduce (nixos).

@gmodena gmodena merged commit bb339db into gmodena:main Jan 23, 2024
1 check passed
@Lillecarl
Copy link
Contributor Author

Hi! NixOS-unstable, home-manager master (without nixos module).

Caught by systemd unit failing and home-manager rebuild reporting a failed unit.

See HM systemd.user.startServices = "SD-switch" ;

@gmodena
Copy link
Owner

gmodena commented Jan 23, 2024

@Lillecarl ack. Thanks for sharing!

@Lillecarl
Copy link
Contributor Author

@gmodena any time, you can find my config at lillecarl/nixos if there's any specific you're looking for.

I'd put coreutils and friends in the systemd path rather than messing with all these package paths.

Coreutils "cost" the same in systemd as in a template reference.

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

Successfully merging this pull request may close these issues.

2 participants