jiebi/CodeConvo huggingface dataset:
Code change:
Package name: ffmpeg-normalize Latest released version: 1.28.3 <!-- Search your package here: --- Current version on the unstable channel: 1.26.1 Current version on the stable/release channel: 1.26.1 <!-- 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! --- 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. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: <del> { lib , buildPythonApplication , fetchPypi , ffmpeg , ffmpeg-progress-yield }: buildPythonApplication rec { pname = "ffmpeg-normalize"; version = "1.26.1"; src = fetchPypi { inherit pname version; hash = "sha256-OwREpfWaP0tdAjMGjGpVIAQn8rlTTjSfT+0t5g/2yjQ="; }; propagatedBuildInputs = [ ffmpeg ffmpeg-progress-yield ]; checkPhase = '' $out/bin/ffmpeg-normalize --help > /dev/null ''; meta = with lib; { description = "Normalize audio via ffmpeg"; homepage = "https://github.com/slhck/ffmpeg-normalize"; license = with licenses; [ mit ]; maintainers = with maintainers; [ prusnak ]; mainProgram = "ffmpeg-normalize"; }; } </del>
Code change:
Package name: ffmpeg-normalize Latest released version: 1.28.3 <!-- Search your package here: --- Current version on the unstable channel: 1.26.1 Current version on the stable/release channel: 1.26.1 <!-- 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! --- 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. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: <ins> { lib, python3Packages, fetchPypi, ffmpeg, }: python3Packages.buildPythonApplication rec { pname = "ffmpeg-normalize"; version = "1.28.3"; src = fetchPypi { inherit pname version; hash = "sha256-8wNPuVRQRQpFK6opgwqdKYMYmAFRqq8p/T5V9kC8QaY="; }; propagatedBuildInputs = [ ffmpeg python3Packages.ffmpeg-progress-yield ]; dependencies = with python3Packages; [ colorlog ]; checkPhase = '' $out/bin/ffmpeg-normalize --help > /dev/null ''; meta = { description = "Normalize audio via ffmpeg"; homepage = "https://github.com/slhck/ffmpeg-normalize"; license = lib.licenses.mit; maintainers = with lib.maintainers; [ luftmensch-luftmensch prusnak ]; mainProgram = "ffmpeg-normalize"; }; } </ins>
Code change:
Package name: ffmpeg-normalize Latest released version: 1.28.3 <!-- Search your package here: --- Current version on the unstable channel: 1.26.1 Current version on the stable/release channel: 1.26.1 <!-- 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! --- 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. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: ffmpegthumbnailer = callPackage ../development/libraries/ffmpegthumbnailer { }; <del> ffmpeg-normalize = python3Packages.callPackage ../applications/video/ffmpeg-normalize { }; </del> ffms = callPackage ../development/libraries/ffms { }; fftw = callPackage ../development/libraries/fftw { };
Code change:
Steps to reproduce the behavior: dvc another dvc hash error, i dont know why <!-- 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]:

Related discussion:
text: buildPythonPackage rec { pname = "dvc"; <del> version = "3.55.0"; </del> <ins> version = "3.55.1"; </ins> pyproject = true; disabled = pythonOlder "3.8";
Code change:
Steps to reproduce the behavior: dvc another dvc hash error, i dont know why <!-- 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]:

Related discussion:
text: owner = "iterative"; repo = "dvc"; rev = "refs/tags/${version}"; <del> hash = "sha256-nKzMHVMaoDu/d1wMAxA2q8DHdwoIb7+YG/7AhAtRVEM="; </del> <ins> hash = "sha256-DrhjmVHCX1lXJUrklVgO/eguL1uRQ33kJjSgCAEIyew="; </ins> }; pythonRelaxDeps = [
Code change:
First failing build (4.9.76, but not always): Latest (4.9.77): Looking through the changes for any obvious culprits...
bisected to (as the first "bad commit"). cc
Reverting the commit on master fixes the build, but need to replace with to at least fix evaluation on cross. Since doesn't seem to build on cross (at least not on any architectures I've tried) I can't tell if this "works", but would be good to fix this on non-cross regardless.
See corresponding PR, this has been handled. Thanks!

