Skip to content

fix binary cache for nixos #1448

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

Merged
merged 2 commits into from
Apr 21, 2022
Merged

fix binary cache for nixos #1448

merged 2 commits into from
Apr 21, 2022

Conversation

goertzenator
Copy link
Contributor

The instructions for setting up the binary cache on NixOS produce the following error:

[root@r9:~]# nixos-rebuild switch  
error: The option `nix.settings' does not exist. Definition values:  
- In `/etc/nixos/configuration.nix':  
   {  
     substituters = [  
       "https://hydra.iohk.io"  
     ];  
     trusted-public-keys = [  
   ...  
(use '--show-trace' to show detailed location information) 

The PR fixes this. I am using this on my NixOS system and it seems to work fine.

@hamishmack
Copy link
Collaborator

It looks like we merged #1410 too soon. I presume you must have a slightly older NixOS. Can you please update this PR to include both old and new settings? It would be good if we could find out when nix.settings.trusted-public-keys support was added to NixOS, but if not just calling them "Newer NixOS" and "Older NixOS" would work.

@goertzenator
Copy link
Contributor Author

Apologies; I will investigate.

add nixos binary cache instruction for both <=21.11 and >21.11
@goertzenator
Copy link
Contributor Author

Fixed.

So, this change arrived after nixos-21.11. Although 21.11 is the current stable release of NixOS, the next release is right around the corner and will make this PR obsolete. To keep things simpler and clearer going forward, I suggest not merging this PR.

@michaelpj
Copy link
Collaborator

I don't see any harm in having a correct addendum like this, I think let's just include it. Thanks!

@michaelpj michaelpj merged commit e4248b3 into input-output-hk:master Apr 21, 2022
@goertzenator goertzenator deleted the patch-1 branch April 21, 2022 16:10
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.

3 participants