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

wavebox: 3.14.10 -> 4.3.0 #47701

Merged
merged 1 commit into from Oct 12, 2018
Merged

wavebox: 3.14.10 -> 4.3.0 #47701

merged 1 commit into from Oct 12, 2018

Conversation

r-ryantm
Copy link
Contributor

@r-ryantm r-ryantm commented Oct 3, 2018

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/wavebox/versions.

meta.description for wavebox is: '"Wavebox messaging application"'.

Checks done (click to expand)
Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

  1. Install cachix from https://cachix.org/
  2. Use r-ryantm's cache:
    cachix use r-ryantm
    

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:

nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/ebe377b46ec8fff1b602d750180b36e37a8aa121.tar.gz -p wavebox
ls -la /nix/store/xijd4hnykcd5pckp6qwiz1fnhw9pjb7z-wavebox-4.3.0

cc @rawkode for testing.

Semi-automatic update generated by
https://github.com/ryantm/nixpkgs-update tools. This update was made
based on information from
https://repology.org/metapackage/wavebox/versions
@rawkode
Copy link
Member

rawkode commented Oct 6, 2018

Installed & Working 👍

@c0bw3b
Copy link
Contributor

c0bw3b commented Oct 8, 2018

Wow pretty cool to see that a major version bump of a quite big Electron app could be "that" easy. ^^

@rawkode out of sheer curiosity and since the maintainer is around -> why is it necessary to pass both gtk2 and gtk3 as build inputs to this?

@c0bw3b c0bw3b merged commit 992b13b into NixOS:master Oct 12, 2018
@rawkode
Copy link
Member

rawkode commented Oct 12, 2018

Hi @c0bw3b, I think only gtk3 is required, now. I'll run some tests

@c0bw3b
Copy link
Contributor

c0bw3b commented Oct 12, 2018

Ok well I went ahead and merge this one but ping me if you submit a follow-up PR to remove gtk2

@r-ryantm r-ryantm deleted the auto-update/wavebox branch October 13, 2018 22:25
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

4 participants