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 pkgs #29319

Closed
wants to merge 7 commits into from
Closed

Update pkgs #29319

wants to merge 7 commits into from

Conversation

matthiasbeyer
Copy link
Contributor

Motivation for this change

This updates a bunch of packages where I'm the current maintainer.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@bjornfor
Copy link
Contributor

Built locally with nox-review. Applied to master (b0237f7 and parents). You had comment about why tests were disabled for "rtv". I added that comment into the expression as well.

@bjornfor bjornfor closed this Sep 13, 2017
@bjornfor
Copy link
Contributor

How well have these been tested? Should I backport to release-17.09 right away?

@matthiasbeyer
Copy link
Contributor Author

@bjornfor I wouldn't backport them, I guess. The updates are too small IMO.

@matthiasbeyer matthiasbeyer deleted the update-pkgs branch September 14, 2017 07:30
@bjornfor
Copy link
Contributor

Ok. I backported only sqlitebrowser (bec8a34).

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

2 participants