jiebi/CodeConvo huggingface dataset:
Code change:
url = "https://github.com/AcademySoftwareFoundation/OpenColorIO/commit/332462e7f5051b7e26ee3d8c22890cd5e71e7c30.patch"; sha256 = "sha256-7xHALhnOkKszgFBgPIbiZQaORnEJ+1M6RyoZdFgjElM="; }) <ins> (fetchpatch { name = "minizip-ng-4.patch"; url = "https://gitlab.archlinux.org/archlinux/packaging/packages/opencolorio/-/raw/5fc40f42f5c05d905793610c37b46ca3649245f3/minizip-ng-4.patch"; hash = "sha256-B+dbBVRn0EuGtJaWxz5ah9el0RN7cLb81hgqnKkvhew="; }) </ins> ]; postPatch = lib.optionalString stdenv.isDarwin ''

Related discussion:
text: branch to nixos-unstable to your with
Bisect has identified (specifically ) as the source of this failure in nixpkgs, cc See also upstream issue
If you're using a flake, this is how I was able to mitigate this until it's fixed. I rolled back the changes from in a branch on my fork, and used it as an overlay for the affected packages like this: This is how you would add them to your overlay: Ofc, depending on how you do stuff, you'll have to fiddle with it a bit to get it to work. I found this helpful:
The fix is now on the nixos-unstable branch, but I still cannot build my system after updating, with the same build failure for opencolorio. Is this expected?
Probably not, since and both build on Hydra.
Oh thank you, this is very bizarre. I will ask on the forum what's going on
This issue has been mentioned on NixOS Discourse. There might be relevant details there:
Code change:
<ins> diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index a0be580e..ba143851 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -219,6 +219,13 @@ if (GLFW_BUILD_X11) if (NOT X11_Xshape_INCLUDE_PATH) message(FATAL_ERROR "X Shape headers not found; install libxext development package") endif() + +    target_link_libraries(glfw PRIVATE ${X11_Xrandr_LIB} +                                       ${X11_Xinerama_LIB} +                                       ${X11_Xkb_LIB} +                                       ${X11_Xcursor_LIB} +                                       ${X11_Xi_LIB} +                                       ${X11_Xshape_LIB}) endif() if (UNIX AND NOT APPLE) </ins>

Related discussion:
text: should hide the cursor but does not on X11. I have not tested this on Wayland or Cocoa. Steps to reproduce the behavior: GLFW 3.3.x (to match unstable, which is currently 3.3.4) GLFW's dependencies ( on X11), build the library's examples. The default CMake build should do the job: using or equivalent. Verify that left clicking the window hides the cursor since is set. NixOS's , build the example manually with . Execute it and now clicking moves the cursor to the center of the window but does not hide it. NixOS build should behave the same as the hand-built one. GLFWCURSOR_HIDDEN is also affected. and
Could you send a strace of you running the example?
Yes I could. It seems that it's not due to how GLFW is built by nix but how dynamic libraries are looked up at runtime. Since the handmade executable only works with I made the wrong assumption: the one linked against nixpkgs' GLFW also work correctly. For the ease of reproducibility this is the program for producing the straces: I'm not sure if this is the cause but something is definitely wrong:
Code change:
upstream-info = (callPackage ./update.nix {}).getChannel channel; mkChromiumDerivation = callPackage ./common.nix { <del> inherit enableNaCl enableHotwording gnomeSupport gnome </del> <ins> inherit enableNaCl gnomeSupport gnome </ins> gnomeKeyringSupport proprietaryCodecs cupsSupport pulseSupport enableWideVine; };

