jiebi/CodeConvo huggingface dataset:
Change:
}; }; <ins> # set this to false, otherwise we'll get a warning indicating that `/etc/klipper.cfg` # is not located in the moonraker config directory. services.moonraker.settings = lib.mkIf (!config.services.klipper.mutableConfig) { file_manager.check_klipper_config_path = false; }; </ins> security.polkit.extraConfig = lib.optionalString cfg.allowSystemControl '' // nixos/moonraker: Allow Moonraker to perform system-level operations //

Discussion:
Moonraker shows warning: Steps to reproduce the behavior: No warnings. ! <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. -- Please run and paste the result.
Annoys me a great deal as well. Trying to figure out, why it even matters.
Well, it's just a no-wont-change, without any reason really. So I guess we either have to patch that part out, or just place the configuration file in the moonraker configuration directory, Which seems rather annoying.
Moonraker provides a script for MainsailOS and FluiddPi which fixes this by . How about we do the same or add a bind mount to the serviceConfig?
That doesn't work, I've tried it out (symlink), and am currently trying to find a solution here . It seems like there will be provided a way to opt out, which would be ideal, or if the symlink part worked then it would also be OK. If symlink worked, then the following could be to the moonraker module For now I'm just patching the moonraker package using the following (not recommended, hopefully moonraker will be updated soon-ish).
I've opened two PRs. One that bumps the moonraker package to include a commit that has the option , which can be set false. And another one, that sets this to false, if NixOS manages the klipper configuration.
Oh, nice! Thank you! :) I... briefly spoke to them about this myself, and they, for some bizarre reason, just 'recommended' I use Debian, and seemed totally disinterested in fixing the problem. I'm glad you got something to work! :D
I think they mostly opposed that it was formulated as NixOS is using "hacks" to run their software (which I understand a maintainer does not want to deal with), however, this warning has nothing to do with anything NixOS related as such.

:) - But yeah, initially they seemed disinterested in discussing any kind of work-around, but I'm happy that we came to an agreement with the maintainer, without (hopefully), being too much of a inconvenience :) If anyone wants to run have a temp override, then it's available here
I guess the word 'hacks' has different connotation for me than it does for some devs. I'll have to watch that in the future! :P
Change:
<ins> { picom, lib, fetchFromGitHub, pcre }: picom.overrideAttrs (previousAttrs: { pname = "picom-pijulius"; version = "8.2-unstable-2024-04-30"; src = fetchFromGitHub { owner = "pijulius"; repo = "picom"; rev = "e7b14886ae644aaa657383f7c4f44be7797fd5f6"; hash = "sha256-YQVp5HicO+jbvCYSY+hjDTnXCU6aS3aCvbux6NFcJ/Y="; }; buildInputs = (previousAttrs.buildInputs or [ ]) ++ [ pcre ]; meta = { inherit (previousAttrs.meta) license platforms mainProgram longDescription ; description = "Pijulius's picom fork with extensive animation support"; homepage = "https://github.com/pijulius/picom"; maintainers = with lib.maintainers; [ YvesStraten ]; }; }) </ins>

