(1) The pull request adds “MPRIS” functionality which seems to require X11, which makes spotifyd fail when X11 is not available. Unsure how to handle this but I would like to see spotifyd working by default without X11.
(2) The version bump broke the spotifyd service module for me and I had to manually clear out its cache. I do not know if we should handle this and if so, how.
This PR went unnoticed but it’s a relatively important fix: it should also be backported. I have been running it on my server so I think it’s already good to be merged.
Would be thankful for a quick function test on Darwin/MacOSX so I can tick the last box in my first PR.
If anyone has a Darwin/MacOSX machine and 5 minutes for this, please do as follows:
Checkout the PR
Run at the top of the worktree nix-shell -E 'with import ./default.nix {}; mkShell { buildInputs = [ ammonite fish ]; }'
If there’s no error, run fish in the resulting shell
I have one question there because of the impact on package closure size of +78.35 MB (46.52%): Should gstreamerSupport be false by default (thus the effect of this PR be a non-default option)?
This is relatively simple PR but it touches the core of the python build system and is therefore a mass rebuild. I need some assistance in testing and reviewing.
The issue is I don’t have a darwin machine so I can’t fix the darwin build without putting in unreasonable amounts of effort. Maybe some darwin wizard will help me out.
Here are two similar patches waiting for boost17x that improve the closure size usage of packages which depend on boost17x (such as nheko). If either one will be merged it’d be great:
My PR doesn’t build on aarch64. I got “error: linker aarch64-linux-gnu-gcc not found” and I have no idea if it’s a problem with one of the Rust dependencies or if it’s a problem with the aarch64 builder.
This PR exists to upgrade fontconfig (the current version is now nearly 3 years out of date), and introduces breaking changes to fontconfig’s configuration. Because of this, it’s causing issues for lots of people who use nix on non-NixOS, because both system’s fontconfig and nix’s fontconfig will try to read configuration from /etc. As far as I can tell the PR is fine, and the only blocker seems to be the huge number of reverse-dependencies (i.e. hydra rebuilds). There’s also something about backporting old versions of fontconfig to generate font caches for older versions, but I’m not really sure what’s going on there.
I have a not too big nodePackages fix for an issue which passes all tests possible, but every once in a while it reaches a merge conflict status… And what’s annoying even more is that running node’s generate.sh takes about half a day (for me) and afterwards I have to cherry-pick only the relevant changes to the commit and discard the rest …
This one is docs-only: crucial steps are missing from the manual in the instructions for nixos lustration. Someone else figured out a solution and posted it in an issue back in 2018, all I did was add the solution to the docs. The PR doesn’t seem to have had any attention yet. It could do with someone who is familiar with bootloaders having a look at whether there is a better way of doing it, and maybe suggesting some commentary for the new steps.
This one isn’t terribly important or complicated. A new package for the virtual ANS microtonal synthesizer. It’s been approved and seems to be waiting for a merge. However, as I have no means of testing the darwin version, it’s simply disabled, even though it would probably work. So perhaps someone with a Mac could test it there. https://github.com/NixOS/nixpkgs/pull/90624
I wrote a package for ILSpy, but the reviewer demands building it from source, something I don’t need or have the time to do, so maybe someone else wants to pick it up?