jiebi/CodeConvo huggingface dataset:
Change:
}; <del> config = mkIf cfg.enable { users.groups = mkMerge [ (mkIf isMLocate { mlocate = { }; }) (mkIf isPLocate { plocate = { }; }) </del> <ins> config = lib.mkIf cfg.enable { users.groups = lib.mkMerge [ (lib.mkIf isMLocate { mlocate = { }; }) (lib.mkIf isPLocate { plocate = { }; }) </ins> ]; security.wrappers =

Discussion:
Using more-or-less the default configuration for (with ), the list of results is printed twice whenever running . For example: Steps to reproduce the behavior: the module like so: the job with some file name that exists on the system Matching files should each be printed once. The issue is fixed if I unset the environment variable. plocate deliberately reads from both and the hard coded default path, even if they are the same: The comment claims that this is also the behavior of mlocate, but I do not have an mlocate database installed and haven't wanted to change my running system to use mlocate. It looks like recently started being set in NixOS in . Please run and paste the result.
Sounds to me like an upstream issue. Can you write a mail to the upstream maintainer? ;a=blob;f=README;h=;hb=HEAD#l40
Sure, I have done so. I'll follow up when I get a response.
The plocate maintainer says they do not consider this to be a bug in plocate, and is intentional behavior. Rather than setting the environment variable, could we just make a wrapper script for that uses the NixOS-configured dbpath, if we don't want to re-build itself when the path is changed?
I would be personally fine to just drop support for a custom place for the db. If people really want to have it somewhere else they could symlink it. Then we don't need a wrapper or to worry about the env.
I can confirm the behaviour with .
Could we just take out the variable setting? It isn't needed for or , and just causes this annoying behaviour. If it's needed for , maybe the setting could be made conditional. It would be nice to get something done about it sooner rather than later. This issue has been open for 4 months now.
Please take a look at
I've reviewed the PR and it LGTM. Thanks for doing this! It will be great to have this fixed.
Change:
<ins> golang.org/x/crypto v0.25.0 h1:ypSNr+bnYL2YhwoMt2zPxHFmbAN1KZs/njMG3hxUp30= golang.org/x/crypto v0.25.0/go.mod h1:T+wALwcMOSE0kXgUAnPAHqTLW+XHgcELELW8VaDgm/M= </ins>

Discussion:
Steps to reproduce the behavior: ncdns.goModules with or 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]:
The goModules derivation can't be built since go 1.17, see I don't think I can fix it.
Change:
}; enableParallelBuilding = true; <del> inherit enableMultilib; </del> <ins> inherit enableShared enableMultilib; </ins> inherit (stdenv) is64bit;

Discussion:
EDIT: I am also getting this now trying to build in CI under Linux To Reproduce Screenshots 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 Maintainer information:
Looks like this is also similar to and and
As I mentioned in the audiofile ticket, my attempted workaround is this: However this doesn't appear to work because the .la files are not generated by the time of the preBuild hook. Can anyone help with this?
This issue has been mentioned on NixOS Discourse. There might be relevant details there:
I marked this as stale due to inactivity. &rarr;
Change:
{ stdenvNoCC, lib, fetchFromGitHub, makeWrapper <del> , python3, binutils-unwrapped, findutils, kmod, pciutils, libraspberrypi </del> <ins> , python3, binutils-unwrapped, findutils, gawk, kmod, pciutils, libraspberrypi </ins> }: stdenvNoCC.mkDerivation rec { pname = "raspberrypi-eeprom";

Discussion:
Steps to reproduce the behavior: inside an activation script. Looks like is not present in wrapper's PATH.
Change:
<ins> { lib, ruby, bundlerApp, bundlerUpdateScript }: bundlerApp { pname = "serverspec"; gemdir = ./.; inherit ruby; exes = ["serverspec-init"]; passthru.updateScript = bundlerUpdateScript "serverspec"; meta = with lib; { description = "RSpec tests for your servers configured by CFEngine, Puppet, Ansible, Itamae or anything else"; homepage = "https://serverspec.org/"; license = licenses.mit; maintainers = with maintainers; [ dylanmtaylor ]; }; } </ins>

Discussion:
Project description <!-- Describe the project a little: --RSpec tests for your servers configured by CFEngine, Puppet, Chef, Ansible, Itamae or anything else even by hand homepage URL: source URL: license: mit platforms: unix
Change:
```nix { lib, buildNimPackage, fetchFromGitHub }: <del> buildNimPackage { } (finalAttrs: { </del> <ins> buildNimPackage (finalAttrs: { </ins> pname = "ttop"; version = "1.2.7";

Discussion:
buildNimPackage example in unstable docs is wrong Should be I am not sure, since doc mismatched, if it was intended to be that way or not. <!-- make sure this issue is not redundant or obsolete --- [x] checked [latest Nixpkgs manual] ([source][nixpkgs-source]) and [latest NixOS manual] ([source][nixos-source]) [x] checked [open documentation issues] for possible duplicates [x] checked [open documentation pull requests] for possible solutions [latest Nixpkgs manual]: [latest NixOS manual]: [nixpkgs-source]: [nixos-source]: [open documentation issues]: [open documentation pull requests]: Related PR: Introduced by Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
dpkg, nss, alsa-lib, <ins> lz4, libgcrypt, </ins> xkeyboard_config, libthai, libsForQt5,

Discussion:
Steps to reproduce the behavior: Eventually nixos-rebuild fails and exits with the last few lines regarding the package build fail: Please run and paste the result. Could you please check it? Can this be related to the following . By the way - the latest available version seems to be - could you please upgrade to it and possibly it will resolve it?
Thanks for the heads-up, I will fix it when I am upgrading my nixpkgs fork, soon.
Please test and upvote:
Change:
# t/169_import_scan.t ........................... Dubious, test returned 1 (wstat 256, 0x100) rm t/169_import_scan.t <del> # Disable a test which passes but reports an incorrect status # t/0601_Dialog_Scan.t .......................... All 14 subtests passed # t/0601_Dialog_Scan.t                        (Wstat: 139 Tests: 14 Failed: 0) #   Non-zero wait status: 139 rm t/0601_Dialog_Scan.t </del> <ins> # Disable a test failing because of a warning interfering with the pinned output # t/3722_user_defined.t ......................... 1/2 #   Failed test 'user_defined caught error injected in queue' #   at t/3722_user_defined.t line 41. #          got: 'error # WARNING: The convert command is deprecated in IMv7, use "magick" instead of "convert" or "magick convert"' #     expected: 'error' # Looks like you failed 1 test of 2. rm t/3722_user_defined.t export XDG_CACHE_HOME="$(mktemp -d)" </ins> xvfb-run -s '-screen 0 800x600x24'  make test '';

Discussion:
: can not open Edit-Preferences : can not open Edit-Preferences A dialogue should happen This is the debug log: Add any other context about the problem here. Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Hi! I'm afraid I could not reproduce the issue; the Preferences dialogue shows up fine for me. I tried with on NixOS 23.11 with Sway.
Thank you for your fast reply. Does the dialogue connect to some service that must be running?
Quoting Jens Getreu (2024-03-03 14:46:03) The whole app might be trying to connect to SANE when it starts, but I don't think anything is needed for that particular dialogue.
works fine. All other dialogues open correctly. When I try to open Preferences the second time, a tiny little empty window opens. On the console the above error is dumped.
I am running the i3 window manager. Gnome has a settings deamon. I tried: with but it did not help. I still get and no Edit-Preferences dialogue appears.
I'm having the same issue - gscan2pdf v2.13.2, GNOME 45.3, kernel 6.6.16-1-lts
Same for me on KDE on current master with v2.13.2: EDIT: Upstream seems to be ware of the issue:
This should be fixed with Please test and review.
Change:
(mkFlag ppaSupport "ENABLE_PPA") (mkFlag vtuneSupport "ENABLE_VTUNE") (mkFlag werrorSupport "WARNINGS_AS_ERRORS") <del> ]; </del> <ins> ] # Clang does not support the endfunc directive so use GCC. ++ lib.optional (stdenv.cc.isClang && !stdenv.targetPlatform.isDarwin) "-DCMAKE_ASM_COMPILER=${gccStdenv.cc}/bin/${gccStdenv.cc.targetPrefix}gcc"; </ins> cmakeStaticLibFlags = [ "-DHIGH_BIT_DEPTH=ON"

Discussion:
Steps to reproduce the behavior: Add any other context about the problem here. <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. -- Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
{ buildPythonPackage <del> , cudaSupport ? torch.cudaSupport or false # by default uses the value from torch </del> , fetchFromGitHub , lib , libjpeg_turbo

Discussion:
I discovered today that overriding torchvision with does not work. At least not without making corresponding changes to torch. For example, We should either remove this argument and pull from torch or improve the ergonomics here somehow. cc Checkout nixpkgs at a relatively recent master commit, eg , and then run
can you test if fixes the problem? To be honest, including the argument for was a mistake in the first place -- it should (must?) use whatever was built with, so it should be inherit directly.
Maybe assert torch.cudaSupport is more explicit? Idk
I'm not super comfortable with asserts in the derivation because (afaik) they can't be overridden. Thoughts on me updating to ensure either both are compiled with CUDA, or neither are?
I understand it's a compromise. IIRC if we set here, we'll just exclude the broken package from ofborg and nixpkgs-review, contrary to the purpose.
Why do we need to be asserting or marking things as broken depending on ? shouldn't it build fine either way?
So the question is, should use , or should it accept as an argument and then possibly . I think that silently using may be surprising to people, but then again we're discussing some furnish while the thing is broken xD
ohhh i see, yeah that's tricky. i could envision someone creating a nix-shell with both and and then being very surprised to learn that is not built with CUDA support

From a brief look at the updated cuda section in the manual, it would be logical to take as an argument, rather than the compiled string. And to expose that in via gets even further beyond the scope, so let's skip it Originally posted by in
This issue has been mentioned on NixOS Discourse. There might be relevant details there:
Change:
openJpegVersion = with stdenv; lib.versions.majorMinor (lib.getVersion openjpeg); <ins> freeglut-mupdf = freeglut.overrideAttrs (old: rec { pname = "freeglut-mupdf"; version = "3.0.0-r${src.rev}"; src = fetchFromGitHub { owner = "ArtifexSoftware"; repo = "thirdparty-freeglut"; rev = "13ae6aa2c2f9a7b4266fc2e6116c876237f40477"; hash = "sha256-0fuE0lm9rlAaok2Qe0V1uUrgP4AjMWgp3eTbw8G6PMM="; }; }); </ins> in stdenv.mkDerivation rec {

Discussion:
is built without clipboard or unicode input support. It prints the following warnings to the console Steps to reproduce the behavior: and observe the CLI output. does not output any warnings and supports clipboard Used mupdf from its last nixpkgs change
It looks like that feature needs the thirdparty freeglut:
Fixed by /
Change:
<xi:include href="android.section.xml" /> <xi:include href="beam.section.xml" /> <xi:include href="bower.xml" /> <del> <xi:include href="coq.xml" /> </del> <ins> <xi:include href="coq.section.xml" /> </ins> <xi:include href="crystal.section.xml" /> <xi:include href="emscripten.section.xml" /> <xi:include href="gnome.xml" />

Discussion:
This issue tracks the conversion of from Docbook to CommonMark per . If you intend to work on this conversion, please comment below! in this issue that you intend to work on it. (Please also post if you stop working on it) a new branch of nixpkgs master. : change to . to . the contents from Docbook to . Use existing sections for inspiration. from the folder to build the manual. the manual on your computer at after running from folder: it looks good, submit a PR and reference this issue.
I can also look at this.
Change:
hash = "sha256-CHL2dy0H/i0pLo653F7aUHFvZHTeZA6jC/rwn1KrEW4="; }; <ins> patches = [ # https://github.com/NixOS/nixpkgs/issues/288478 (fetchpatch { name = "set-default-encoding.patch"; url = "https://github.com/joeyespo/grip/commit/2784eb2c1515f1cdb1554d049d48b3bff0f42085.patch"; hash = "sha256-veVJKJtt8mP1jmseRD7pNR3JgIxX1alYHyQok/rBpiQ="; }) ]; </ins> nativeCheckInputs = [ pytest responses ]; propagatedBuildInputs = [ docopt flask markdown path-and-address pygments requests tabulate ];

Discussion:
Steps to reproduce the behavior: in configuration unstable to: Add any other context about the problem here.
It looks like this has already been fixed upstream in There isn't a tagged release containing this fix though (it seems like there was a 4.6.2 but it's gone now: ), so maybe it just needs pulling in as a patch?
cc
Went ahead and pulled it in as a patch: , And opened a PR:
meta: mentioning to enable searches by that term. (I didn't know the FQDN of the package, so I couldn't find this issue).
Thanks for the quick fix! Verified on unstable now.
Change:
if (-e $prevUnitFile && ($state->{state} eq "active" || $state->{state} eq "activating")) { if (! -e $newUnitFile || abs_path($newUnitFile) eq "/dev/null") { <del> my %unitInfo = parseUnit($prevUnitFile); </del> <ins> my %unitInfo = parse_unit($prevUnitFile); </ins> $unitsToStop{$unit} = 1 if parseSystemdBool(%unitInfo, "Unit", "X-StopOnRemoval", 1); } elsif ($unit =~ /.target$/) { <del> my %unitInfo = parseUnit($newUnitFile); </del> <ins> my %unitInfo = parse_unit($newUnitFile); </ins> # Cause all active target units to be restarted below. # This should start most changed units we stop here as

Discussion:
Switching NIxOS configurations when using produces the following warning: Steps to reproduce the behavior: (current master) file in the directory with the following content: test fails and produces the warning should not produce warnings. It looks like currently can't handle service file names containing a . I tracked the first failing commit down to . The warning seems to not create any problems for me at the moment and all services are running.
Great… I'll take a look
Change:
typer = callPackage ../development/python-modules/typer { }; <ins> types-colorama = callPackage ../development/python-modules/types-colorama { }; </ins> types-dateutil = callPackage ../development/python-modules/types-dateutil { }; types-decorator = callPackage ../development/python-modules/types-decorator { };

Discussion:
Project description From the website : "This program takes upon itself the hard task of beautifying Bash scripts" Metadata homepage URL: source URL: license: MIT platforms: any
Change:
url = "https://github.com/fail2ban/fail2ban/commit/5ac303df8a171f748330d4c645ccbf1c2c7f3497.patch"; sha256 = "sha256-aozQJHwPcJTe/D/PLQzBk1YH3OAP6Qm7wO7cai5CVYI="; }) <ins> # fix use of MutableMapping with Python >= 3.10 # https://github.com/fail2ban/fail2ban/issues/3142 (fetchpatch { url = "https://github.com/fail2ban/fail2ban/commit/294ec73f629d0e29cece3a1eb5dd60b6fccea41f.patch"; sha256 = "sha256-Eimm4xjBDYNn5QdTyMqGgT5EXsZdd/txxcWJojXlsFE="; }) </ins> ]; preConfigure = ''

Discussion:
Fail2ban fails to launch Please run and paste the result.
Appears to be an upstream issue involving Python 3.9:
Change:
{ lib , fetchFromGitHub <ins> , glib , gsettings-desktop-schemas </ins> , python3 , unstableGitUpdater <ins> , wrapGAppsHook </ins> }: python3.pkgs.buildPythonApplication rec {

Discussion:
Using the file manager in chirp (specifically the version used in nixpkgs-unstable) causes the program to crash Console errors: Similar issues seem to have been fixed with wrapGAppsHook, but I can't seem to figure it out. Steps to reproduce the behavior: from nixpkgs-unstable to open a file via the file-open dialogue, or try to save a file. The program does not crash, allows file saving and loading. None listed so pinging last few contributors: Please run and paste the result. system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
I can't reproduce the issue on both GNOME 44 and Xfce 4.18, so I'm not sure what the solution is. But maybe adding both and to would help?
Using i3wm myself. Adding to the package (with the appropriate argument lines at the top) results in the same error
Oh! Adding wrapGAppshook to nativebuildInputs as well as the above change seems to fix it.
Change:
{ stdenv, fetchurl, pkgconfig, wayland, mesa, libxkbcommon, cairo, libxcb , libXcursor, x11, udev, libdrm, mtdev, libjpeg, pam, dbus, libinput , pango ? null, libunwind ? null, freerdp ? null, vaapi ? null, libva ? null <del> , libwebp ? null </del> <ins> , libwebp ? null, xwayland ? null </ins> }: let version = "1.6.0"; in

Discussion:
A in the xwayland package output causes nixos-rebuild to fail due to permission issues, since /var/tmp contains private directories (mode 700) System: NixOS 16.09 Nix version: 1.11.4 Nixpkgs version: 16.09.901.
Essentially the same problem as in vanilla X server:
found some more info on the problem the issue, is that both wayland and xorg have the same "fix" for packages wanting to write to the store, at the same place /nix/store/dd65ni8yzibvq2aj5jdylnv557i3hcd1-xwayland-1.18.4/share/X11/xkb/compiled and /nix/store/v82zrx8dnm5j8lq4a2fc5g0rcx2jx04k-xorg-server-1.18.4/share/X11/xkb/compiled are "conflicting" so buildEnv tries to cd into both, find every file within, and then create a compiled directory with symlinks to merge the 2 except, both versions of compiled are symlinks to /var/tmp in this case, there is no need to merge the contents of "compiled" since its 2 identical symlinks that are conflicting, so id say buildEnv should be patched and i have a simple testcase:
I'm guessing that this works OK when built in a sandbox?
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 .
As both the referenced packages build now, I guess it's safe to close this.
Change:
foo-yc20 = callPackage ../applications/audio/foo-yc20 { }; <del> fossil = callPackage ../applications/version-management/fossil { }; </del> <ins> fossil = callPackage ../applications/version-management/fossil { sqlite = sqlite.override { enableDeserialize = true; }; }; </ins> freebayes = callPackage ../applications/science/biology/freebayes { };

Discussion:
, CVSSv3=7.5 (nixos-unstable) Fossil before 2.14.2 and 2.15.x before 2.15.2 often skips the hostname check during TLS certificate validation. Scanned versions: nixos-unstable: . Cc Cc
Change:
in python3.pkgs.buildPythonApplication rec { pname = "matrix-synapse"; <del> version = "1.111.0"; </del> <ins> version = "1.112.0"; </ins> format = "pyproject"; src = fetchFromGitHub { owner = "element-hq"; repo = "synapse"; rev = "v${version}"; <del> hash = "sha256-CgoJJK2pqkHU8X6oisY19uN6zyjGL8W3irTsraFOYQM="; </del> <ins> hash = "sha256-8iXw9C91kPWDlzo/3AA/iVCQqq47eGSORMTzEQTTS+8="; </ins> }; cargoDeps = rustPlatform.fetchCargoTarball { inherit src; name = "${pname}-${version}"; <del> hash = "sha256-uKyy2m3bvo6U++Qx6t7maeIp84QfMzslPGV1so4ZT3U="; </del> <ins> hash = "sha256-hx/IMOxk4vUHXMMIcnxnC3RJcIvJL+IooZnf+m+VKSs="; </ins> }; postPatch = ''

Discussion:
Package name: matrix-synapse Latest released version: v1.112.0 <!-- Search your package here: --- Current version on the unstable channel: 1.111.0 Current version on the stable/release channel: 1.111.0 <!-- 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 Note for maintainers: Please tag this issue in your PR. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
They released 1.112.0 very quickly after 1.111.1, so I changed the issue.
Change:
, qtwebsockets , qtquickcontrols2 , qtgraphicaleffects <del> , libmlt </del> , qmake , qttools , genericUpdater , common-updater-scripts }: <del> assert stdenv.lib.versionAtLeast libmlt.version "6.22.1"; </del> assert stdenv.lib.versionAtLeast mlt.version "6.22.1"; mkDerivation rec {

Discussion:
Shotcut in theory supports LADSPA plugins. However, LADSPA plugins are not found by Shotcut on NixOS version Install a bunch of LADSPA plugins (by listing them in ) Open Shotcut Add effects to a track Look for audio effects: no LADSPA plugins are shown Please run and paste the results.
cc maintainers
I tried to build shotcut and mlt with ladspa-sdk and set LADSPA_PATH, but did not manage to get the ladspa plugins working yet. ,-(
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 .
found out why: configure is using , which is not available, and therefore does disable ladspa. Preparing a PR.
Change:
<xi:include href="kakoune.section.xml" /> <xi:include href="linux.xml" /> <xi:include href="locales.xml" /> <del> <xi:include href="nginx.xml" /> </del> <ins> <xi:include href="nginx.section.xml" /> </ins> <xi:include href="opengl.section.xml" /> <xi:include href="shell-helpers.xml" /> <xi:include href="steam.xml" />

Discussion:
This issue tracks the conversion of from Docbook to CommonMark per . If you intend to work on this conversion, please comment below! here that you intend to work on it. (Please also post if you stop working on it.) a new branch of nixpkgs master. : change to . to . the contents from Docbook to . Follow . Use existing sections for inspiration. from the folder to build the manual. the manual on your computer at after running from folder: it looks good, submit a PR and reference this issue.
Hey. I would like to work on this