Discussion:
Project Description An actively maintained picom fork that adds extended animation support. Metadata homepage URL: source URL: license: mit, mpl-2.0 platforms: linux Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
One minute, I need to push the right branch
has been done, please react onto the pr so that it is marked as being more important. Have a very nice day
Thanks a lot! Added the reaction to the PR too.
Is this working? I have changed my picom configuration to use this package but none of the new options work.
It should be working, at least it does in my configuration, which includes the package before I ported it to nixpkgs. If you could state what problem you are having, that would be nice. Unfortunately I won't have much of a way to help you because the IC of my Linux laptop is fried, meaning that currently I have no way of being on Linux right now to test anything.
Change:
, glib, libsndfile, zlib, bzip2, xz, libsamplerate, intltool , pcre, util-linux, libselinux, libsepol }: <del> stdenv.mkDerivation { </del> <ins> let </ins> inherit (callPackage ./common-drv-attrs.nix { <del> version = "3.2.7"; </del> <ins> version = "3.2.9"; </ins> pname = "libmirage"; <del> hash = "sha256-+okkgNeVS8yoKSrQDy4It7PiPlTSiOsUoFxQ1FS9s9M="; </del> <ins> hash = "sha256-JBd+wHSZRyRW1SZsaAaRO2dNUFkpwRCr3s1f39KyWIs="; </ins> }) pname version src meta; <ins> in stdenv.mkDerivation { inherit pname version src; </ins> PKG_CONFIG_GOBJECT_INTROSPECTION_1_0_GIRDIR = "${placeholder "out"}/share/gir-1.0"; PKG_CONFIG_GOBJECT_INTROSPECTION_1_0_TYPELIBDIR = "${placeholder "out"}/lib/girepository-1.0";

Discussion:
Steps to reproduce the behavior: cdemu/vhba on latest nixos-unstable Looks like there's a new vhba module version: <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
broken by
Change:
inherit (darwin.stubs) rez setfile; }; <del> qemu-riscv = lowPrio (callPackage ../applications/virtualization/qemu/riscv.nix {}); </del> qgis = callPackage ../applications/gis/qgis { inherit (darwin.apple_sdk.frameworks) IOKit ApplicationServices; };

Discussion:
Qemu 2.12.0 has released today. We need to update package. Please run and paste the results.
Change:
antlr4, pyyaml, setuptools, <del> six, </del> urwid, parameterized, wcwidth,

Discussion:
python-fx fails at runtime, while doing its initial imports. Steps to reproduce the behavior: (hash is of as-of-writing nixpkgs-unstable) pyfx tui opens. Doesn't happen when e.g. installing current python-fx with pip on arch linux. (I considered packaging instead, but only found more breakage there. Anyway, I wouldn't be terribly sad if the answer to this is "Package abandonned, will be removed".) Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
I have not used the TUI as so far only served as dependency for . Looks like
I poked around and I don't think that's the same issue at all. (It's the same cause, an old generated file is lying around, but it's a different old generated file.) The prePatch is pretty fishy, it first removes the antlr-related files and then invokes antlr. Fixing that runs into a few type errors, which can be fixed: Now you'd think those type errors are due to the new antler version, but that's only partially true? Downgrading still requires some patching. If I have time I'll submit a PR fixing this, but just patching out the type checks from the feels wrong, so I might consult with upstream first.
Change:
# Hardcode path to fix use with cmake sed -i cmake/scripts/ROOTConfig.cmake.in  <del> -e 'iset(nlohmann_json_DIR "${nlohmann_json}/lib/cmake/nlohmann_json/")' </del> <ins> -e '1iset(nlohmann_json_DIR "${nlohmann_json}/lib/cmake/nlohmann_json/")' </ins> patchShebangs build/unix/ '' + lib.optionalString noSplash ''