Related discussion:
text: stdenv.mkDerivation { name = "perf-linux-${kernel.version}"; <del> inherit (kernel) src makeFlags; </del> <ins> inherit (kernel) src; </ins> preConfigure = '' cd tools/perf
Code change:
First failing build (4.9.76, but not always): Latest (4.9.77): Looking through the changes for any obvious culprits...
bisected to (as the first "bad commit"). cc
Reverting the commit on master fixes the build, but need to replace with to at least fix evaluation on cross. Since doesn't seem to build on cross (at least not on any architectures I've tried) I can't tell if this "works", but would be good to fix this on non-cross regardless.
See corresponding PR, this has been handled. Thanks!

Related discussion:
text: "-Wno-error=unused-const-variable" "-Wno-error=misleading-indentation" ]; <ins> makeFlags = if stdenv.hostPlatform == stdenv.buildPlatform then null else "CROSS_COMPILE=${stdenv.cc.targetPrefix}"; </ins> installFlags = "install install-man ASCIIDOC8=1"; preFixup = ''
Code change:
Project description <!-- Describe the project a little: --Syntax Tree Surfer is a plugin for Neovim that helps you surf through your document and move elements around using the nvim-treesitter API. Metadata homepage URL: source URL: license: mit platforms: unix, linux, darwin, ... Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: meta.homepage = "https://github.com/vim-syntastic/syntastic/"; }; <ins> syntax-tree-surfer = buildVimPlugin { pname = "syntax-tree-surfer"; version = "2023-10-06"; src = fetchFromGitHub { owner = "ziontee113"; repo = "syntax-tree-surfer"; rev = "732ea6d0f868bcccd2f526be73afa46997d5a2fb"; sha256 = "0x7kpmxq3n8531ca2bz506swww92wd3x16d125wrk54rxp1lv1cd"; }; meta.homepage = "https://github.com/ziontee113/syntax-tree-surfer/"; }; </ins> tabby-nvim = buildVimPlugin { pname = "tabby.nvim"; version = "2024-07-31";
Code change:
Project description <!-- Describe the project a little: --Syntax Tree Surfer is a plugin for Neovim that helps you surf through your document and move elements around using the nvim-treesitter API. Metadata homepage URL: source URL: license: mit platforms: unix, linux, darwin, ... Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: }; }; <ins> syntax-tree-surfer = super.syntax-tree-surfer.overrideAttrs { dependencies = with self; [ nvim-treesitter ]; meta.maintainers = with lib.maintainers; [ callumio ]; }; </ins> taskwarrior3 = buildVimPlugin { inherit (taskwarrior3) version pname; src = "${taskwarrior3.src}/scripts/vim";
Code change:
Project description <!-- Describe the project a little: --Syntax Tree Surfer is a plugin for Neovim that helps you surf through your document and move elements around using the nvim-treesitter API. Metadata homepage URL: source URL: license: mit platforms: unix, linux, darwin, ... Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: https://github.com/AndrewRadev/switch.vim/,, https://github.com/simrat39/symbols-outline.nvim/,, https://github.com/vim-syntastic/syntastic/,, <ins> https://github.com/ziontee113/syntax-tree-surfer/,HEAD, </ins> https://github.com/nanozuki/tabby.nvim/,HEAD, https://github.com/kdheepak/tabline.nvim/,, https://github.com/vim-scripts/tabmerge/,,
Code change:
is currently using this command creates the temporary directory inside the current working directory instead of in . This can cause the generated directory to be included in the final package, as I'm assuming accidentally picked it up. I encountered this while working on and The temporary directory is not included in the output use use (output is of the format ) note: this isn't really used across nixpkgs use something else <!-- 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]:

Related discussion:
text: --offline ) <del> local -r tmpDir="$(mktemp -d yarnInstallHook.XXXXXX)" </del> <ins> local -r tmpDir="$(mktemp -d)" </ins> # yarn pack does not work at all with bundleDependencies. # Since we are imediately unpacking, we can just remove them from package.json
Code change:
The program described by this man page is not available in master. ag ipvsadm
Poked at this, looks like it needs which because it was old and krufty even back then (note the patches required). That seems like a bit of a show-stopper, unfortunately.
Thanks! Is there a replacement for that I am missing? is still a part of the Linux kernel.
Enjoy! LMK if you're able to actual test it or know how to employ the tools :+1:.
I plan to do that tomorrow. I hope the tools will tell me the crucial information I need to make it work.
This works great and should be merged. It told me something unexpected, but that's good.

