PRs already reviewed

I need some help/review for updating vagrant : vagrant: 2.2.3 -> 2.2.5 by angristan · Pull Request #64302 · NixOS/nixpkgs · GitHub

The Gemfile/Gemfile.lock were removed a while back and now it is unclear how to proceed to update the package since it seems to be the usual ruby/gem thing with bundix. I got something working but nobody wants to look at my PR :smiley:

Close (I think?): error message honores nested dots in attribute path elements by qknight · Pull Request #39060 · NixOS/nixpkgs · GitHub
There are significant merge issues, and there hasn’t been any response on the PR in a few weeks. I’m not sure if that is sufficient to close the PR in this community, though.

Merge: https://github.com/NixOS/nixpkgs/pull/63013

Merge: mdbook: 0.1.8 -> 0.3.1 by tazjin · Pull Request #65890 · NixOS/nixpkgs · GitHub
Close old alternative: mdbook: 0.1.8 -> 0.2.0 by dywedir · Pull Request #44494 · NixOS/nixpkgs · GitHub

Merge: spotifyd: update cargoSha256 hash by BenSchZA · Pull Request #66233 · NixOS/nixpkgs · GitHub

1 Like

Close: #64341 (Obsoleted by #66691)
Close: #64501 (Obsoleted by #66691)
Close: #64483 (Obsoleted by #66608)
Close: #62922 (Obsoleted by #65629)
Merge: #66418 (Minor version update, I was using the generated binary for a week)
Merge: #66344 (New package, reviewed, I was using the generated binary for a week)

1 Like

edited: I had made a mistake in the links

I have pr I submitted and have been using without issue for a while now. Basically it allows sound.mediaKeys to work with pulseaudio. Doesn’t seem like anyone else has had time to review it but it works just fine and hasn’t caused me any issues. Should be ready to merge. Hopefully someone can take a look.

https://github.com/NixOS/nixpkgs/pull/67227

Thank you for your PR, but please refer to PRs ready for review _ - #46 by pstch (it says May 2019, but it has become the de-facto long running thread) for review requests :slight_smile:

1 Like

merge: dhall-json: fix build by marsam · Pull Request #67438 · NixOS/nixpkgs · GitHub ([haskell] dhall-json: fix build)
merge: Pass hoogle-local's buildCommand as a file. by TravisWhitaker · Pull Request #67228 · NixOS/nixpkgs · GitHub ([haskell] Pass hoogle-local’s buildCommand as a file)

close: pandoc-placetable: enable inlineMarkdown by unode · Pull Request #67464 · NixOS/nixpkgs · GitHub ([haskell] pandoc-placetable: enable inlineMarkdown)

merge: plan9port: use C compiler from Nix by eraserhd · Pull Request #66404 · NixOS/nixpkgs · GitHub - plan9port: use correct C compiler (right?)

The following PR was approved almost 2 weeks ago but is just waiting on being merged

~~https://github.com/NixOS/nixpkgs/pull/69576~~

I have some:

https://github.com/NixOS/nixpkgs/pull/70573

Updates qtcreator. The change used to be a single line version update. However, the last time, github:@deadloko added a piece of code to add some functionality. He is the author of this PR as well and to me his changes in this commit seem consistent with the changes in previous commit.

Besides, the maintainer has reviewed and approved.

Update of cups for staging: ~~https://github.com/NixOS/nixpkgs/pull/70969~~
Tested and reviewed.

1 Like

merge: ~~https://github.com/NixOS/nixpkgs/pull/72398~~

merge: ~~https://github.com/NixOS/nixpkgs/pull/71507~~

merge: ~~https://github.com/NixOS/nixpkgs/pull/72187~~

merge: smplayer: 19.5.0 -> 19.10.0 by r-ryantm · Pull Request #72660 · NixOS/nixpkgs · GitHub

merge: du-dust: 0.2.3 -> 0.4.1.2 by marsam · Pull Request #72688 · NixOS/nixpkgs · GitHub

1 Like

merge: gnash: fix compilation with libgif 5.2 by marius851000 · Pull Request #73040 · NixOS/nixpkgs · GitHub

merge: ~~https://github.com/NixOS/nixpkgs/pull/73057~~

merge: [r19.03] varnish6: add patch for CVE-2019-15892 by risicle · Pull Request #73078 · NixOS/nixpkgs · GitHub
(Verified other branches for potential updates and validated why we’re applying a vendor patch instead of waiting for a minor version revision: (spoiler - it will never come))

merge zsh-fast-syntax-highlighting:init at 1.54 by pasqui23 · Pull Request #65429 · NixOS/nixpkgs · GitHub

merge: ~~https://github.com/NixOS/nixpkgs/pull/73526~~

merge: nixos/xserver:services.xserver.xkbOptions is now commas by pasqui23 · Pull Request #73394 · NixOS/nixpkgs · GitHub