jiebi/CodeConvo huggingface dataset:
Code change:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.

Related discussion:
text: python3.pkgs.buildPythonApplication rec { pname = "sticky"; <del> version = "1.14"; </del> <ins> version = "1.16"; </ins> format = "other"; src = fetchFromGitHub { owner = "linuxmint"; repo = pname; rev = version; <del> hash = "sha256-7UZbCbzQ1ZrSzxTUdbA+wsH3p27qj/c/cM4GY/kzG6E="; </del> <ins> hash = "sha256-+opzjvSdGwwn1imNzSsfD9rfs4naSXzcMBJRPhKW2WM="; </ins> }; postPatch = ''
Code change:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.

Related discussion:
text: stdenv.mkDerivation rec { pname = "bulky"; <del> version = "2.8"; </del> <ins> version = "2.9"; </ins> src = fetchFromGitHub { owner = "linuxmint"; repo = "bulky"; rev = version; <del> hash = "sha256-DZjX4xPyA30TTyOX+VyGSY/FWPX2316CYImDWRma9r0="; </del> <ins> hash = "sha256-eCu7AgL4bzstu8vsqBJImRZm1qMVnnycuOShY5fB+C4="; </ins> }; nativeBuildInputs = [
Code change:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.

Related discussion:
text: -e s,/usr/share/locale,/run/current-system/sw/share/locale,g  {} + <del> sed "s|/usr/share/sounds|/run/current-system/sw/share/sounds|g" -i ./files/usr/share/cinnamon/cinnamon-settings/bin/SettingsWidgets.py sed "s|'python3'|'${pythonEnv.interpreter}'|g" -i ./files/usr/share/cinnamon/cinnamon-settings/bin/CinnamonGtkSettings.py sed "s|/usr/bin/cinnamon-screensaver-command|/run/current-system/sw/bin/cinnamon-screensaver-command|g"  -i ./files/usr/share/cinnamon/applets/menu@cinnamon.org/applet.js -i ./files/usr/share/cinnamon/applets/user@cinnamon.org/applet.js sed "s|"/usr/lib"|"${cinnamon-control-center}/lib"|g" -i ./files/usr/share/cinnamon/cinnamon-settings/bin/capi.py sed 's|"lspci"|"${pciutils}/bin/lspci"|g' -i ./files/usr/share/cinnamon/cinnamon-settings/modules/cs_info.py </del> <ins> # All optional and may introduce circular dependency. find ./files/usr/share/cinnamon/applets -type f -exec sed -i  -e '/^#/!s,/usr/bin,/run/current-system/sw/bin,g'  {} + </ins> <del> sed "s| cinnamon-session| ${cinnamon-session}/bin/cinnamon-session|g" -i ./files/usr/bin/cinnamon-session-cinnamon  -i ./files/usr/bin/cinnamon-session-cinnamon2d </del> <ins> pushd ./files/usr/share/cinnamon/cinnamon-settings substituteInPlace ./bin/capi.py                     --replace '"/usr/lib"' '"${cinnamon-control-center}/lib"' substituteInPlace ./bin/CinnamonGtkSettings.py      --replace "'python3'" "'${pythonEnv.interpreter}'" substituteInPlace ./bin/SettingsWidgets.py          --replace "/usr/share/sounds" "/run/current-system/sw/share/sounds" substituteInPlace ./bin/Spices.py                   --replace "msgfmt" "${gettext}/bin/msgfmt" substituteInPlace ./modules/cs_info.py              --replace "lspci" "${pciutils}/bin/lspci" popd </ins> <del> sed "s|msgfmt|${gettext}/bin/msgfmt|g" -i ./files/usr/share/cinnamon/cinnamon-settings/bin/Spices.py </del> <ins> sed "s| cinnamon-session| ${cinnamon-session}/bin/cinnamon-session|g" -i ./files/usr/bin/cinnamon-session-{cinnamon,cinnamon2d} </ins> patchShebangs src/data-to-c.pl '';
Code change:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.

Related discussion:
text: --prefix XDG_DATA_DIRS : "${gnome.caribou}/share" ) <ins> buildPythonPath "$out ${python3.pkgs.xapp}" # https://github.com/NixOS/nixpkgs/issues/200397 patchPythonScript $out/bin/cinnamon-spice-updater </ins> # https://github.com/NixOS/nixpkgs/issues/129946 <del> buildPythonPath "${python3.pkgs.xapp}" </del> patchPythonScript $out/share/cinnamon/cinnamon-desktop-editor/cinnamon-desktop-editor.py <ins> # Called as `pkexec cinnamon-settings-users.py`. wrapGApp $out/share/cinnamon/cinnamon-settings-users/cinnamon-settings-users.py </ins> ''; passthru = {
Code change:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.

Related discussion:
text: stdenvNoCC.mkDerivation rec { pname = "mint-cursor-themes"; <del> version = "1.0.1"; </del> <ins> version = "1.0.2"; </ins> src = fetchFromGitHub { owner = "linuxmint"; repo = pname; # They don't really do tags, this is just a named commit. <del> rev = "e17f8a4620827235dabbe5221bd0ee8c44dad0d5"; hash = "sha256-yLUmIVh884uDVkNil7qxf6t/gykipzBvPgzwmY3zvQk="; </del> <ins> rev = "d2c1428b499a347c291dafb13c89699fdbdd4be7"; hash = "sha256-i2Wf+OKwal9G5hkcAdmGSgX6txu1AHajqqPJdhpJoA0="; </ins> }; installPhase = ''
Code change:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.

Related discussion:
text: in stdenv.mkDerivation rec { pname = "warpinator"; <del> version = "1.6.2"; </del> <ins> version = "1.6.3"; </ins> src = fetchFromGitHub { owner = "linuxmint"; repo = pname; rev = version; <del> hash = "sha256-crGW1FBfTEiKqx981EzG414we4icyBv/keGtMlr8UFc="; </del> <ins> hash = "sha256-iCq2cwZgsq5/bJqjYuZ6dZLqz2nIpgHCrIQeMVxJNXY="; </ins> }; nativeBuildInputs = [
Code change:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.

Related discussion:
text: # Patch pastebin & inxi location sed "s|/usr/bin/pastebin|$out/bin/pastebin|" -i scripts/upload-system-info sed "s|'inxi'|'${inxi}/bin/inxi'|" -i scripts/upload-system-info <del> # Patch gtk3 module target dir substituteInPlace libxapp/meson.build  --replace "gtk3_dep.get_pkgconfig_variable('libdir')" "'$out'" </del> ''; <ins> # Fix gtk3 module target dir. Proper upstream solution should be using define_variable. PKG_CONFIG_GTK__3_0_LIBDIR = "${placeholder "out"}/lib"; </ins> meta = with lib; { homepage = "https://github.com/linuxmint/xapp"; description = "Cross-desktop libraries and common resources";
Code change:
When running , I get Restarting the system did not help. Steps to reproduce the behavior: The program should launch without crashing. If applicable, add screenshots to help explain your problem.
Hello Thanks for reporting this issue, and sorry for breaking the nvtop package! I made a pull-request to fix nvtop: What do you think about this fix?
Thank you for your fast reaction ! The PR looks good to me. I hope it will get merged soon :)

Related discussion:
text: , cudatoolkit , libdrm , ncurses <ins> , nvtop , testers , udev </ins> , addOpenGLRunpath , amd ? true , nvidia ? true <del> , udev </del> }: let pname-suffix = if amd && nvidia then "" else if amd then "-amd" else "-nvidia"; nvidia-postFixup = "addOpenGLRunpath $out/bin/nvtop"; <del> libPath = lib.makeLibraryPath [ libdrm ncurses ]; </del> <ins> libPath = lib.makeLibraryPath [ libdrm ncurses udev ]; </ins> amd-postFixup = '' patchelf  --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)"
Code change:
When running , I get Restarting the system did not help. Steps to reproduce the behavior: The program should launch without crashing. If applicable, add screenshots to help explain your problem.
Hello Thanks for reporting this issue, and sorry for breaking the nvtop package! I made a pull-request to fix nvtop: What do you think about this fix?
Thank you for your fast reaction ! The PR looks good to me. I hope it will get merged soon :)

Related discussion:
text: cmakeFlags = with lib; [ "-DCMAKE_BUILD_TYPE=Release" "-DBUILD_TESTING=ON" <ins> "-DUSE_LIBUDEV_OVER_LIBSYSTEMD=ON" </ins> ] ++ optional nvidia "-DNVML_INCLUDE_DIRS=${cudatoolkit}/include" ++ optional nvidia "-DNVML_LIBRARIES=${cudatoolkit}/targets/x86_64-linux/lib/stubs/libnvidia-ml.so" ++ optional (!amd) "-DAMDGPU_SUPPORT=OFF"
Code change:
When running , I get Restarting the system did not help. Steps to reproduce the behavior: The program should launch without crashing. If applicable, add screenshots to help explain your problem.
Hello Thanks for reporting this issue, and sorry for breaking the nvtop package! I made a pull-request to fix nvtop: What do you think about this fix?
Thank you for your fast reaction ! The PR looks good to me. I hope it will get merged soon :)

Related discussion:
text: doCheck = true; <ins> passthru = { tests.version = testers.testVersion { inherit version; package = nvtop; command = "nvtop --version"; }; }; </ins> meta = with lib; { description = "A (h)top like task monitor for AMD, Intel and NVIDIA GPUs"; longDescription = ''
Code change:
pam_p11 causes login segfault in NixOS 22.11. GDM and Gnome lock screen are also affected. Steps to reproduce the behavior: to log in on the console or run `: Bug was not present in 22.05.
Reverting does not help.
Fixed by: Can someone with commit access apply this patch?
Can you open a pull request with that patch?
I hope that NixOS can be unbureaucratic enough that PRs are not required for this kind of patch.
It's generally preferred to do basically everything through PRs, so some basic CI checks can run & maintainers can be pinged automatically. We can just self-merge though. I created containing your diff and adding a comment for explanation.
This seems to be whats causing my 22.11-beta19 install on an Alder Lake 12700H (no gpu) to not boot into GDM. This is a pretty heinous issue that should be tested for. The graphical installer doesn't install Vim, WPA_Supplicant or anything useful for figuring out what went wrong, despite having Vim and WPA Supplicant available on the installation media/calamares.
Well I just did - you're welcome. That's what beta releases are for, aren't they?

Related discussion:
text: enableParallelBuilding = true; <ins> passthru = { inherit openssl; }; </ins> meta = with lib; { description = "Small layer on top of PKCS#11 API to make PKCS#11 implementations easier"; homepage = "https://github.com/OpenSC/libp11";
Code change:
pam_p11 causes login segfault in NixOS 22.11. GDM and Gnome lock screen are also affected. Steps to reproduce the behavior: to log in on the console or run `: Bug was not present in 22.05.
Reverting does not help.
Fixed by: Can someone with commit access apply this patch?
Can you open a pull request with that patch?
I hope that NixOS can be unbureaucratic enough that PRs are not required for this kind of patch.
It's generally preferred to do basically everything through PRs, so some basic CI checks can run & maintainers can be pinged automatically. We can just self-merge though. I created containing your diff and adding a comment for explanation.
This seems to be whats causing my 22.11-beta19 install on an Alder Lake 12700H (no gpu) to not boot into GDM. This is a pretty heinous issue that should be tested for. The graphical installer doesn't install Vim, WPA_Supplicant or anything useful for figuring out what went wrong, despite having Vim and WPA Supplicant available on the installation media/calamares.
Well I just did - you're welcome. That's what beta releases are for, aren't they?

Related discussion:
text: <del> { lib, stdenv, fetchFromGitHub, autoreconfHook, pkg-config, openssl, libp11, pam, libintl }: </del> <ins> { lib, stdenv, fetchFromGitHub, autoreconfHook, pkg-config, libp11, pam, libintl }: </ins> stdenv.mkDerivation rec { pname = "pam_p11";
Code change:
pam_p11 causes login segfault in NixOS 22.11. GDM and Gnome lock screen are also affected. Steps to reproduce the behavior: to log in on the console or run `: Bug was not present in 22.05.
Reverting does not help.
Fixed by: Can someone with commit access apply this patch?
Can you open a pull request with that patch?
I hope that NixOS can be unbureaucratic enough that PRs are not required for this kind of patch.
It's generally preferred to do basically everything through PRs, so some basic CI checks can run & maintainers can be pinged automatically. We can just self-merge though. I created containing your diff and adding a comment for explanation.
This seems to be whats causing my 22.11-beta19 install on an Alder Lake 12700H (no gpu) to not boot into GDM. This is a pretty heinous issue that should be tested for. The graphical installer doesn't install Vim, WPA_Supplicant or anything useful for figuring out what went wrong, despite having Vim and WPA Supplicant available on the installation media/calamares.
Well I just did - you're welcome. That's what beta releases are for, aren't they?

Related discussion:
text: }; nativeBuildInputs = [ autoreconfHook pkg-config ]; <del> buildInputs = [ pam openssl libp11 ] </del> <ins> buildInputs = [ pam libp11.passthru.openssl libp11 ] </ins> ++ lib.optionals stdenv.isDarwin [ libintl ]; meta = with lib; {
Code change:
pam_p11 causes login segfault in NixOS 22.11. GDM and Gnome lock screen are also affected. Steps to reproduce the behavior: to log in on the console or run `: Bug was not present in 22.05.
Reverting does not help.
Fixed by: Can someone with commit access apply this patch?
Can you open a pull request with that patch?
I hope that NixOS can be unbureaucratic enough that PRs are not required for this kind of patch.
It's generally preferred to do basically everything through PRs, so some basic CI checks can run & maintainers can be pinged automatically. We can just self-merge though. I created containing your diff and adding a comment for explanation.
This seems to be whats causing my 22.11-beta19 install on an Alder Lake 12700H (no gpu) to not boot into GDM. This is a pretty heinous issue that should be tested for. The graphical installer doesn't install Vim, WPA_Supplicant or anything useful for figuring out what went wrong, despite having Vim and WPA Supplicant available on the installation media/calamares.
Well I just did - you're welcome. That's what beta releases are for, aren't they?

Related discussion:
text: ++ lib.optionals stdenv.hostPlatform.isMusl [ argp-standalone ] ++ lib.optionals withJitterEntropy [ jitterentropy ] ++ lib.optionals withNistBeacon    [ curl jansson libxml2 ] <del> ++ lib.optionals withPkcs11        [ libp11 openssl ] </del> <ins> ++ lib.optionals withPkcs11        [ libp11 libp11.passthru.openssl ] </ins> ++ lib.optionals withRtlsdr        [ librtlsdr ]; enableParallelBuilding = true;
Code change:
pam_p11 causes login segfault in NixOS 22.11. GDM and Gnome lock screen are also affected. Steps to reproduce the behavior: to log in on the console or run `: Bug was not present in 22.05.
Reverting does not help.
Fixed by: Can someone with commit access apply this patch?
Can you open a pull request with that patch?
I hope that NixOS can be unbureaucratic enough that PRs are not required for this kind of patch.
It's generally preferred to do basically everything through PRs, so some basic CI checks can run & maintainers can be pinged automatically. We can just self-merge though. I created containing your diff and adding a comment for explanation.
This seems to be whats causing my 22.11-beta19 install on an Alder Lake 12700H (no gpu) to not boot into GDM. This is a pretty heinous issue that should be tested for. The graphical installer doesn't install Vim, WPA_Supplicant or anything useful for figuring out what went wrong, despite having Vim and WPA Supplicant available on the installation media/calamares.
Well I just did - you're welcome. That's what beta releases are for, aren't they?

Related discussion:
text: libow = callPackage ../development/libraries/libow { }; <del> libp11 = callPackage ../development/libraries/libp11 { }; </del> <ins> libp11 = callPackage ../development/libraries/libp11 { openssl = openssl_1_1; }; </ins> libpam-wrapper = callPackage ../development/libraries/libpam-wrapper { };
Code change:
pam_p11 causes login segfault in NixOS 22.11. GDM and Gnome lock screen are also affected. Steps to reproduce the behavior: to log in on the console or run `: Bug was not present in 22.05.
Reverting does not help.
Fixed by: Can someone with commit access apply this patch?
Can you open a pull request with that patch?
I hope that NixOS can be unbureaucratic enough that PRs are not required for this kind of patch.
It's generally preferred to do basically everything through PRs, so some basic CI checks can run & maintainers can be pinged automatically. We can just self-merge though. I created containing your diff and adding a comment for explanation.
This seems to be whats causing my 22.11-beta19 install on an Alder Lake 12700H (no gpu) to not boot into GDM. This is a pretty heinous issue that should be tested for. The graphical installer doesn't install Vim, WPA_Supplicant or anything useful for figuring out what went wrong, despite having Vim and WPA Supplicant available on the installation media/calamares.
Well I just did - you're welcome. That's what beta releases are for, aren't they?

Related discussion:
text: pam_mysql = callPackage ../os-specific/linux/pam_mysql { }; <del> pam_p11 = callPackage ../os-specific/linux/pam_p11 { openssl = openssl_1_1; }; </del> <ins> pam_p11 = callPackage ../os-specific/linux/pam_p11 { }; </ins> pam_pgsql = callPackage ../os-specific/linux/pam_pgsql { };
Code change:
i3 has been updated to 4.13. I attempted to update our derivation but they switched to and I don't know how that works :) If somebody could update i3, I would really appreciate that. /cc maintainers
FYI, for to work it is sufficient to add to the (see ).
Thanks :) I think with the main i3 the issue was that the needed to change and I did not know what to set them to.
Yes, and indeed the and are already included with the source distribution for , but I had to add the for .

Related discussion:
text: { fetchurl, stdenv, which, pkgconfig, makeWrapper, libxcb, xcbutilkeysyms <del> , xcbutil, xcbutilwm, libstartup_notification, libX11, pcre, libev, yajl , xcb-util-cursor, coreutils, perl, pango, perlPackages, libxkbcommon </del> <ins> , xcbutil, xcbutilwm, xcbutilxrm, libstartup_notification, libX11, pcre, libev , yajl, xcb-util-cursor, coreutils, perl, pango, perlPackages, libxkbcommon </ins> , xorgserver, xvfb_run, dmenu, i3status }: stdenv.mkDerivation rec { name = "i3-${version}"; <del> version = "4.12"; </del> <ins> version = "4.13"; </ins> src = fetchurl { url = "http://i3wm.org/downloads/${name}.tar.bz2"; <del> sha256 = "1d3q3lgpjbkmcwzjhp0dfr0jq847silcfg087slcnj95ikh1r7p1"; </del> <ins> sha256 = "12ngz32swh9n85xy0cz1lq16aqi9ys5hq19v589q9a97wn1k3hcl"; </ins> }; <ins> nativeBuildInputs = [ which pkgconfig makeWrapper ]; </ins> buildInputs = [ <del> which pkgconfig makeWrapper libxcb xcbutilkeysyms xcbutil xcbutilwm libxkbcommon </del> <ins> libxcb xcbutilkeysyms xcbutil xcbutilwm xcbutilxrm libxkbcommon </ins> libstartup_notification libX11 pcre libev yajl xcb-util-cursor perl pango perlPackages.AnyEventI3 perlPackages.X11XCB perlPackages.IPCRun perlPackages.ExtUtilsPkgConfig perlPackages.TestMore perlPackages.InlineC xorgserver xvfb_run ]; <ins> configureFlags = [ "--disable-builddir" ]; enableParallelBuilding = true; </ins> postPatch = '' patchShebangs . '';
Code change:
i3 has been updated to 4.13. I attempted to update our derivation but they switched to and I don't know how that works :) If somebody could update i3, I would really appreciate that. /cc maintainers
FYI, for to work it is sufficient to add to the (see ).
Thanks :) I think with the main i3 the issue was that the needed to change and I did not know what to set them to.
Yes, and indeed the and are already included with the source distribution for , but I had to add the for .

Related discussion:
text: ! grep -q '^not ok' testcases/latest/complete-run.log ''; <del> configurePhase = "makeFlags=PREFIX=$out"; </del> postInstall = '' wrapProgram "$out/bin/i3-save-tree" --prefix PERL5LIB ":" "$PERL5LIB" <del> mkdir -p $out/man/man1 cp man/*.1 $out/man/man1 </del> for program in $out/bin/i3-sensible-*; do sed -i 's/which/command -v/' $program done
Code change:
i3 has been updated to 4.13. I attempted to update our derivation but they switched to and I don't know how that works :) If somebody could update i3, I would really appreciate that. /cc maintainers
FYI, for to work it is sufficient to add to the (see ).
Thanks :) I think with the main i3 the issue was that the needed to change and I did not know what to set them to.
Yes, and indeed the and are already included with the source distribution for , but I had to add the for .

Related discussion:
text: meta = with stdenv.lib; { description = "A tiling window manager"; homepage    = "http://i3wm.org"; <del> maintainers = with maintainers; [ garbas modulistic ]; </del> <ins> maintainers = with maintainers; [ garbas modulistic fpletz ]; </ins> license     = licenses.bsd3; platforms   = platforms.all;
Code change:
The srb2 assets are outdated. Steps to reproduce the behaviour: the game or run the command while in game The that the assets are being downloaded from hasn't been updated since 2020
I just noticed this pr, but it seems to have gone dead 3 months ago
closing as I'm making a new issue

Related discussion:
text: <ins> diff --git a/CMakeLists.txt b/CMakeLists.txt index 915912af5..f5c2cf9cc 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -91,11 +91,6 @@ if("${CMAKE_HOST_SYSTEM_NAME}" STREQUAL Windows) endif() endif() else() -	CPMAddPackage( -		NAME Ccache.cmake -		GITHUB_REPOSITORY TheLartians/Ccache.cmake -		VERSION 1.2 -	) endif() # Dependencies -- 2.40.1 </ins>
Code change:
The srb2 assets are outdated. Steps to reproduce the behaviour: the game or run the command while in game The that the assets are being downloaded from hasn't been updated since 2020
I just noticed this pr, but it seems to have gone dead 3 months ago
closing as I'm making a new issue

Related discussion:
text: , stdenv , fetchurl , fetchFromGitHub <del> , substituteAll </del> , cmake , curl , nasm