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

incomplete substituters file #98

Closed
benneti opened this issue Jan 21, 2021 · 2 comments
Closed

incomplete substituters file #98

benneti opened this issue Jan 21, 2021 · 2 comments
Labels
APIv1 related to 1.0 api stabilization blocked something upstream needs fixing first bug Something isn't working priority upstream concerned with upstream

Comments

@benneti
Copy link
Contributor

benneti commented Jan 21, 2021

Describe the bug
I think the incompleteness of
nixConfig.substituters = [ "https://nrdxp.cachix.org" ];
confuses nix, I follow nixos-unstable with my pkgs and had to add

# add links for binary caches here
  nixConfig.substituters = [
    "https://cache.nixos.org/"
    "https://nrdxp.cachix.org"
    "https://nix-community.cachix.org"
  ];

to again use the binary cache.

To Reproduce
Use flakes

  1. maybe switch to nixos.url = "nixpkgs/nixos-unstable";
  2. rebuild system

To Fix
add

  nixConfig.substituters = [
    "https://cache.nixos.org/"
    "https://nrdxp.cachix.org"
    "https://nix-community.cachix.org"
  ];

Expected behavior
have completer substituters (or none?).
Also I am a bit confused, do we still rely on the cachix configuration -- if not I think it should be deleted for it to be less confusing.

I think this PR is related: #71.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@benneti benneti changed the title icomplete substituters file incomplete substituters file Jan 21, 2021
@nrdxp
Copy link
Collaborator

nrdxp commented Jan 23, 2021

After reviewing upstream, this config isn't even sent to the daemon yet. Once it is, we should unify the nix.conf everywhere using this mechanism, but for now it's causing issues with no benefit and should simply be reverted.

Relevant:
NixOS/nix@343239f#diff-3a977e1e533a3b628a371bf19103f87d5501dc35df5fb1142551a9896cf3b03aR539

I'll leave this open until the time comes.

@nrdxp nrdxp mentioned this issue Feb 15, 2021
17 tasks
@nrdxp nrdxp added bug Something isn't working priority blocked something upstream needs fixing first labels Feb 17, 2021
@blaggacao blaggacao added the APIv1 related to 1.0 api stabilization label Apr 11, 2021
@blaggacao blaggacao added the upstream concerned with upstream label Jun 13, 2021
@Pacman99
Copy link
Member

Closing this, I believe substituters in nixConfig and cachix profile are good to go. Feel free to re-open if this still needs some work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APIv1 related to 1.0 api stabilization blocked something upstream needs fixing first bug Something isn't working priority upstream concerned with upstream
Projects
None yet
Development

No branches or pull requests

4 participants