search

cachix / install-nix-action

Installs Nix on GitHub Actions for the supported platforms: Linux and macOS.
Apache License 2.0
513 stars 79 forks source link

Don't add daemon option if no-daemon is supplied #150

Closed carlosdagos closed 1 year ago

carlosdagos commented 1 year ago

Hey @domenkozar 👋🏼

I am using Fedora builders which don't like the daemon option (because of selinux).

I supplied --no-daemon to the installer options but saw that the installer still adds those.

This should make it so that we can do single-user installs.

Thanks!

blaggacao commented 1 year ago

I'd favor this over #137 since the installer interface is already explicit enough so that there is no need to wrap it as in #137, in my opinion.

domenkozar commented 1 year ago

Seems like CI is broken?

blaggacao commented 1 year ago

Seems like CI is broken?

According to the logs I glanced at the other day, presumably because of https://github.com/cachix/install-nix-action/issues/148