Discussion:
In the build result of package , is full of It seems that all the lines involving has been mistakenly replaced with this one. This is the probable cause of the symbol-not-found error when trying to run the tests (). Steps to reproduce the behavior: Build the package against nixos-unstable View See the repeated lines. CMake variables other than nlohmannjsonDIR to appear so that dependent CMake-based packages and tests can build. ! <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. -- Please run and paste the result.
Change:
stdenv.mkDerivation rec { pname = "snowsql"; majorVersion = "1.2"; <del> version = "${majorVersion}.9"; </del> <ins> version = "${majorVersion}.21"; </ins> src = fetchurl { url = "https://sfc-repo.snowflakecomputing.com/snowsql/bootstrap/${majorVersion}/linux_x86_64/snowflake-snowsql-${version}-1.x86_64.rpm"; <del> sha256 = "1k9dyr4vyqivpg054kbvs0jdwhbqbmlp9lsyxgazdsviw8ch70c8"; </del> <ins> sha256 = "10pzxqk71amvaz951jl666ia6v0z22bgpvr108l3q62950hhhwmn"; </ins> }; nativeBuildInputs = [ rpmextract makeWrapper ];

Discussion:
<!-- Note that these are hard requirements --<!-- You can use the "Go to file" functionality on GitHub to find the package Then you can go to the history for this package Find the latest "packagename: oldversion -newversion" commit The "newversion" is the current version of the package --- <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --- name: <!-- The current version can be found easily with the same process as above for checking the master branch If an open PR is present for the package, take this version as the current one and link to the PR --current version: desired version: <!-- Search your package here: If no maintainer is listed for your package, tag the person that last updated the package --maintainers: Please tag this issue in your PR.
Change:
dmtx-utils = callPackage (callPackage ../tools/graphics/dmtx-utils) { }; <del> inherit (callPackage ../applications/virtualization/docker { }) </del> <ins> # go 1.9 pin until https://github.com/moby/moby/pull/35739 inherit (callPackage ../applications/virtualization/docker { go = go_1_9; }) </ins> docker_17_12 docker_18_02;

Discussion:
nixos/tests for docker-tools and kubernetes fail to build on current master It looks like this is caused by being broken under go 1.10. On current master (with go 1.10): Fails with: Similar error for kubernetes test: system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
Here's an example of what I think is wrong with Given a tar file on stdin, it should print a checksum on stdout but results in an error:
Any idea what the Discard is doing? If I comment it out, it builds, but I'm not sure the hashes produced are correct. if _, err = io.Copy(ioutil.Discard, ts); err != nil { fmt.Println(err) os.Exit(1) }
might know.
Oh, yeah, getting rid of that is definitely nonsense. Every hash will be the same. :)
: makes sense, let docker upstream take care of this.
Change:
oci8 = callPackage ../development/php-packages/oci8 { }; <ins> pdlib = callPackage ../development/php-packages/pdlib { }; </ins> pcov = callPackage ../development/php-packages/pcov { }; pcs = buildPecl {

Discussion:
Project description PDlib - A PHP extension for Dlib Metadata homepage URL: source URL: license: mit platforms: all
cc
I marked this as stale due to inactivity. &rarr;
Change:
platforms = []; source = { remotes = ["https://rubygems.org"]; <del> sha256 = "0rsgvdyh828k6bz95b4391ydv6710d4dsdrrfr684glks7599ldf"; </del> <ins> sha256 = "1jbqw7hkx08i5dj74fwfdc16980hsj3mhrxp4mmjxsdzw1kndrvp"; </ins> type = "gem"; }; <del> version = "3.2.3"; </del> <ins> version = "3.2.4"; </ins> }; liquid = { groups = ["default"];

Discussion:
Project description Metadata license: MIT platforms: Any
I guess the issue can be closed due to . Reopen if I am incorrect
Oh, weird. The " in wasn't applied automatically. Thank you!
Change:
<del> { lib , copyDesktopItems , fetchFromGitHub , makeDesktopItem , python3 , wrapQtAppsHook }: python3.pkgs.buildPythonApplication rec { pname = "onthespot"; version = "0.5"; format = "pyproject"; src = fetchFromGitHub { owner = "casualsnek"; repo = "onthespot"; rev = "refs/tags/v${version}"; hash = "sha256-VaJBNsT7uNOGY43GnzhUqDQNiPoFZcc2UaIfOKgkufg="; }; nativeBuildInputs = with python3.pkgs; [ copyDesktopItems pythonRelaxDepsHook wrapQtAppsHook ]; propagatedBuildInputs = with python3.pkgs; [ charset-normalizer defusedxml librespot music-tag packaging pillow protobuf pyogg pyqt5 pyqt5-sip pyxdg requests setuptools show-in-file-manager urllib3 zeroconf ]; pythonRemoveDeps = [ "PyQt5-Qt5" "PyQt5-stubs" ]; pythonRelaxDeps = true; preFixup = '' makeWrapperArgs+=("''${qtWrapperArgs[@]}") ''; meta = with lib; { description = " QT based Spotify music downloader written in Python"; homepage = "https://github.com/casualsnek/onthespot"; changelog = "https://github.com/casualsnek/onthespot/releases/tag/v${version}"; license = licenses.gpl2Only; maintainers = with maintainers; [ onny ]; platforms = platforms.linux; }; } </del>

Discussion:
Steps to reproduce the behavior: onthespot Add any other context about the problem here. <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
meta = with stdenv.lib; { description = "facetimehd firmware"; <del> homepage = https://support.apple.com/downloads/DL1877; </del> <ins> homepage = https://support.apple.com/kb/DL1877; </ins> license = licenses.unfree; maintainers = with maintainers; [ womfoo grahamc ]; platforms = [ "i686-linux" "x86_64-linux" ];

Discussion:
I don't really know why my system configuration for an XPS 13 is pulling this in, either: EDIT: I had set in my system configuration, thus causing me to (fail to) build .
I have the same problem on 3 laptops As mentioned in above post issue is related to enableAllFirmware = true; Chnaging to enableRedistributableFirmware = true; fixed the issue for now another example
cc maintainers:
I think, the new url could be found on this page
I'll try to find time over the weekend to update new offsets and hashes for the new download path. In the meantime, facetimehd users can extract the firmware from their OSX partition with this override: ``
See
Change:
stdenv.mkDerivation rec { pname = "gnome-shell-extension-pop-shell"; <del> version = "unstable-2023-04-27"; </del> <ins> version = "unstable-2023-11-10"; </ins> src = fetchFromGitHub { owner = "pop-os"; repo = "shell"; <del> rev = "b5acccefcaa653791d25f70a22c0e04f1858d96e"; hash = "sha256-w6EBHKWJ4L3ZRVmFqZhCqHGumbElQXk9udYSnwjIl6c="; </del> <ins> rev = "aafc9458a47a68c396933c637de00421f5198a2a"; hash = "sha256-74lZbEYHj7fufRSbuI2SN9rqbB3gpRa0V96qjAFc01s="; </ins> }; nativeBuildInputs = [ glib gjs typescript ];

Discussion:
Package name: gnome-shell-extension-pop-shell Latest released version: unstable-2023-11-10 <!-- Search your package here: --- Current version on the unstable channel: unstable-2023-04-27 Current version on the stable/release channel: unstable-2023-04-27 <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --This brings about Gnome 45 support, as far as I know. Notify maintainers <!-- If the result shows no maintainers, tag the person that last updated the package. --Note for maintainers: Please tag this issue in your PR. Priorities Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
thanks for alle the involved to fix it so fast
Change:
serviceConfig = { ExecStart = "${cfg.package}/bin/ntfy serve -c ${configuration}"; User = cfg.user; <ins> StateDirectory = "ntfy-sh"; </ins> AmbientCapabilities = "CAP_NET_BIND_SERVICE"; PrivateTmp = true;

Discussion:
The ntfy-sh nixos module doesn't include the creation of the auth file which is necessary to create users and restrict access to ntfy topics. Currently, doing it by hand involves creating a data folder (the folder /var/lib/ntfy-sh is not currently created by the module) and setting the option `. As in most cases a user is going to want to create users for their instance, the module should handle this use case.
Change:
meme-suite = callPackage ../applications/science/biology/meme-suite { }; <del> # Needs qtwebkit which is broken on qt5.15 mendeley = libsForQt514.callPackage ../applications/office/mendeley { </del> <ins> mendeley = libsForQt5.callPackage ../applications/office/mendeley { </ins> gconf = gnome2.GConf; };

Discussion:
libsForQt514.qt5.qtwebengine currently fails to build locally and on . The interessting line seems to be: The last successful build on hydra was linked to . After that it seems to fail (starting with ) Bisecting takes forever. I would hazard a guess that it has to do with and PR which broke some other packages as well. EDIT: EDIT2: The error seems to stem from the c++ upgrade. The error message is referenced e.g. here Steps to reproduce the behavior: nixpkgs master -build -A libsForQt514.qt5.qtwebengine . Build should succeed. If applicable, add screenshots to help explain your problem. Add any other context about the problem here. Please run and paste the result.
It's most likely due to mixing gcc versions, as that's not 100% compatible in C++. qt514 (and qt512) get overridden to gcc10, whereas the default is gcc11. (For x86_64-linux; some others have other combinations.)
Closing since libsForQt514 was removed from master.
Change:
# a newer libxml2 version has changed some interfaces ./fix-xml2.patch <ins> # Don't try to override the ANTLR_JAR_PATH specified in cmakeFlags ./dont-search-for-antlr-jar.patch </ins> ]; <del> # 1. have it look for 4.12.0 instead of 4.11.1 # 2. for some reason CMakeCache.txt is part of source code preConfigure = '' substituteInPlace CMakeLists.txt  --replace "antlr-4.11.1-complete.jar" "antlr-4.12.0-complete.jar" </del> <ins> postPatch = '' # For some reason CMakeCache.txt is part of source code, remove it </ins> rm -f build/CMakeCache.txt <ins> patchShebangs tools/get_wb_version.sh </ins> ''; nativeBuildInputs = [

Discussion:
Steps to reproduce the behavior: mysql-workbench-8.0.36 Add any other context about the problem here. <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Problem introduced in Thanks for reporting, I'll try to fix it
Change:
fi popd &>/dev/null ''; <ins> meta = { license = lib.licenses.${license}; longDescription = description; inherit homepage; } // lib.optionalAttrs (description != null) { longDescription = description; } // lib.optionalAttrs (homepage != null) { inherit homepage; }; </ins> }; <del> } </del> <ins> } // lib.optionalAttrs (appName != null && appVersion != null) { name = "nextcloud-app-${appName}-${appVersion}"; }) </ins>

Discussion:
The nextcloud package set offers apps, that have licensing information attached, but not exposed as part of their meta attribute. Also, there are some apps, like news, where the licensing information says , which requires clarification. This needs to be fixed.
Thank you for opening the draft PR in order to fix my mistakes and broken packages. I'm sorry for the bad communication and the inability to resolve this serious issue. Fixing this for Wordpress packages and themes (wp4nix) and Nextcloud apps (nc4nix) in a foreseeable time exceeds my skills and competence. Therefore I would like to offer to add a flag to these package sets and remove them for the next release 23.11 if it's not resolved by then.
Change:
buildInputs = [ alsa-lib libjack2 liblo qtbase ]; postPatch = '' <del> for d in libseq66/include libseq66/src libsessions/include libsessions/src seq_qt5/src seq_rtmidi/include seq_rtmidi/src Seqtool/src; do substituteInPlace "$d/Makefile.am" --replace '$(git_info)' '${version}' </del> <ins> for d in libseq66/src libsessions/include libsessions/src seq_qt5/src seq_rtmidi/src; do substituteInPlace "$d/Makefile.am" --replace-fail '$(git_info)' '${version}' </ins> done <del> # gcc-13 headers compatibilty. TODO: try to remove with next version # update sed -e '1i #include <cstdint>' -i libseq66/src/os/daemonize.cpp </del> ''; enableParallelBuilding = true;

Discussion:
Package name: seq66 Latest released version: 0.90.10 Current version on the unstable channel: 0.90.5 Current version on the stable/release channel: 0.90.5 Notify maintainers Note for maintainers: Please tag this issue in your PR. Priorities Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
, regex , toml , typed-ast <del> , typing-extensions }: </del> <ins> , typing-extensions , uvloop }: </ins> buildPythonPackage rec { pname = "black"; <del> version = "21.5b1"; </del> <ins> version = "21.6b0"; </ins> disabled = pythonOlder "3.6"; src = fetchPypi { inherit pname version; <del> sha256 = "1cdkrl5vw26iy7s23v2zpr39m6g5xsgxhfhagzzflgfbvdc56s93"; </del> <ins> sha256 = "016f6bhnnnbcrrh3cvmpk77ww0nykv5n1qvgf8b3044dm14264yw"; </ins> }; nativeBuildInputs = [ setuptools-scm ];

Discussion:
Describe the bug Fails during tests on unstable channel: Seems to be caused by To Reproduce Steps to reproduce the behavior: Expected behavior Build succeeds. Notify maintainers Metadata Maintainer information:
May be hitting this:
Simply adding makes the test pass and the build succeed.
Change:
example = literalExample "[ pkgs.splix ]"; description = '' CUPS drivers to use. Drivers provided by CUPS, cups-filters, Ghostscript <del> and Samba are added unconditionally. </del> <ins> and Samba are added unconditionally. For adding Gutenprint, see <literal>gutenprint</literal>. </ins> ''; };

Discussion:
Current way to install new printers () is not really the thing user will figure out without a manual, and I think we might try to do better. Maybe add a list of installed PPDs to services.printing or something like that, but I don't really know how CUPS finds or stores PPDs, so I don't know if this really can be made better.
There are two things that tripped me: obvious way how to use gutenprint in the docs (for someone not familiar with how cups works) the helped the naïve way (installing gutenprint as package) doesn’t work and it doesn’t tell you why generating the ppds is necessary. If possible, this should be done automatically while realizing the derivation Does it take too much storage?
When I my printer again recently, the gutenprint ppds where directly accessible from the interface, so all one has to do now is add gutenprint to . I just noticed there is now. Huzzah!
Oh, I fixed that by myself recently and forgot. Let's close!
Change:
# Include first args to make sure we're patching the right thing substituteInPlace shell/key-bindings.bash  --replace " perl -n " " ${ourPerl}/bin/perl -n " <ins> # fzf-tmux depends on bc substituteInPlace bin/fzf-tmux  --replace "bc" "${bc}/bin/bc" </ins> ''; postInstall = ''

Discussion:
Installing and using fzf breaks with the following error message: Steps to reproduce the behavior: fzf with tmux integration, I did it via home manager modules fzf with tmux integration within tmux The tmux pop up appears with history to filter If applicable, add screenshots to help explain your problem. Add any other context about the problem here. Please run and paste the result. Looks like this got introduced with The runtime dependency was with that new version: Line 183 of file I wasn't able to find a proper URL to link to that line within the diff.
Workaround: Install as another package.
due to providing the update to 0.39.0 at
Should be fixed by , would be helpful if you could test it.
Thanks I'm pretty new, is there any doc how I could test this on my machine?
If you're using Nix channels then you can replace your channel with and rebuild, which will build all your packages from my PR branch, including . If you're using Nix Flakes then you can change the nixpkgs input to be which has the same effect.