Related discussion:
text: When building chromium with args like this and the gn args aren't known/accepted by chromium (like the above one actually is that is still given incorrectly in 18.03), it will print: but continue to build anyway, which can be super surprising when your build doesn't do what you expect. We should catch that when building. Unfortunately doesn't seem to have a feature to warn. But we can run to see the in stdout and grep for it: I have confirmed that only seems to work that way when giving a build target, like . CC
Suggestion: We switch to
PR
pull request was merged.
Code change:
''; patches = [ <ins> # Upstream insists on not allowing bindir and other dir options # outside of prefix for some reason: # https://github.com/mesonbuild/meson/issues/2561 # We remove the check so multiple outputs can work sanely. ./allow-dirs-outside-of-prefix.patch </ins> # Unlike libtool, vanilla Meson does not pass any information # about the path library will be installed to to g-ir-scanner, # breaking the GIR when path other than ${!outputLib}/lib is used.

Related discussion:
text: To prevent issues like , meson setup hook should define install directories just like stdenv and cmake do Unfortunately, simple patch like the following will not work Meson does not like it very much when the directories are not descendants of prefix See also
Code change:
<del> { lib, stdenv, fetchhg, fetchurl, gtk2, glib, pkg-config, unzip, ncurses, zip }: </del> <ins> { lib, gcc10Stdenv, fetchhg, fetchurl, gtk2, glib, pkg-config, unzip, ncurses, zip }: </ins> <del> stdenv.mkDerivation rec { </del> <ins> gcc10Stdenv.mkDerivation rec { </ins> version = "10.8"; pname = "textadept";

Related discussion:
text: Steps to reproduce the behavior: textadept (10.8) The error is in building Scintilla <!-- 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.
Code change:
desktopItems = [ (makeDesktopItem { name = "AnyDesk"; <del> exec = "${placeholder "out"}/bin/anydesk %u"; </del> <ins> exec = "anydesk %u"; </ins> icon = "anydesk"; desktopName = "AnyDesk"; genericName = description;

Related discussion:
text: Hi. So I'm running KDE and whenever I try to launch the AnyDesk application, I get the notification: Launching AnyDesk (Failed) Could not find the program '/nix/store/a1cic900s29g1hfwv731cm3bsgni7p9i-AnyDesk.desktop/bin/anydesk' I have to run from the terminal to open AnyDesk. Please run and paste the result. Content of /run/current-system/sw/share/applications/AnyDesk.desktop: However, that Nix store path only has the desktop file and no binaries. The AnyDesk binary is in .
Code change:
sha256 = "0q3kg7dqvsiar3m70mvmn6bkkm4k1zn4fbh5r8ynib753wlhf504"; } , ... <del> }: </del> <ins> } @ args: </ins> let majorMinor = lib.versions.majorMinor linux.modDirVersion;

Related discussion:
text: Mainline / hardened / testing / ... kernels can be painlessly modified via : Sadly, this does not work with a bunch of other kernel variants: All kernels should honor I am not entirely sure how dynamic binding is implemented in kernel stuff, however slightly changing xanmod parts this way seems to fix : zen and lqx look similar to xanmod, no idea about libre Pinging people who seem to update affected packages often Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
same underlying issue as it seems
Can confirm that this patch fix the issue for zen and lqx kernels:
have you submitted these changes as a PR that I can review?
sorry for the late response. I created a new PR with this changes.
This issue has been mentioned on NixOS Discourse. There might be relevant details there:
Code change:
versionPolicy = "odd-unstable"; }; <del> mkSessionForWm = { wmName, wmLabel, wmCommand, enableGnomePanel }: </del> <ins> mkSessionForWm = { wmName, wmLabel, wmCommand, enableGnomePanel, panelModulePackages }: </ins> let wmApplication = writeTextFile { name = "gnome-flashback-${wmName}-wm";

Related discussion:
text: Panel applets are being picked up from environment variable in and a completely hardcoded path in . First, we will need to patch the latter manager to use an environment variable as well. Second, we will need to do something similar to what . Or better, create  derivation. Originally posted by in
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 . on the on .
I'd like to tackle this, do you have any pointers? We need to get it to pick up at least right?
Code change:
pkg; in lib.makeExtensible (self: ({ <del> nix_2_3 = (common rec { version = "2.3.16"; src = fetchurl { url = "https://nixos.org/releases/nix/nix-${version}/nix-${version}.tar.xz"; hash = "sha256-fuaBtp8FtSVJLSAsO+3Nne4ZYLuBj2JpD2xEk7fCqrw="; }; </del> <ins> nix_2_3 = (common { version = "2.3.17"; hash = "sha256-EK0pgHDekJFqr0oMj+8ANIjq96WPjICe2s0m4xkUdH4="; </ins> patches = [ patch-monitorfdhup ]; <ins> maintainers = with lib.maintainers; [ flokli raitobezarius ]; </ins> }).override { boehmgc = boehmgc-nix_2_3; }; nix_2_10 = common {

Related discussion:
text: Package name: nix23 Latest released version: 2.3.17 (see ) Current version on the unstable channel: 2.3.16 Current version on the stable/release channel: 2.3.16 Notify maintainers Note for maintainers: Please tag this issue in your PR. See for context. The release script apparently doesn't upload source tarballs anymore, so 2.3.17 sources look more like the other ones. There's a bunch of expressions in that might need to be revisited.
Code change:
--replace "clvm_rs==0.1.8" "clvm_rs>=0.1.8"  --replace "clvm==0.9.7" "clvm>=0.9.7"  --replace "bitstring==3.1.7" "bitstring>=3.1.9"  <ins> ln -sf ${cacert}/etc/ssl/certs/ca-bundle.crt mozilla-ca/cacert.pem </ins> ''; preCheck = ''

Related discussion:
text: chia farmer fails to register with a pool over https. Relevant log contains: Steps to reproduce the behavior: a new-style chia plot with a pool up a farming node Chia should successfully run. If applicable, add screenshots to help explain your problem. Based on , the missing file is . Upstream normally but this appears to fail here. (This is in an LXC container so it's a bit weird) system: host os: multi-user?: sandbox: version: nixpkgs: Maintainer information:
Code change:
example = [ "/home" "/etc" "/var/log/syslog*" "/home/user/path with spaces" ]; }; repositories = mkOption { <del> type = nullOr (listOf repository); default = null; </del> <ins> type = listOf repository; default = []; </ins> description = '' A required list of local or remote repositories with paths and optional labels (which can be used with the --repository flag to

Related discussion:
text: I've set up a borgmatic backup script using the borgmatic service. I can run the backup manually by running as root, but fails with the following error: Here is the service definition I used: I then tried to pass an absolute path to cat, by replacing with and got the following similar error: Steps to reproduce the behavior: one of the service definitions above and observe that running it manually works. the output in No errors, as both and are present on the system, and I can run I also noticed that borgmatics default value for , , causes an error when the borgmatic service is run. Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
As I know, NixOS package doesn't use your settings, which mean what you can use in your shell environment doesn't mean the package can use. User can use and whatever command they like in , I can't put everything in . So my suggestion is to use the format when using a command provided by other packages. As for your pgdump command not found, I suggest change it into Good caught, I will file a PR to change the default value.
This works, but I'm curious why it's not the default? As is, the service doesn't work for postgres dumps?
If we make this default, I think it will add those packages (pkgs.{mysql,postgresql,mariadb}) to nix store even the user doesn't need it. And I prefer to keep the package in minimal. Personally, I use the following code to use mysqldump
That's fair. Thanks for the feedback :-) Do we leave this issue open until has been merged?
You can keep it open, it will auto close once merged.
Code change:
{ lib , stdenv <del> , fetchgit </del> <ins> , fetchFromGitHub </ins> , cmake , pkg-config , libusb1

Related discussion:
text: <!-- 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: rtl-sdr <!-- 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: 0.6.0 desired version: 0.8.0 <!-- Search your package here: If no maintainer is listed for your package, tag the person that last updated the package --maintainers: This package is currenty using but it should probably switch to Please tag this issue in your PR.
Code change:
in stdenv.mkDerivation rec { pname = "vulkan-validation-layers"; <del> version = "1.2.189.1"; </del> <ins> version = "1.2.198.0"; </ins> # If we were to use "dev" here instead of headers, the setupHook would be # placed in that output instead of "out". outputs = ["out" "headers"]; outputInclude = "headers"; <del> src = (assert version == vulkan-headers.version; </del> <ins> src = (assert (lib.all (pkg: pkg.version == version) [vulkan-headers glslang spirv-tools spirv-headers]); </ins> fetchFromGitHub { owner = "KhronosGroup"; repo = "Vulkan-ValidationLayers"; rev = "sdk-${version}"; <del> sha256 = "0a5plvvffidgnqh5ymq315xscl08w298sn9da48b3x2rdbdqgw90"; </del> <ins> sha256 = "sha256-/pnXT55EQZcnjOzY2vBwp+gM6l2hktZHwB9yKP8vVTU="; </ins> }); # Include absolute paths to layer libraries in their associated

Related discussion:
text: name: All packages in current version: 1.2.189.1 desired version: 1.2.200 maintainers: Please tag this issue in your PR.
Code change:
<ins> { lib , stdenv , fetchFromGitHub , autoreconfHook }: stdenv.mkDerivation rec { pname = "xmake-core-sv"; version = "1.1"; src = fetchFromGitHub { owner = "xmake-io"; repo = pname; rev = "v${version}"; hash = "sha256-icvGQi6FNSZXNGs2oLiUKu6rrVsWcXh1r91kycGjnwY="; }; nativeBuildInputs = [ autoreconfHook ]; meta = with lib; { description = "Public domain cross-platform semantic versioning in c99"; homepage = "https://github.com/xmake-io/xmake-core-sv"; license = licenses.unlicense; platforms = platforms.linux; maintainers = with maintainers; [ rewine ]; }; } </ins>

Related discussion:
text: Project description A lightweight cross-platform build utility based on Lua Metadata homepage URL: source URL: license: asl20 platforms: unix
Code change:
KDEDIRS = [ "" ]; QT_PLUGIN_PATH = [ "/lib/qt4/plugins" "/lib/kde4/plugins" ]; QTWEBKIT_PLUGIN_PATH = [ "/lib/mozilla/plugins/" ]; <del> GTK_PATH = [ "/lib/gtk-2.0" "/lib/gtk-3.0" ]; </del> <ins> GTK_PATH = [ "/lib/gtk-2.0" "/lib/gtk-3.0" "/lib/gtk-4.0" ]; </ins> XDG_CONFIG_DIRS = [ "/etc/xdg" ]; XDG_DATA_DIRS = [ "/share" ]; MOZ_PLUGIN_PATH = [ "/lib/mozilla/plugins" ]; LIBEXEC_PATH = [ "/lib/libexec" ]; }; <ins> environment.pathsToLink = [ "/lib/gtk-2.0" "/lib/gtk-3.0" "/lib/gtk-4.0" ]; </ins> environment.extraInit = '' unset ASPELL_CONF

Related discussion:
text: Upstream ibus-1.5.24 (in nixpkgs) supports gtk4. It will be good if we enable gtk4 support in nixpkgs. I fail to enable it with the following patch.
Code change:
propagatedBuildInputs = [ babel python-dateutil <ins> python-jose </ins> html2text lxml packaging

Related discussion:
text: When used through the command line, complains about imports and fails. <!-- 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]:
I've opened but haven't had the time to test it quite yet. If you get around to it before me feel free to merge it
I suggest another fix in
Code change:
nativeBuildInputs = [ zip makeWrapper ]; <ins> # https://github.com/NixOS/nixpkgs/issues/120738 patches = [ ./tsh.patch ]; </ins> postBuild = '' pushd . mkdir -p build

Related discussion:
text: Describe the bug In order to be able to invoke the command from teleport is using  call. There is a wrapper around to bring into PATH . That means is invoked from without wrapper and the browser page never opens (due to missing ). To Reproduce Steps to reproduce the behavior: Expected behavior The path to the wrapper must be persisted in the Screenshots If applicable, add screenshots to help explain your problem. Additional context Notify maintainers <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --Metadata Please run and paste the result. Maintainer information:
Code change:
hash = "sha256-yO9/HlUkaQmW/n9N3vboHw//YMzBjxIHA2zAxgZNEv0="; }; <ins> patches = [ # Support rule files in NixOS store paths. Required for the networkd-dispatcher # module to work ./support_nix_store_path.patch ]; </ins> postPatch = '' # Fix paths in systemd unit file substituteInPlace networkd-dispatcher.service 

Related discussion:
text: refuses to work since it requires the directories containing the scripts to have mode 755, but the nixos module creates them with mode 750. Steps to reproduce the behavior: (re)-configure an interface (for example by unplugging and replugging a network cable) something like in the journal and that does not exist, i.e. the hook was never run. networkd-dispatcher works. <!-- 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.
Thank you for the report! I'm thinking about adding a rules option to the networkd-dispatcher module so this might solve your issue
Code change:
<ins> { libsForQt5 , stdenv , lib , fetchFromGitHub , cmake , nix-update-script , fetchpatch , grim , makeBinaryWrapper , enableWlrSupport ? false }: stdenv.mkDerivation { pname = "flameshot"; # wlr screenshotting is currently only available on unstable version (>12.1.0) version = "12.1.0-unstable-2024-07-02"; src = fetchFromGitHub { owner = "flameshot-org"; repo = "flameshot"; rev = "ccb5a27b136a633911b3b1006185530d9beeea5d"; hash = "sha256-JIXsdVUR/4183aJ0gvNGYPTyCzX7tCrk8vRtR8bcdhE="; }; patches = [ # https://github.com/flameshot-org/flameshot/pull/3166 # fixes fractional scaling calculations on wayland (fetchpatch { name = "10-fix-wayland.patch"; url = "https://github.com/flameshot-org/flameshot/commit/5fea9144501f7024344d6f29c480b000b2dcd5a6.patch"; hash = "sha256-SnjVbFMDKD070vR4vGYrwLw6scZAFaQA4b+MbI+0W9E="; }) ]; passthru = { updateScript = nix-update-script { extraArgs = [ "--version=branch" ]; }; }; cmakeFlags = [ (lib.cmakeBool "USE_WAYLAND_CLIPBOARD" true) (lib.cmakeBool "USE_WAYLAND_GRIM" enableWlrSupport) ]; nativeBuildInputs = [ cmake libsForQt5.qttools libsForQt5.qtsvg libsForQt5.wrapQtAppsHook makeBinaryWrapper ]; buildInputs = [ libsForQt5.qtbase libsForQt5.kguiaddons ]; dontWrapQtApps = true; postFixup = '' wrapProgram $out/bin/flameshot  ${lib.optionalString enableWlrSupport "--prefix PATH : ${lib.makeBinPath [ grim ]}"}  ''${qtWrapperArgs[@]} ''; meta = with lib; { description = "Powerful yet simple to use screenshot software"; homepage = "https://github.com/flameshot-org/flameshot"; mainProgram = "flameshot"; maintainers = with maintainers; [ scode oxalica ]; license = licenses.gpl3Plus; platforms = platforms.linux ++ platforms.darwin; }; } </ins>

Related discussion:
text: -Krey, make a new issue?
Sure, but I don't know if upstream even supports mutter; doesn't seem like it based on , , etc.
For context it's currently broken, but it worked with the fix in until recently, so thought it was relevant to this.
Code change:
patches = [ # https://github.com/neomutt/neomutt/issues/3773#issuecomment-1493295144 ./fix-open-very-large-mailbox.patch <ins> (fetchpatch { name = "disable-incorrect-tests.patch"; url = "https://github.com/neomutt/neomutt/pull/3933.patch"; hash = "sha256-Plei063T8XyXF4/7/nAb6/4OyXz72vBAXHwls9WL1vM="; excludes = [".github/workflows/macos.yml"]; }) </ins> ]; buildInputs = [

Related discussion:
text: Steps to reproduce the behavior: on darwin aarch64 see Since neomutt is broken on darwin-aarch64. See the CI output: Tangent: is it normal that a failing CI build wasn't marked as a failed check, but rather just ignored? <!-- 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.
Broken on Darwin x86-64 as well.
seems to work — I can't tell whether they are running the tests, though, or not. Here are the failing tests.
Looking at the latest comment on the already closed issue here: This issue could use some love. Neomutt is broken on MacOS and it looks like it's the derivation, not upstream.
I can't install neomutt on MacOS with home-manager too
The failing tests were disabled upstream in . I tried simply upgrading to HEAD with the following patch, and the succeeded (darwin-aarch64):
( That's a different bug which you should probably raise separately.)
Unfortunately, the latest fix didn't work for my Intel Mac, some tests are failing for me. I tested using because I didn't know how to do overrides from the CLI
Yes, the failure on should be a separate bug (and could probably be fixed by putting into ).
Code change:
<ins> { stdenv, lib, fetchFromGitHub, qmake, qtbase, wrapQtAppsHook }: stdenv.mkDerivation { pname = "selectdefaultapplication"; version = "unstable-2021-08-12"; src = fetchFromGitHub { owner = "sandsmark"; repo = "selectdefaultapplication"; rev = "c752df6ba8caceeef54bcf6527f1bccc2ca8202a"; sha256 = "C/70xpt6RoQNIlAjSJhOCyheolK4Xp6RiSZmeqMP4fw="; }; nativeBuildInputs = [ qmake wrapQtAppsHook ]; buildInputs = [ qtbase ]; installPhase = '' runHook preInstall mkdir -p $out/bin cp selectdefaultapplication $out/bin install -Dm644 -t "$out/share/applications" selectdefaultapplication.desktop install -Dm644 -t "$out/share/icons/hicolor/48x48/apps" selectdefaultapplication.png runHook postInstall ''; meta = with lib; { description = "A very simple application that lets you define default applications on Linux in a sane way"; homepage = "https://github.com/sandsmark/selectdefaultapplication"; maintainers = with maintainers; [ nsnelson ]; license = licenses.gpl2; platforms = platforms.linux; }; } </ins>

Related discussion:
text: Project description  Selectdefaultapplication is a very simple GUI application that lets you define default applications on Linux in a sane way. Basically it just loads all installed applications by reading their .desktop files, reads the MimeType fields to see what it supports, and updates with what the user wants. Not all desktop environments provide such a tool, prompting for an agnostic tool. Metadata homepage URL: source URL: license: gpl2 platforms: linux Additional info: AUR package:
You can try using this in an overlay, use .
Seems to work!
Code change:
stdenv.mkDerivation rec { pname = "spirv-headers"; <del> version = "unstable-2021-08-11"; </del> <ins> version = "1.2.198.0"; </ins> src = fetchFromGitHub { owner = "KhronosGroup"; repo = "SPIRV-Headers"; <del> rev = "e71feddb3f17c5586ff7f4cfb5ed1258b800574b"; sha256 = "sha256-9m0EBcgdya+KCNJHC3x+YV2sXoSNToTcgDkpeKzId6U="; </del> <ins> rev = "sdk-${version}"; sha256 = "sha256-cdEiRSCoX0New8ecUh7UTDz/is2v29zhf6Il2N1j3mw="; </ins> }; nativeBuildInputs = [ cmake ];

Related discussion:
text: name: All packages in current version: 1.2.189.1 desired version: 1.2.200 maintainers: Please tag this issue in your PR.