thmzlt
March 5, 2020, 12:43pm
109
Addressed your comments on the PR.
Shou
March 8, 2020, 12:14am
110
These two PRs of mine have been reviewed and sitting around for a while now – can probably be merged
NixOS:master
← Shou:init-obinskit
opened 08:57PM - 16 Oct 19 UTC
###### Motivation for this change
This is the configuration software for the … enthusiast Anne Pro mechanical keyboard, used to update the firmware, change keyboard settings, clear Bluetooth connections from the device, set up backlighting colours, and so forth. I'm an owner of one and don't think I'm alone so I thought it'd be convenient to have this packaged. [They have an official Github](https://github.com/obinslab), but the software is proprietary to my knowledge (no license provided).
I've tested that the software works with my own Anne Pro II.
###### Things done
- [x] Tested using sandboxing ([nix.useSandbox](http://nixos.org/nixos/manual/options.html#opt-nix.useSandbox) on NixOS, or option `sandbox` in [`nix.conf`](http://nixos.org/nix/manual/#sec-conf-file) on non-NixOS)
- Built on platform(s)
- [x] NixOS
- [ ] macOS
- [ ] other Linux distributions
- [ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
- [ ] Tested compilation of all pkgs that depend on this change using `nix-shell -p nix-review --run "nix-review wip"`
- [x] Tested execution of all binary files (usually in `./result/bin/`)
- [ ] Determined the impact on package closure size (by running `nix path-info -S` before and after)
- [ ] Ensured that relevant documentation is up to date
- [x] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md).
NixOS:master
← Shou:init-mkchromecast
opened 05:49PM - 07 Mar 19 UTC
###### Motivation for this change
###### Things done
- [x] Tested us… ing sandboxing ([nix.useSandbox](http://nixos.org/nixos/manual/options.html#opt-nix.useSandbox) on NixOS, or option `sandbox` in [`nix.conf`](http://nixos.org/nix/manual/#sec-conf-file) on non-NixOS)
- Built on platform(s)
- [ ] NixOS
- [ ] macOS
- [x] other Linux distributions
- [ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
- [ ] Tested compilation of all pkgs that depend on this change using `nix-shell -p nox --run "nox-review wip"`
- [x] Tested execution of all binary files (usually in `./result/bin/`)
- [ ] Determined the impact on package closure size (by running `nix path-info -S` before and after)
- [ ] Assured whether relevant documentation is up to date
- [ ] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md).
---
Merge: ripcord: init at 0.4.23 by bqv · Pull Request #80402 · NixOS/nixpkgs · GitHub
I tested many of the features that the application has and it all seems to work great!
jonringer has already accepted this PR but worldofpeace hasn’t re-reviewed the latest changes. All their requested changes have been applied however so I think it should be fine.
Thank you all for your hard work!
clkamp
March 10, 2020, 8:51am
112
palo
March 11, 2020, 6:09am
113
This pull request is reviewd for quite some time.
https://github.com/NixOS/nixpkgs/pull/72160
FelschR
March 14, 2020, 11:39am
115
This has been open a while but I feel it is trivial:
NixOS:master
← alexarice:variant-ant-themes
opened 08:31AM - 28 Aug 19 UTC
###### Things done
- [x] Tested using sandboxing ([nix.useSandbox](http://ni… xos.org/nixos/manual/options.html#opt-nix.useSandbox) on NixOS, or option `sandbox` in [`nix.conf`](http://nixos.org/nix/manual/#sec-conf-file) on non-NixOS)
- Built on platform(s)
- [x] NixOS
- [ ] macOS
- [ ] other Linux distributions
- [ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
- [ ] Tested compilation of all pkgs that depend on this change using `nix-shell -p nix-review --run "nix-review wip"`
- [x] Tested execution of all binary files (usually in `./result/bin/`)
- [ ] Determined the impact on package closure size (by running `nix path-info -S` before and after)
- [ ] Ensured that relevant documentation is up to date
- [x] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md).
###### Notify maintainers
cc
Enteee
March 19, 2020, 6:31pm
119
I added some comments, which the author addressed. Looks good to go!
I cross checked debian and arch, should be good to merge:
NixOS:staging
← primeos:tcp-cong-switch-to-cubic
opened 10:00AM - 02 Mar 20 UTC
This will switch the default TCP congestion control algorithm from
new Reno to … CUBIC. CUBIC is the default since Linux kernel 2.6.19
(see 597811ec167fa) and most (all?) distributions keep this default
(e.g. Debian and Ubuntu). On NixOS the default was still new Reno
because generate-config.pl changes TCP_CONG_CUBIC from y to m (since we
try to build everything as a module by default).
To check the active and available algorithms:
$ sysctl net.ipv4.tcp_congestion_control
net.ipv4.tcp_congestion_control = cubic
$ sysctl net.ipv4.tcp_available_congestion_control
net.ipv4.tcp_available_congestion_control = cubic reno
Note: E.g. x86_64_defconfig sets TCP_CONG_CUBIC=y indirectly via
CONFIG_TCP_CONG_ADVANCED=y (but CUBIC is also the default if set to no,
see net/ipv4/Kconfig).
```
Edit: Some existing diffs (old NixOS kernel configuration vs. other distributions):
- Ubuntu: https://github.com/NixOS/nixpkgs/issues/58275#issuecomment-477203331
- Arch: https://github.com/NixOS/nixpkgs/issues/39225#issuecomment-385281048
```
###### Motivation for this change
###### Things done
- [ ] Tested using sandboxing ([nix.useSandbox](http://nixos.org/nixos/manual/options.html#opt-nix.useSandbox) on NixOS, or option `sandbox` in [`nix.conf`](http://nixos.org/nix/manual/#sec-conf-file) on non-NixOS linux)
- Built on platform(s)
- [ ] NixOS
- [ ] macOS
- [ ] other Linux distributions
- [ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
- [ ] Tested compilation of all pkgs that depend on this change using `nix-shell -p nixpkgs-review --run "nixpkgs-review wip"`
- [ ] Tested execution of all binary files (usually in `./result/bin/`)
- [ ] Determined the impact on package closure size (by running `nix path-info -S` before and after)
- [ ] Ensured that relevant documentation is up to date
- [ ] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md).
Close:
NixOS:master
← oxij:nixos/doc-generate-config
opened 01:44AM - 04 Mar 18 UTC
###### Motivation for this change
Combination of https://github.com/NixOS/nix… pkgs/issues/17126#issuecomment-315190218 and https://github.com/NixOS/nixpkgs/pull/35292#issuecomment-367714173 got me over the edge.
###### Things done
- [X] It evaluates.
Feel free to point to typos I probably missed.
Reason: it’s a good attempt at making better documentation. But some of the documentation is controversial, and the author does not seem interested in pursuing this PR any longer. And it’s been about 18 months since he last commented on it.