Related discussion:
text: <ins> { stdenv, fetchurl, pkgconfig, libnl, popt, gnugrep }: stdenv.mkDerivation rec { name = "ipvsadm-${version}"; version = "1.29"; src = fetchurl { url = "mirror://kernel/linux/utils/kernel/ipvsadm/${name}.tar.xz"; sha256 = "c3de4a21d90a02c621f0c72ee36a7aa27374b6f29fd4178f33fbf71b4c66c149"; }; postPatch = '' substituteInPlace Makefile --replace "-lnl" "$(pkg-config --libs libnl-genl-3.0)" ''; nativeBuildInputs = [ pkgconfig ]; buildInputs = [ libnl popt ]; preBuild = '' makeFlagsArray+=( INCLUDE=$(pkg-config --cflags libnl-genl-3.0) BUILD_ROOT=$out MANDIR=share/man ) ''; postInstall = '' sed -i -e "s|^PATH=.*|PATH=$out/bin:${gnugrep}/bin|" $out/sbin/ipvsadm-{restore,save} ''; meta = with stdenv.lib; { description = "Linux Virtual Server support programs"; homepage = http://www.linuxvirtualserver.org/software/ipvs.html; license = licenses.gpl2; platforms = platforms.linux; }; } </ins>
Code change:
The program described by this man page is not available in master. ag ipvsadm
Poked at this, looks like it needs which because it was old and krufty even back then (note the patches required). That seems like a bit of a show-stopper, unfortunately.
Thanks! Is there a replacement for that I am missing? is still a part of the Linux kernel.
Enjoy! LMK if you're able to actual test it or know how to employ the tools :+1:.
I plan to do that tomorrow. I hope the tools will tell me the crucial information I need to make it work.
This works great and should be merged. It told me something unexpected, but that's good.

Related discussion:
text: iio-sensor-proxy = callPackage ../os-specific/linux/iio-sensor-proxy { }; <ins> ipvsadm = callPackage ../os-specific/linux/ipvsadm { }; </ins> lynis = callPackage ../tools/security/lynis { }; mathics = pythonPackages.mathics;
Code change:
Package name: libssh Latest released version: 0.11.1 <!-- Search your package here: --- Current version on the unstable channel: 0.10.6 Current version on the stable/release channel: 0.10.6 <!-- 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! --- Notify maintainers Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: stdenv.mkDerivation rec { pname = "libssh"; <del> version = "0.10.6"; </del> <ins> version = "0.11.1"; </ins> src = fetchurl { url = "https://www.libssh.org/files/${lib.versions.majorMinor version}/libssh-${version}.tar.xz"; <del> hash = "sha256-GGHUmPW28XQbarxz5ghHhJHtz5ydS2Yw7vbnRZbencE="; </del> <ins> hash = "sha256-FLfcxy6R4IFRxYuYGntXCrJmP2MOfSg3ZF1anGEsG3k="; </ins> }; <del> # Do not split 'dev' output until lib/cmake/libssh/libssh-config.cmake # is fixed to point INTERFACE_INCLUDE_DIRECTORIES to .dev output. # Otherwise it breaks `plasma5Packages.kio-extras`: #   https://hydra.nixos.org/build/221540008/nixlog/3/tail #outputs = [ "out" "dev" ]; </del> <ins> outputs = [ "out" "dev" ]; </ins> postPatch = '' # Fix headers to use libsodium instead of NaCl
Code change:
Package name: libssh Latest released version: 0.11.1 <!-- Search your package here: --- Current version on the unstable channel: 0.10.6 Current version on the stable/release channel: 0.10.6 <!-- 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! --- Notify maintainers Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: # included in `buildInputs` such as libX11. cmakeFlags = [ "-DWITH_EXAMPLES=OFF" ]; <del> # single output, otherwise cmake and .pc files point to the wrong directory # outputs = [ "out" "dev" ]; </del> buildInputs = [ zlib openssl libsodium ]; nativeBuildInputs = [ cmake pkg-config ]; <ins> postFixup = '' substituteInPlace $dev/lib/cmake/libssh/libssh-config.cmake  --replace-fail "set(_IMPORT_PREFIX "$out")" "set(_IMPORT_PREFIX "$dev")" ''; </ins> passthru.tests = { inherit ffmpeg sshping wireshark; };
Code change:
Steps to reproduce the behavior: -rebuild test on settings.datadir = "/mnt/garage0"; resolves fine Tested on nixos24.05 but code of the module is unchanged in nixos_unstable. I'm using flakes as well Please run and paste the result.
Looks like doesn't apply, maybe should tinker a bit on my own
It looks like the option type was adjusted, but not the other code in the module that checks that value. Shouldn't be too hard to fix.

Related discussion:
text: <del> { config, lib, pkgs, ... }: </del> <ins> { config, lib, pkgs, ... }: </ins> with lib;
Code change:
Steps to reproduce the behavior: -rebuild test on settings.datadir = "/mnt/garage0"; resolves fine Tested on nixos24.05 but code of the module is unchanged in nixos_unstable. I'm using flakes as well Please run and paste the result.
Looks like doesn't apply, maybe should tinker a bit on my own
It looks like the option type was adjusted, but not the other code in the module that checks that value. Shouldn't be too hard to fix.

Related discussion:
text: cfg = config.services.garage; toml = pkgs.formats.toml { }; configFile = toml.generate "garage.toml" cfg.settings; <ins> anyHasPrefix = prefix: strOrList: if isString strOrList then hasPrefix prefix strOrList else any ({ path, ... }: hasPrefix prefix path) strOrList; </ins> in { meta = { doc = ./garage.md; <del> maintainers = [ ]; </del> <ins> maintainers = [ maintainers.mjm ]; </ins> }; options.services.garage = {
Code change:
Steps to reproduce the behavior: -rebuild test on settings.datadir = "/mnt/garage0"; resolves fine Tested on nixos24.05 but code of the module is unchanged in nixos_unstable. I'm using flakes as well Please run and paste the result.
Looks like doesn't apply, maybe should tinker a bit on my own
It looks like the option type was adjusted, but not the other code in the module that checks that value. Shouldn't be too hard to fix.

Related discussion:
text: type = types.attrsOf types.str; description = "Extra environment variables to pass to the Garage server."; default = { }; <del> example = { RUST_BACKTRACE = "yes"; }; </del> <ins> example = { RUST_BACKTRACE = "yes"; }; </ins> }; environmentFile = mkOption {
Code change:
Steps to reproduce the behavior: -rebuild test on settings.datadir = "/mnt/garage0"; resolves fine Tested on nixos24.05 but code of the module is unchanged in nixos_unstable. I'm using flakes as well Please run and paste the result.
Looks like doesn't apply, maybe should tinker a bit on my own
It looks like the option type was adjusted, but not the other code in the module that checks that value. Shouldn't be too hard to fix.

Related discussion:
text: }; logLevel = mkOption { <del> type = types.enum ([ "error" "warn" "info" "debug" "trace" ]); </del> <ins> type = types.enum ([ "error" "warn" "info" "debug" "trace" ]); </ins> default = "info"; example = "debug"; description = "Garage log level, see <https://garagehq.deuxfleurs.fr/documentation/quick-start/#launching-the-garage-server> for examples.";
Code change:
Steps to reproduce the behavior: -rebuild test on settings.datadir = "/mnt/garage0"; resolves fine Tested on nixos24.05 but code of the module is unchanged in nixos_unstable. I'm using flakes as well Please run and paste the result.
Looks like doesn't apply, maybe should tinker a bit on my own
It looks like the option type was adjusted, but not the other code in the module that checks that value. Shouldn't be too hard to fix.

Related discussion:
text: data_dir = mkOption { default = "/var/lib/garage/data"; <del> example = [ { path = "/var/lib/garage/data"; capacity = "2T"; } ]; </del> <ins> example = [ { path = "/var/lib/garage/data"; capacity = "2T"; } ]; </ins> type = with types; either path (listOf attrs); description = '' The directory in which Garage will store the data blocks of objects. This folder can be placed on an HDD.
Code change:
Steps to reproduce the behavior: -rebuild test on settings.datadir = "/mnt/garage0"; resolves fine Tested on nixos24.05 but code of the module is unchanged in nixos_unstable. I'm using flakes as well Please run and paste the result.
Looks like doesn't apply, maybe should tinker a bit on my own
It looks like the option type was adjusted, but not the other code in the module that checks that value. Shouldn't be too hard to fix.

Related discussion:
text: # These assertions can be removed in NixOS 24.11, when all users have been # warned once. { <del> assertion = (cfg.settings ? replication_factor || cfg.settings ? replication_mode) || lib.versionOlder cfg.package.version "1.0.0"; </del> <ins> assertion = (cfg.settings ? replication_factor || cfg.settings ? replication_mode) || lib.versionOlder cfg.package.version "1.0.0"; </ins> message = '' Garage 1.0.0 requires an explicit replication factor to be set. Please set replication_factor to 1 explicitly to preserve the previous behavior.
Code change:
Steps to reproduce the behavior: -rebuild test on settings.datadir = "/mnt/garage0"; resolves fine Tested on nixos24.05 but code of the module is unchanged in nixos_unstable. I'm using flakes as well Please run and paste the result.
Looks like doesn't apply, maybe should tinker a bit on my own
It looks like the option type was adjusted, but not the other code in the module that checks that value. Shouldn't be too hard to fix.

Related discussion:
text: systemd.services.garage = { description = "Garage Object Storage (S3 compatible)"; <del> after = [ "network.target" "network-online.target" ]; wants = [ "network.target" "network-online.target" ]; </del> <ins> after = [ "network.target" "network-online.target" ]; wants = [ "network.target" "network-online.target" ]; </ins> wantedBy = [ "multi-user.target" ]; <del> restartTriggers = [ configFile ] ++ (lib.optional (cfg.environmentFile != null) cfg.environmentFile); </del> <ins> restartTriggers = [ configFile ] ++ (lib.optional (cfg.environmentFile != null) cfg.environmentFile); </ins> serviceConfig = { ExecStart = "${cfg.package}/bin/garage server"; <del> StateDirectory = mkIf (hasPrefix "/var/lib/garage" cfg.settings.data_dir || hasPrefix "/var/lib/garage" cfg.settings.metadata_dir) "garage"; </del> <ins> StateDirectory = mkIf ( anyHasPrefix "/var/lib/garage" cfg.settings.data_dir || hasPrefix "/var/lib/garage" cfg.settings.metadata_dir ) "garage"; </ins> DynamicUser = lib.mkDefault true; ProtectHome = true; NoNewPrivileges = true;
Code change:
Steps to reproduce the behavior: try to install/build the package using nix-shell or nixos the Domain is no longer configured. Teamspeak Download Pages redirects to this place: Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
You might want to check out
Not merged yet
As a workaround, until this is fixed in nixpkgs: The file is still available at the Internet Archive. You can fetch it from there and write it into the store, with something like this:

Related discussion:
text: <del> { lib, stdenv, fetchurl, makeWrapper, makeDesktopItem, zlib, glib, libpng, freetype, openssl , xorg, fontconfig, qtbase, qtwebengine, qtwebchannel, qtsvg, qtwebsockets, xkeyboard_config , alsa-lib, libpulseaudio ? null, libredirect, quazip, which, unzip, perl, llvmPackages </del> <ins> { lib, stdenv, fetchurl, fetchzip, makeWrapper, makeDesktopItem, zlib, glib, libpng, freetype, openssl, xorg, fontconfig, qtbase, qtwebengine, qtwebchannel, qtsvg, qtwebsockets, xkeyboard_config, alsa-lib, libpulseaudio ? null, libredirect, quazip, which, perl, llvmPackages, </ins> }: let
Code change:
Steps to reproduce the behavior: try to install/build the package using nix-shell or nixos the Domain is no longer configured. Teamspeak Download Pages redirects to this place: Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
You might want to check out
Not merged yet
As a workaround, until this is fixed in nixpkgs: The file is still available at the Internet Archive. You can fetch it from there and write it into the store, with something like this:

Related discussion:
text: libDir = "lib64"; <del> deps = [ zlib glib libpng freetype xorg.libSM xorg.libICE xorg.libXrender openssl xorg.libXrandr xorg.libXfixes xorg.libXcursor xorg.libXinerama xorg.libxcb fontconfig xorg.libXext xorg.libX11 alsa-lib qtbase qtwebengine qtwebchannel qtsvg qtwebsockets libpulseaudio quazip llvmPackages.libcxx ]; </del> <ins> deps = [ zlib glib libpng freetype xorg.libSM xorg.libICE xorg.libXrender openssl xorg.libXrandr xorg.libXfixes xorg.libXcursor xorg.libXinerama xorg.libxcb fontconfig xorg.libXext xorg.libX11 alsa-lib qtbase qtwebengine qtwebchannel qtsvg qtwebsockets libpulseaudio quazip llvmPackages.libcxx ]; </ins> desktopItem = makeDesktopItem { name = "teamspeak";