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

Update add-sri.py script to work with current nixpkgs and Nix #7

Merged

Conversation

LeSuisse
Copy link
Contributor

@LeSuisse LeSuisse commented Feb 6, 2023

  • Sources can already be using a SRI hash
  • nix hash to-sri has replaced nix to-sri

@Artturin
Copy link
Member

Artturin commented Feb 7, 2023

you'll also want to run

niv modify nixpkgs --branch nixos-22.11
niv update
niv init

to update the niv pin

* Sources can already be using a SRI hash
* `nix hash to-sri` has replaced `nix to-sri`
@LeSuisse LeSuisse force-pushed the update-sri-script-current-state branch from f6208c5 to d22c7a8 Compare February 7, 2023 13:16
@nlewo
Copy link
Member

nlewo commented Feb 26, 2023

Thank you!

@nlewo nlewo merged commit 5523569 into nix-community:master Feb 26, 2023
@nlewo
Copy link
Member

nlewo commented Feb 26, 2023

I would have preferred to set the integrity attribute at Nix evaluation but a builtin was lacking. I opened an issue and a PR is in progress 🎉

@LeSuisse LeSuisse deleted the update-sri-script-current-state branch February 26, 2023 18:08
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.

None yet

3 participants