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

clightning: 0.8.0 -> 0.8.1 #80496

Merged
merged 1 commit into from Feb 19, 2020
Merged

Conversation

r-ryantm
Copy link
Contributor

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://github.com/ElementsProject/lightning/releases.

meta.description for clightning is: '"A Bitcoin Lightning Network implementation in C"'.

meta.homepage for clightning is: '"https://github.com/ElementsProject/lightning"

Release on GitHub

Compare changes on GitHub

Checks done (click to expand)
Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
clightning

Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/qkjxjn5cw8dsscg8lpr36hcf2zp90r38-clightning-0.8.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A clightning https://github.com/r-ryantm/nixpkgs/archive/3f3ead76fdb741a0dd71a694a7b24efa024848d8.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/qkjxjn5cw8dsscg8lpr36hcf2zp90r38-clightning-0.8.1
ls -la /nix/store/qkjxjn5cw8dsscg8lpr36hcf2zp90r38-clightning-0.8.1/bin

cc @jb55 for testing.

@ofborg ofborg bot requested a review from jb55 February 19, 2020 03:47
@mmahut mmahut merged commit 1a6d657 into NixOS:master Feb 19, 2020
@r-ryantm r-ryantm deleted the auto-update/clightning branch February 19, 2020 14:35
@jb55
Copy link
Contributor

jb55 commented Feb 19, 2020

post-merge ACK, I checked the signatures. @mmahut I think we should have at least one signature check ACK before merging updates to financial software 😅 or at least until we have signature checking as a part of the derivation

@mmahut
Copy link
Member

mmahut commented Feb 20, 2020

@jb55 I did check that update as I was already playing with it few hours ago.

I'm sorry, I should have at least written something before merging it. I will be more careful next time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants