tristan/nixinate
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity

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

Merged
MagicRB merged 1 commit from master into master 2022-06-10 14:40:30 +00:00
Conversation 5 Commits 1 Files changed 2 +11 -2
MagicRB commented 2022-06-07 21:20:18 +00:00 (Migrated from github.com)
Copy link

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

Signed-off-by: Magic_RB <magic_rb@redalder.org>
Kranzes commented 2022-06-07 21:24:09 +00:00 (Migrated from github.com)
Copy link

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

Isn't using substitutors on the target machine inherently better, shouldn't that just be the default/always enabled?
MagicRB commented 2022-06-07 21:24:19 +00:00 (Migrated from github.com)
Copy link

thats up to matthew to decide

thats up to matthew to decide
Kranzes commented 2022-06-07 21:26:52 +00:00 (Migrated from github.com)
Copy link

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

I guess it isn't inherently better if downloading from the builder is faster than from the substitutors.
MatthewCroughan commented 2022-06-10 14:39:38 +00:00 (Migrated from github.com)
Copy link

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.

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 2022-06-10 14:40:13 +00:00 (Migrated from github.com)
Copy link

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

This passed CI https://hercules-ci.com/github/MatthewCroughan/nixinate/jobs/38
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tristan/nixinate#22
No description provided.
Delete branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?