MatthewCroughan / nixinate

Another NixOS Deployment Tool - Nixinate your systems 🕶️
MIT License
240 stars 32 forks source link

Add option to substitute on the target machine if local build #22

Closed MagicRB closed 2 years ago

MagicRB commented 2 years ago

Signed-off-by: Magic_RB magic_rb@redalder.org

Kranzes commented 2 years ago

Isn't using substitutors on the target machine inherently better, shouldn't that just be the default/always enabled?

MagicRB commented 2 years ago

thats up to matthew to decide

Kranzes commented 2 years ago

I guess it isn't inherently better if downloading from the builder is faster than from the substitutors.

MatthewCroughan commented 2 years ago

I've decided not to enable it by default, since it's too risky that the remote will have a low upload bandwidth. Part of the reason someone would want to build locally is because they don't want to be hampered by the remote, so it's best to leave this up to the user rather than decide for them.

MatthewCroughan commented 2 years ago

This passed CI https://hercules-ci.com/github/MatthewCroughan/nixinate/jobs/38