jiebi/CodeConvo huggingface dataset:
Change:
propagatedBuildInputs = [ HeapFibonacci ]; }; <ins> GraphicsTIFF = buildPerlPackage { pname = "Graphics-TIFF"; version = "9"; src = fetchurl { url = "mirror://cpan/authors/id/R/RA/RATCLIFFE/Graphics-TIFF-9.tar.gz"; sha256 = "1n1r9r7f6hp2s6l361pyvb1i1pm9xqy0w9n3z5ygm7j64160kz9a"; }; buildInputs = [ pkgs.libtiff ExtUtilsDepends ExtUtilsPkgConfig ]; propagatedBuildInputs = [ Readonly ]; checkInputs = [ TestRequires TestDeep pkgs.hexdump ]; meta = { description = "Perl extension for the libtiff library"; license = with lib.licenses; [ artistic1 gpl1Plus ]; }; }; </ins> GraphViz = buildPerlPackage { pname = "GraphViz"; version = "2.24";

Discussion:
Describe the bug in seems to be broken, in a way (I think) related to how other things have broken in the past, e.g. . From looking at examples of fixes, e.g. , it looks like it might be necessary to patch the Perl source in , but I don't really understand what's going on so I don't know how. To Reproduce Steps to reproduce the behavior: to for this package so that the tests actually run, as in my PR tests fail Expected behavior I tried making a package that depends on , and got the following error at runtime: Maintainer information:
Thank you for your contributions. This has been automatically marked as stale because it has had no activity for 180 days. If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity. Here are suggestions that might help resolve this more quickly: for maintainers and people that previously touched the related code and @ mention them in a comment. on the . 3. Ask on the on .
Change:
. ./external.sh rm external.sh ''; <del> hash = "sha256-0RlI30zxiueeXdLEXPZevc8QyKr667juHk0bTcqBB1w="; </del> <ins> hash = "sha256-dXfpu+ufBfAWl1OlpQ1i8CC7N8f0NbxfaMH6BdKr28c="; </ins> }; in ((buildMozillaMach { pname = "betterbird";

Discussion:
Steps to reproduce the behavior: betterbird Please run and paste the result.
I don't see any error, so likely your machine is just to weak. You can try reducing cores/max-jobs for nix.
I think job on hydra also failed, which cause a rebuild on my laptop. See Not sure if I'm making any mistake here?
There the error is: error[E0635]: unknown feature which means we need to update betterbird to be hopefully based on a newer thunderbird version which supports a newer rust version

Package name: betterbird Latest released version: 115.12 <!-- Search your package here: --- Current version on the unstable channel: 115.9 Current version on the stable/release channel: 115.9 Notify maintainers Note for maintainers: Please tag this issue in your PR. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Could you create a PR, please?
I can try but I don't understand what file is the hash of line 39 from?
You just delete it (let it be ) and run the build, it will fail with the right hash. Before submitting PRs you may also read
Thank you. I opened it here :
Change:
<del> { stdenv, fetchurl, unzip, autoreconfHook }: stdenv.mkDerivation rec { name = "libpo6-${version}"; version = "0.5.2"; src = fetchurl { url = "https://github.com/rescrv/po6/archive/releases/${version}.zip"; sha256 = "17grzkh6aw1f68qvkhivbb6vwbm6jd41ysbfn88pypf5lczxrxly"; }; buildInputs = [ unzip autoreconfHook ]; meta = with stdenv.lib; { description = "POSIX wrappers for C++"; homepage = https://github.com/rescrv/po6; license = licenses.bsd3; }; } </del>

Discussion:
Starting the HyperDex coordinator by running does most of its work by , which must be on . This does not work with the current derivation. This could be fixed either by ensuring replicant is to PATH (would making it a propagated build input be sufficient?) or patching hyperdex to exec replicant by its store path. Install hyperdex, run . It fails. System: 16.03.1242. (Emu) Nix version: nix-env (Nix) 1.11.2 Nixpkgs version: 16.03.1242.
Thanks! Upstream hyperdex is not super maintained and in my experience not stable enough to be used in production. I vote for removing it from nixos. are you relying in hyperdex in nixos or is are you just testing? [1]
I was just testing, not relying on it. Removal sounds okay to me. Alternately, I tested and adding to (in addition to ) fixes the problem.
Change:
# libraries not on the upstream include list, but nevertheless expected # by at least one appimage libtool.lib # for Synfigstudio <ins> at-spi2-core </ins> ]; }; }

Discussion:
Describe the bug Can't run To Reproduce Steps to reproduce the behavior: Pennywise's Appimage to run it Metadata Please run and paste the result. system: host os: multi-user?: sandbox: version: channels(pinage404): channels(root): nixpkgs: Maintainer information: don't know, but it should be here
This is due to missing dependency. I also encountered some AppImage applications which require . Should we add to ? cc
The doesn't mention any libraries, but as there seem to be multiple AppImages requiring it, we already have in the list, and it doesn't increase closure size by much, please go ahead and add it. :) In the meantime, use this overlay until the addition reaches your system:
i don't know where i need to put it, i tried several things but nothing is a valid nix expression (i'm a noob with Nix syntax and Nixos package system)
You could paste the code below somewhere in and run .
Change:
LC_ALL = "en_US.UTF-8"; checkInputs = [ glibcLocales ]; <ins> # tests which assert on strings don't decode results correctly doCheck = isPy3k; pythonImportsCheck = [ "urwid" ]; </ins> meta = with stdenv.lib; { description = "A full-featured console (xterm et al.) user interface library"; homepage = "http://excess.org/urwid";

Discussion:
I'm not sure how to tackle this, but I recently updated my channels (unstable) and getting this issue with urwid (python2.7) after went in. Here's the logs just in case it's helpful to anyone: trying to build just that package fails in the same way: Notify maintainers Metadata
To my own dismay, this was circumvented in my system by disabling tests. However, again, in my system, the only reason I needed was because targetcli is hard set to build with python2. It would probably make sense to switch whatever made you hit this issue to py3 too.
this looks more like urwid doesn't support python2 for testing
isn't that what does? (disable tests on python2?). I tried to install the package with those modifications locally and worked. (Skipped tests). (for other newbies' future ref): I figured out the reason why was installed was as a dependency for . I'm not using , so just ended up removing wicd. was the command that helped me to figure it out.
I made a PR to disable the python2 tests, however, I'm not sure that upstream is checking the results of python2 anymore as the tests seemed to have issues decoding string correctly. This could be just the tests are python3 only, but I wouldn't use python2.... for anything right now
Change:
homepage = http://www.bitcoin.org/; maintainers = with maintainers; [ roconnor AndersonTorres ]; license = licenses.mit; <del> # bitcoin needs hexdump to build, which doesn't seem to build on darwin at the moment. platforms = platforms.linux; </del> <ins> platforms = platforms.unix; </ins> }; }

Discussion:
Describe the bug nix-shell -p bitcoind fails with: Metadata Please run and paste the result. (I'm running nixpkgs-unstable, and I just updated the channel) Maintainer information:
Change:
hash = "sha256-icCerrv6ihBjSTS4Fsw7qhoA5ha8yegfMVRiIOhTvvY="; }; <del> nativeBuildInputs = [ cython ]; </del> <ins> nativeBuildInputs = [ cython setuptools ]; </ins> buildInputs = [ zstd.dev ]; <ins> postPatch = "cython -3 --cplus indexed_zstd/indexed_zstd.pyx"; </ins> # has no tests doCheck = false;

Discussion:
Steps to reproduce the behavior: python3Packages.ratarmountcore This first occurs after updates to and itself. The failure is not actually in a test, and occurs after the tests run. This indicates that some difference in the way the derivation is executing tests compared to upstream's CI is causing the error. Please run and paste the result. Not relevant; the log is on Hydra. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
It seems that only the rapidgzip backend got updated but not ratarmountcore. Either downgrade rapidgzip or update ratarmountcore to 0.6.3 if possible. I'll have to see to get the latter update merged, it seems the auto-update bot does not work for ratarmountcore. This should fix the and output. I have no idea about the last 2 error lines. I cannot reproduce it with nix 2.16.1 on Ubuntu. The command runs fine:
Change:
<ins> { lib, fetchzip }: let version = "1.0.0"; in fetchzip rec { name = "JetBrainsMono-${version}"; url = "https://download.jetbrains.com/fonts/JetBrainsMono-${version}.zip"; sha256 = "0mwqi66d56v4ml1w7wjsiidrh153jvh0czafyic47rkvmxhnrrhv"; postFetch = '' unzip $downloadedFile install -m444 -Dt $out/share/fonts/truetype *.ttf ''; meta = with lib; { description = "A typeface made for developers"; homepage = "https://jetbrains.com/mono/"; license = licenses.asl20; maintainers = [ maintainers.marsam ]; platforms = platforms.all; }; } </ins>

Discussion:
Project description There is a new programming font with ligatures in town, from the people at Jetbrains, known for their IDE's. I have a sneaking suspicion that this is going to be very popular very soon. Metadata homepage URL: source URL: license: Apache 2+ platforms: All(?)
Change:
pdb2pqr = with python3Packages; toPythonApplication pdb2pqr; <del> pymol = callPackage ../applications/science/chemistry/pymol { }; </del> <ins> pymol = callPackage ../applications/science/chemistry/pymol { python3Packages = python311Packages; }; </ins> quantum-espresso = callPackage ../applications/science/chemistry/quantum-espresso { hdf5 = hdf5-fortran;

Discussion:
Steps to reproduce the behavior: pymol 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]:
Note: This seems to be failure of test check. With no check (doCheck=false;), the package builds and runs as expected.
Change:
AppKit ]; <del> # Add missing vulkan dependency to rpath preFixup = lib.optionalString stdenv.isLinux '' patchelf --add-needed ${vulkan-loader}/lib/libvulkan.so.1 $out/bin/lapce ''; </del> postInstall = '' install -Dm0644 $src/extra/images/logo.svg $out/share/icons/hicolor/scalable/apps/lapce.svg '';

Discussion:
lapce: current version: 0.1.0 desired version: 0.1.1 maintainers: The window creation is switched from winit to druid, which fixed lots of keyboard bugs. so maybe require new dependencies and removal of many dependencies. arrow keys in terminal should be working now. vulkan part should be removed as it switched from Wgpu to OpenGL. Please tag this issue in your PR.
Change:
<ins> { bash , gawk , git , gnugrep , fetchFromGitHub , installShellFiles , lib , makeWrapper , stdenv , unixtools , unzip , wget , xdotool , xorg , yad }: stdenv.mkDerivation rec { pname = "steamtinkerlaunch"; version = "11.11"; src = fetchFromGitHub { owner = "sonic2kk"; repo = pname; rev = "v${version}"; hash = "sha256-rWENtgV6spokzkhnmrrzsAQ19dROJ50ofEulU5Jx5IE="; }; # hardcode PROGCMD because #150841 postPatch = '' substituteInPlace steamtinkerlaunch --replace 'PROGCMD="''${0##*/}"' 'PROGCMD="steamtinkerlaunch"' ''; nativeBuildInputs = [ makeWrapper ]; installFlags = [ "PREFIX=${out}" ]; postInstall = '' wrapProgram $out/bin/steamtinkerlaunch --prefix PATH : ${lib.makeBinPath [ bash gawk git gnugrep unixtools.xxd unzip wget xdotool xorg.xprop xorg.xrandr xorg.xwininfo yad ]} ''; meta = with lib; { description = "Linux wrapper tool for use with the Steam client for custom launch options and 3rd party programs"; homepage = "https://github.com/sonic2kk/steamtinkerlaunch"; license = licenses.gpl3; maintainers = with maintainers; [ urandom ]; platforms = lib.platforms.linux; }; } </ins>

Discussion:
Project description Steam Tinker Launch is a Linux wrapper tool for use with the Steam client which allows customizing and start tools and options for games quickly on the fly (see ) Metadata homepage URL: source URL: license: GPL-3.0 platforms: linux
I guess it's related to (partially, since it can also be used as launch command).
Yes true. You think we should merge these requests? Or is this different enough as it's also a launch command?
Change:
qtpass = libsForQt5.callPackage ../applications/misc/qtpass { }; <del> qtractor = libsForQt5.callPackage ../applications/audio/qtractor { }; </del> quantomatic = callPackage ../applications/science/physics/quantomatic { }; quassel = libsForQt5.callPackage ../applications/networking/irc/quassel {

Discussion:
Package name: qtractor Latest released version: 0.9.38 Current version on the unstable channel: 0.9.25 Current version on the stable/release channel: 0.9.25 Notify maintainers Note for maintainers: Please tag this issue in your PR.
Updated to 0.9.37
Updated to 0.9.38
Change:
<ins> { lib , stdenv , fetchFromGitHub , desktop-file-utils , meson , ninja , pkg-config , wrapGAppsHook , evolution-data-server-gtk4 , glib , gnutls , gst_all_1 , json-glib , libadwaita , libpeas , libportal-gtk4 , pulseaudio , sqlite }: stdenv.mkDerivation rec { pname = "valent"; version = "unstable-2023-03-02"; src = fetchFromGitHub { owner = "andyholmes"; repo = "valent"; rev = "4b60f28f46bc948c5f3b30189bb9b5fbe29d2745"; fetchSubmodules = true; sha256 = "sha256-ltf/srQLqtqE71sxEh7VTQqXy2wOpTSdGDsjITOt3f8="; }; nativeBuildInputs = [ desktop-file-utils meson ninja pkg-config wrapGAppsHook ]; buildInputs = [ evolution-data-server-gtk4 glib gnutls gst_all_1.gstreamer gst_all_1.gst-plugins-base json-glib libadwaita libpeas libportal-gtk4 pulseaudio sqlite ]; mesonFlags = [ "-Dplugin_bluez=true" ]; meta = with lib; { description = "An implementation of the KDE Connect protocol, built on GNOME platform libraries"; homepage = "https://github.com/andyholmes/valent/"; changelog = "https://github.com/andyholmes/valent/blob/${src.rev}/CHANGELOG.md"; license = with licenses; [ gpl3Plus cc0 ]; maintainers = with maintainers; [ federicoschonborn ]; platforms = platforms.linux; }; } </ins>

Discussion:
Project description <!-- Describe the project a little: --Valent is an implementation of the KDE Connect protocol, built on GNOME platform libraries. Seems to be a re-implementation of without the GNOME Shell bits made by the same author, intended to work on more desktop environments but it also has a (separate) GNOME Shell extension. I have packaged it myself () and it works surprisingly well for an alpha. ! Metadata homepage URL: source URL: license: gpl3Plus, cc0 platforms: linux
The linked package works for the part. Tested the clipboard syncing, presentation mode, and remote input (among other things). I've yet to test the app with the GNOME Shell extension though. Looking forward for the PR. !
Change:
sed -e 's/pam_rhosts//g' -i modules/Makefile.in ''; <ins> configureFlags = [ "--includedir=${placeholder "out"}/include/security" "--enable-sconfigdir=/etc/security" ]; installFlags = [ "SCONFIGDIR=${placeholder "out"}/etc/security" ]; </ins> doCheck = false; # fails meta = with stdenv.lib; {

Discussion:
Describe the bug pamtime only uses the example config file in NixOS and is useless without overriding and rebuilding almost every other package. To Reproduce Steps to reproduce the behavior: to add to all pam.d config files. and add . and attempt to ssh as into the machine log in. Expected behavior In the example above, I would expect ssh to forbid from logging in at any time. Additional context Ideally, upstream linux-pam would allow pamtime to be configured at run time similarly to pamenv. However, I would still expect to be able to use pamtime without overriding the linux-pam package to manually change the location of In the current setup, pamtime only ever looks at the example file in the linux-pam package which makes it effectively useless. If the package is overridden to use a different location for that one file, a massive number of packages then needs to be rebuilt manually as they in one way or another depend on pam. Metadata system: host os: multi-user?: sandbox: version: channels(root): channels(locus): nixpkgs:
It looks like we may just need to pass and to fix this.
Try this PR:
Change:
glm gtk3 hidapi <del> imgui </del> <ins> imgui' </ins> libpng libusb1 libzip

Discussion:
Steps to reproduce the behavior: cemu In ImGui changelog for 1.90.2: <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --cc Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Always paste the full log or big parts from it, not just the short log.
my build log failure
apparently decided to ignore this build failure in . Not sure why we even have tests if we just blindly ignore them.
Change:
# For building runtime libs depsBuildTarget = toolsForTarget; <del> buildInputs = [ perl ] ++ (libDeps hostPlatform); </del> <ins> buildInputs = [ perl bash ] ++ (libDeps hostPlatform); </ins> propagatedBuildInputs = [ targetPackages.stdenv.cc ] ++ stdenv.lib.optional useLLVM llvmPackages.llvm;

Discussion:
The shebang line for is using instead of a path from the nix store. Here is a minimal example, showing the problem appearing between and : By using , I landed on , and I think the problematic commit is This commit reverts a previous reversal, The initial reversal mentions problems with shebangs for packages using bootstrap . cc who both worked on the pull request and did the various reversals.
Thank you and for the speedy fix.
Change:
useLLVM = stdenv.hostPlatform.useLLVM or false; bareMetal = stdenv.hostPlatform.parsed.kernel.name == "none"; haveLibc = stdenv.cc.libc != null; <del> inherit (stdenv.hostPlatform) isMusl; </del> <ins> inherit (stdenv.hostPlatform) isMusl isAarch64; </ins> in

Discussion:
Compile fails on aarch64 master . Steps to reproduce the behavior: to install ungoogled-chromium on aarch64 master compile <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --hardly relevant, as this is marked broken in master Maintainer information:
possible workaround is to disable building the libfuzzer, I'll check that now.
Sorry wrong patch, tested system build reboot chromium & firefox with this on aarch64
Change:
users.mutableUsers = false; boot.initrd.systemd.enable = true; boot.kernelPackages = pkgs.linuxPackages_latest; <ins> time.timeZone = "Utc"; </ins> specialisation.new-generation.configuration = { environment.etc."newgen".text = "newgen";

Discussion:
Locale module is the only place in nixpkgs that mentions etc entry type: overlay generation code acknowledges the existence only of the type (). The provided system configuration builds. Mentioning since they committed the overlay code. Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Simply replacing with in the locale module makes the build succeed. reports the correct timezone, and the symlink correctly points to the selected timezone within the .
Ah, it's much more complicated than that. The reason why exists is because some apps try to read the timezone name directly from the symlink path itself (, ). While the solution above makes the provided system configuration build, Java applications will fallback to using UTC because they are unable to parse the timezone name from the generated symlink.
Change:
users.mutableUsers = false; boot.initrd.systemd.enable = true; boot.kernelPackages = pkgs.linuxPackages_latest; <ins> time.timeZone = "Utc"; </ins> specialisation.new-generation.configuration = { environment.etc."newgen".text = "newgen";

Discussion:
If I try to make my system perlless, my system rebuild fails. Steps to reproduce the behavior: the following module to your system configuration: The command succeeds. Commands: succeeds. I use flakes. <!-- 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]:
Odd, thought was meant to fix this. Perhaps more digging is needed.
What is the revision of nixpkgs that you are trying to build your system with? is merged without a 24.05 backport.
Per nix-info they're on 24.05 not unstable, so that makes sense. Is it backportable?
The PR itself is backportable, but I'm not sure if it complies with nixpkgs' .
Right, that's what I meant to ask - is it a breaking change? (i.e. is it possible for someone to rely on the old behaviour?) or simply a bugfix with low chance of regressions?
Is it not a breaking change because the option is marked as experimental (and also because the behavior before the PR was erroneous in the first place).
Fixed in .
Change:
"-Dv4l2=disabled" # Linux-only "-Dximagesrc=disabled" # Linux-only "-Dpulse=disabled" # TODO check if we can keep this enabled <del> ] ++ optionals (!(stdenv.isLinux && stdenv.hostPlatform.isAarch64)) [ "-Drpicamsrc=disabled" # only works on Linux aarch64, see https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/blob/428c9b60532917c0ac49c9d48b15bdcd00a1370b/sys/rpicamsrc/meson.build#L10 </del> <ins> ] ++ optionals (!raspiCameraSupport) [ "-Drpicamsrc=disabled" </ins> ]; postPatch = ''

Discussion:
In found that the upgrade from gstreamer 0.16 to 0.18 broke : I think is a RaspberryPi specific file and pulled in via . I don't know though whether this could be build on general , and if yes, how to get , so some help is needed.
can be found in and , which appear to be identical packages.
And can/should that be as a general gstreamer dependency on aarch64?
I feel like it should be an optional dependency. I'll file a PR real quick
Change:
stdenv.mkDerivation rec { pname = "bpftune"; <del> version = "unstable-2023-12-20"; </del> <ins> version = "0-unstable-2024-05-17"; </ins> src = fetchFromGitHub { owner = "oracle"; repo = "bpftune"; <del> rev = "0e6bca2e5880fcbaac6478c4042f5f9314e61463"; hash = "sha256-y9WQrQb9U5YdzKAR63FzC8V1+jZL027pzAmQPpgM3jM="; </del> <ins> rev = "83115c56cf9620fe5669f4a3be67ab779d8f4536"; hash = "sha256-er2i7CEUXF3BpWTG//s8C0xfIk5gSVOHB8nE1r7PX78="; </ins> }; postPatch = ''

Discussion:
bpftune: 2024-05-17: Current version on the unstable channel: 2023-12-20 Current version on the stable/release channel: 2023-09-11 The upstream developer fixed an issue three days ago that fixed the now borked bpftune in NixOS when using kernel 6.9 or later.
Change:
<ins> { buildPythonPackage , cython , fetchPypi , jdk , lib , six }: buildPythonPackage rec { pname = "pyjnius"; version = "1.4.1"; src = fetchPypi { inherit pname version; sha256 = "8bc1a1b06fb11df8dd8b8d56f5ecceab614d4ba70cf559c64ae2f146423d53ce"; }; propagatedBuildInputs = [ six ]; nativeBuildInputs = [ jdk cython ]; pythonImportsCheck = [ "jnius" ]; meta = with lib; { description = "A Python module to access Java classes as Python classes using the Java Native Interface (JNI)"; homepage = "https://github.com/kivy/pyjnius"; license = licenses.mit; maintainers = with maintainers; [ ifurther ]; }; } </ins>

Discussion:
Traceback (most recent call last): File "/nix/store/z4sd700l05shq1z72g21w0fav2xs955d-pipenv-2022.1.8/lib/python3.9/site-", line 83, in androidfolder from jnius import autoclass # noqa: SC200 ModuleNotFoundError: No module named 'jnius' The error is belong in pipenv using jnius on Android. <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --- system: host os: multi-user?: sandbox: version: channels(nix-on-droid): nixpkgs: