jiebi/CodeConvo huggingface dataset:
Change:
systemd.services."escapedx2ddash".serviceConfig.X-Test = "test"; }; <ins> unitWithMultilineValue.configuration = { systemd.services.test.serviceConfig.ExecStart = '' ${pkgs.coreutils}/bin/true  # ignored ; ignored blah blah ''; }; </ins> unitStartingWithDash.configuration = { systemd.services."-" = { wantedBy = [ "multi-user.target" ];

Discussion:
See title. It also may fail to correctly parse comments within such blocks as well, but unsure. a systemd unit in config e.g. to switch configuration (though this might only happen on the second switch, trying to parse existing config?) with Per , this is valid syntax: The value of should thus be parsed to (I may have gotten the spaces wrong there) If applicable, add screenshots to help explain your problem. Patch for failing example test case (may not be perfect): also due to authoring Please run and paste the result. I was running dry-activate to a config using nixpkgs @ Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
thanks for the report! I've looked a little into this so far and switch-to-configuration doesn't actually use that much of the unit file contents, which could be fine considering that systemd's DBus interface exposes almost all of their unit properties as DBus properties. However, since NixOS uses some custom unit properties (think ), these aren't exposed as DBus properties. So instead I've gone the route of adding support for multi-line values in the INI parser used for switch-to-configuration-ng, with the eventual goal of upstreaming this support to that crate. This isn't going to be something that's immediately ready though, I should have a PR for review within the coming days. For now, I would recommend opting out of the switch-to-configuration-ng implementation with the following config until it's fixed:
The problem with INI in general is that there is no standard. The systemd syntax, after all, “is inspired by XDG Desktop Entry Specification .desktop files, which are in turn inspired by Microsoft Windows .ini files”.
Change:
<del> { stdenv , lib , fetchurl , autoPatchelfHook , libidn , zlib , bzip2 }: let libidn11 = libidn.overrideAttrs (old: { pname = "libidn"; version = "1.34"; src = fetchurl { url = "mirror://gnu/libidn/libidn-1.34.tar.gz"; sha256 = "0g3fzypp0xjcgr90c5cyj57apx1cmy0c6y9lvw2qdcigbyby469p"; }; }); in stdenv.mkDerivation rec { pname = "sratoolkit"; version = "2.11.3"; src = fetchurl { url = "https://ftp-trace.ncbi.nlm.nih.gov/sra/sdk/${version}/sratoolkit.${version}-ubuntu64.tar.gz"; sha256 = "1590lc4cplxr3lhjqci8fjncy67imn2h14qd2l87chmhjh243qvx"; }; nativeBuildInputs = [ autoPatchelfHook ]; buildInputs = [ libidn11 zlib bzip2 stdenv.cc.cc.lib ]; sourceRoot = "sratoolkit.${version}-ubuntu64/bin"; installPhase = '' find -L . -executable -type f -! -name "*remote-fuser*" -exec install -m755 -D {} $out/bin/{} ; ''; meta = with lib; { homepage = "https://github.com/ncbi/sra-tools"; description = "SRA Toolkit and SDK from NCBI is a collection of tools and libraries for using data in the INSDC Sequence Read Archives"; license = licenses.ncbiPd; maintainers = with maintainers; [ thyol ]; platforms = [ "x86_64-linux" ]; }; } </del>

Discussion:
Package name: sratoolkit Latest released version: 3.1.1 Current version on the unstable channel: 2.11.3 Current version on the stable/release channel: 2.11.3 <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --- Notify maintainers <!-- If the result shows no maintainers, tag the person that last updated the package. --Note for maintainers: Please tag this issue in your PR. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
<del> diff --git a/libraries/cmake/source/augeas/gnulib/generated/linux/aarch64/lib/locale.h b/libraries/cmake/source/augeas/gnulib/generated/linux/aarch64/lib/locale.h index 4f9baece2..afe947956 100644 --- a/libraries/cmake/source/augeas/gnulib/generated/linux/aarch64/lib/locale.h +++ b/libraries/cmake/source/augeas/gnulib/generated/linux/aarch64/lib/locale.h @@ -48,9 +48,9 @@ /* NetBSD 5.0 mis-defines NULL.  */ #include <stddef.h> -/* Mac OS X 10.5 defines the locale_t type in <xlocale.h>.  */ +/* Mac OS X 10.5 defines the locale_t type in <locale.h>.  */ #if 1 -# include <xlocale.h> +# include <locale.h> #endif /* The definitions of _GL_FUNCDECL_RPL etc. are copied here.  */ diff --git a/libraries/cmake/source/augeas/gnulib/generated/linux/x86_64/lib/locale.h b/libraries/cmake/source/augeas/gnulib/generated/linux/x86_64/lib/locale.h index 4f9baece2..afe947956 100644 --- a/libraries/cmake/source/augeas/gnulib/generated/linux/x86_64/lib/locale.h +++ b/libraries/cmake/source/augeas/gnulib/generated/linux/x86_64/lib/locale.h @@ -48,9 +48,9 @@ /* NetBSD 5.0 mis-defines NULL.  */ #include <stddef.h> -/* Mac OS X 10.5 defines the locale_t type in <xlocale.h>.  */ +/* Mac OS X 10.5 defines the locale_t type in <locale.h>.  */ #if 1 -# include <xlocale.h> +# include <locale.h> #endif /* The definitions of _GL_FUNCDECL_RPL etc. are copied here.  */ </del>

Discussion:
Package name: Latest released version: <!-- Search your package here: --- Current version on the unstable channel: Current version on the stable/release channel: <!-- 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 /cc <!-- If the result shows no maintainers, tag the person that last updated the package. --Note for maintainers: Please tag this issue in your PR. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
NIX_LDFLAGS = optionalString selinuxSupport "-lsepol"; FORCE_UNSAFE_CONFIGURE = optionalString stdenv.hostPlatform.isSunOS "1"; <ins> NIX_CFLAGS_COMPILE = [] # Work around a bogus warning in conjunction with musl. ++ optional stdenv.hostPlatform.isMusl "-Wno-error" ++ optional stdenv.hostPlatform.isAndroid "-D__USE_FORTIFY_LEVEL=0"; </ins> # Works around a bug with 8.26: # Makefile:3440: *** Recursive variable 'INSTALL' references itself (eventually).  Stop.

Discussion:
<!-- 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: coreutils <!-- 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: 9.0 desired version: 9.1 <!-- Search your package here: If no maintainer is listed for your package, tag the person that last updated the package --maintainers: Please tag this issue in your PR.
I'm already on it :) might take some while before I do the pr because I want to build stdenv first
Change:
{ luarocks, fetchFromGitHub }: luarocks.overrideAttrs(old: { pname = "luarocks-nix"; <del> version = "2019-09-07"; </del> <ins> version = "2021-01-22"; </ins> src = fetchFromGitHub { owner = "nix-community"; repo = "luarocks-nix"; <del> rev = "73b8772e56fd39dfffda9e3b13e9eb31e93d5cde"; sha256 = "00jgshygw439pbaxg7yph3ijia6nid9r1br416wdbyl5wqhlhm1y"; </del> <ins> rev = "v3.5.0_nix"; sha256 = "sha256-Ea3PVkCaUPO/mvVZtHtD1G9T/Yom28M9oN6duY4ovHk="; </ins> }; <del> patches = [ ./darwin-3.1.3.patch ]; </del> })

Discussion:
I tried adding to , then running and noticed in the diff this still uses , and non-string URLs. To Reproduce Steps to reproduce the behavior: Expected behavior The diff shouldn't contain non-string URLs, and not use via . Notify maintainers
luarocks-nix needs to be updated. Its current state is messy (not clear which branch to use), I will update it.
Change:
nativeBuildInputs = [ qmake makeWrapper python3.pkgs.wrapPython ]; <del> patches = [ ./0001-replace-git-with-normal-python-package.patch ]; </del> postPatch = '' substituteInPlace deployment.pri  --replace '/usr/bin' "$out/bin"

Discussion:
Describe the bug On current nixos-unstable, yubioath-desktop starts up, but does not detect a connected yubikey and spews error messages to the terminal. I believe this is related to and might be fixable by updating the package. To Reproduce Steps to reproduce the behavior: a yubikey (4 nano); set and rebuild (optional, I believe) Expected behavior Would expect the key to be discovered and silence on the terminal. Screenshots Additional context (none) Notify maintainers Metadata Please run and paste the result. system: host os: multi-user?: sandbox: version: nixpkgs: `/nix/store/d7bsnyyw1wbmkv197krfd0fbaw1rqcpv-source Maintainer information:
Version 5.0.5 seems to include the fix and will probably be released soon. I'll update the package once it has made it to their release website.
Thank you! I could work around the issue by using: for now (this may break some of the things fixed in patches)
Looks like 5.0.5 is released:
tarballs haven't been published yet.
Change:
homepage = "https://www.jool.mx/"; description = "Fairly compliant SIIT and Stateful NAT64 for Linux - kernel modules"; platforms = platforms.linux; <ins> license = licenses.gpl2Only; </ins> maintainers = with maintainers; [ fpletz ]; }; }

Discussion:
<!-- 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: <!-- 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: 4.1.6 desired version: 4.1.7 <!-- Search your package here: If no maintainer is listed for your package, tag the person that last updated the package --maintainers: Please tag this issue in your PR.
Change:
preFixup = '' # The following libraries need libtiff.so.5, but nixpkgs provides libtiff.so.6 patchelf --replace-needed libtiff.so.5 libtiff.so $out/opt/kingsoft/wps-office/office6/{libpdfmain.so,libqpdfpaint.so,qt/plugins/imageformats/libqtiff.so,addons/pdfbatchcompression/libpdfbatchcompressionapp.so} <ins> patchelf --add-needed libtiff.so $out/opt/kingsoft/wps-office/office6/libwpsmain.so # Fix: Wrong JPEG library version: library is 62, caller expects 80 patchelf --add-needed libjpeg.so $out/opt/kingsoft/wps-office/office6/libwpsmain.so </ins> # dlopen dependency patchelf --add-needed libudev.so.1 $out/opt/kingsoft/wps-office/office6/addons/cef/libcef.so '';

Discussion:
can not use export to pdf function Steps to reproduce the behavior: a doc/docx file to pdf successfully export to pdf ! There might be a explain: <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Reproducible on my side. Will try to investigate and push a pr if feasible
pretty sure we already handle the dependency: as mentioned in the thread, can you try running followed by (or applicable command) to confirm thi actual cause?
I have tried to run with , here is the log: Seems like there is nothing to do about , maybe something else is broken in otherwise.
Never mind. When I tried to create link to on somewhere and running with , PDF export working like a charm. Maybe the workaround is not working as expected.
Can you try this pr and see whether this pr fix the issue for you?
crash
Fixed in latest push Feel free to drop a message if further issues appear.
ok, it works fine for me
Change:
<ins> { lib , buildPythonPackage , fetchFromGitHub , pythonOlder , requests }: buildPythonPackage rec { pname = "meilisearch"; version = "0.18.0"; format = "setuptools"; disabled = pythonOlder "3.7"; src = fetchFromGitHub { owner = "meilisearch"; repo = "meilisearch-python"; rev = "v${version}"; hash = "sha256-iIFTZKORCXr4mNeWBtbOPWXwORuTV/IKhLYkqFgd3Hw="; }; propagatedBuildInputs = [ requests ]; pythonImportsCheck = [ "meilisearch" ]; # Tests spin up a local server and are not mocking the requests doCheck = false; meta = with lib; { description = "Client for the Meilisearch API"; homepage = "https://github.com/meilisearch/meilisearch-python"; license = with licenses; [ mit ]; maintainers = with maintainers; [ fab ]; }; } </ins>

Discussion:
Project description Meilisearch is a search engine. It can be accessed through an API, for which there is a Python wrapper. Note: this packaging request is not for the server, just for the Python API wrapper. Metadata homepage URL: source URL: license: MIT platforms: Linux, Darwin
Change:
pkfix-helper.extraBuildInputs = [ ghostscript_headless ]; ps2eps.extraBuildInputs = [ ghostscript_headless ]; pst2pdf.extraBuildInputs = [ ghostscript_headless ]; <ins> tex4ebook.extraBuildInputs = [ html-tidy ]; </ins> tex4ht.extraBuildInputs = [ ruby ]; texlive-scripts.extraBuildInputs = [ gnused ]; texlive-scripts-extra.extraBuildInputs = [ coreutils findutils ghostscript_headless gnused ];

Discussion:
I'm trying to produce html and epub files from a LaTeX document with the ht tools and it looks like they don't work properly. Given the following simple input, with a PDF file , the commands and give the following errors In both cases the output is produced, but the image is missing. <!-- 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.
How did you install TeXLive / which version of TeXLive do you have installed? It looks like you have 2 commands missing: and . I ran the same commands (on your test file) and only got the second error which was easily fixed for me by installing the package from the repos (seems to be an optional dependency, so not part of TeXLive per se). seems to be part of all the current unstable TeXLive packages, but none of the 23.05 TeXLive packages.
Indeed I'm on the 20.05 channel. I installed with the following in my NixOS is not a problem, as you mentioned. It's just that causes the problem. The TeXLive version is (in nixpkgs). I see the following in the store .
Ok thanks. Just to confirm, are you on or did you mean to say ?
Yes, sorry. I'm on .
cc this must have been fixed by the recent changes to scripts packaging? Is this is a wontfix for 23.05?
The script should work now on unstable, I see listed among the binaries on However is most likely an external dependency that must be to the path via .
PS: I'll try slip an extra link in the 23.05 , since 23.11 is still some time away. (It should be enough to add to the loop that reads from – not in directly as it would cause a useless rebuild – so if somebody has a moment to do it before me, go ahead!)
Thanks for the rapid fix!
Change:
environment.variables.QT_QPA_PLATFORMTHEME = cfg.platformTheme; <del> environment.variables.QT_STYLE_OVERRIDE = mkIf (! (isQt5ct || isLxqt)) cfg.style; </del> <ins> environment.variables.QT_STYLE_OVERRIDE = mkIf (! (isQt5ct || isLxqt || isKde)) cfg.style; </ins> environment.systemPackages = packages;

Discussion:
The reason why I didn't open a PR, is described in . I'm aware of four platforms for Qt's themes right now: Using GTK themes through + ; Using GNOME themes through + ; Using Qt themes, with the non-KDE solution , which uses ; Using Plasma-targeted Qt themes, with as , as done natively in a plasma session, using the theme's colorscheme. Steps to reproduce the behavior:  and it does not provide ; equivalent to  providing ; , then add some Pasma-target themes like , then adjust envvars , set the theme in or with , you'll get the right theme with the proper color scheme. should provide (or ) as an option, it should add only the required packages to the system, and set the proper envvars. (or maybe rename it, since qt6 also uses the same variables and envvars) qbittorrent + breeze + + dark: ! qbittorrent + breeze + + dark: ! qbittorrent + (ignored my theme): ! qbittorrent + (also ignored my theme): ! I tried to discover what package that is being when evaluating provides this platform-theme, but I failed miserably. stated as maintainers of plasma-integration: stated as maintainer of qt5ct: qt5 module contributors: cc Please run and paste the result.
Not exactly your bug, but we should remove and integrate it with configs instead.
is another QT platform theme that can be integrated into config.
Change:
{stdenv, vim, vimPlugins, vim_configurable, buildEnv, writeText, writeScriptBin <del> , nix-prefetch-scripts }: </del> <ins> , nix-prefetch-hg, nix-prefetch-git }: </ins> /*

Discussion:
It's frustrating to have to build all the version control systems (in master or another development branch without binary caches) when I just need to prefetch git :frowning:
Make the expression modular so you can disable those?
Implementation at:
Change:
platforms = platforms.linux; maintainers = with maintainers; [ mmahut rople380 Crafter ]; }; <del> } </del> <ins> }) </ins>

Discussion:
Package name: exodus Latest released version: <!-- Search your package here: --- Current version on the unstable channel: 24.13.3 Current version on the stable/release channel: 23.10.24 <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --- Notify maintainers <!-- If the result shows no maintainers, tag the person that last updated the package. -- Note for maintainers: Please tag this issue in your PR. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
# When upgrading the Nextcloud package, Nextcloud can report errors such as # "The files of the app [all apps in /var/lib/nextcloud/apps] were not replaced correctly" # Restarting phpfpm on Nextcloud package update fixes these issues (but this is a workaround). <del> phpfpm-nextcloud.restartTriggers = [ cfg.package ]; </del> <ins> phpfpm-nextcloud.restartTriggers = [ webroot ]; </ins> nextcloud-setup = let c = cfg.config;

Discussion:
The extraApp called displays as blank. This was working fine on nextcloud27. Rest of the site is working fine. Steps to reproduce the behavior: am currently on to webpage is blank on: Firefox Chrome Edge It should display the normal forms page. ! Found non-nixos issue relating to it here: Seems it may be a related issue? <!-- 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:
pysocks qrcode requests <del> tlslite-ng </del> certifi jsonpatch # plugins

Discussion:
Steps to reproduce the behavior: electrum <!-- 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]:
Fix in
Change:
<para> <literal>services.xserver.desktopManager.xfce</literal> now includes Xfce’s screen locker, <del> <literal>xfce4-screensaver</literal>. </del> <ins> <literal>xfce4-screensaver</literal> that is enabled by default. You can disable it by setting <literal>false</literal> to <link linkend="opt-services.xserver.desktopManager.xfce.enableScreensaver">services.xserver.desktopManager.xfce.enableScreensaver</link>. </ins> </para> </listitem> <listitem>

Discussion:
introduced a locker by consequence if you enable the XFCE4 DE and the new locker replaces any other already defined without a trivial way to disable. Steps to reproduce the behavior: any other locker using something like 4 locker is used anyway A way to enable XFCE4 stuff but without the locker, it can be enabled by default but with a option to disable. (I can't screenshot on the lockscreen) This is the current commit of my config: <!-- 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.
Change:
"hosts" "resolv.conf" "nsswitch.conf" <ins> # User profiles "profiles" </ins> # Sudo & Su "login.defs" "sudoers"

Discussion:
Describe the bug Apps packaged as AppImage don't seem to respect the system's fontconfig and cursor. Additionally, opening URLs is broken as it seems that doesn't work. To Reproduce Steps to reproduce the behavior: AppImage a URL Expected behavior It should open a browser at the given URL. Screenshots Proper cursor "outside" AppImage: ! Default X11 cursor "inside" AppImage: ! 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 system: host os: multi-user?: sandbox: version: channels(root): nixpkgs: Maintainer information:
CC: (seems to be the people that did the most significant work here, sorry if you don't have enough context).
BTW, are you using unstable or you're using with backports? Asking this because I had a similar issue with Discord not opening links, and using the proper Discord from instead of using a backport did the trick: .
I'm using unstable. Updated the issue with metadata.
Fixed in . I suppose you are installing stuff globally and hence why you didn't have the issues I did.
I had this problem too, but I thought it was because the difference between and . Reviewing and testing it ASAP.
Was this also fixed? for me (zettlr uses appimage)
Change:
disabled = pythonOlder "3.9"; src = fetchFromGitHub { <del> owner = "QCoDeS"; </del> <ins> owner = "microsoft"; </ins> repo = "Qcodes"; rev = "refs/tags/v${version}"; hash = "sha256-AggAVq/yfJUZRwoQb29QoIbVIAdV3solKCjivqucLZk=";

Discussion:
This breaks qcodes-contrib-drivers. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
makeScope makeScopeWithSplicing; inherit (self.meta) addMetaAttrs dontDistribute setName updateName appendToName mapDerivationAttrset setPrio lowPrio lowPrioSet hiPrio <del> hiPrioSet getLicenseFromSpdxId; </del> <ins> hiPrioSet getLicenseFromSpdxId getExe; </ins> inherit (self.sources) pathType pathIsDirectory cleanSourceFilter cleanSource sourceByRegex sourceFilesBySuffices commitIdFromGitRepo cleanSourceWith pathHasContext

Discussion:
There are a couple of different ways this is determined: use or splits the derivation name  uses It would be nice if there were a helper to get this value in a consistent way, and perhaps also a way to directly get an absolute path to it.
The absolute path can be as simple as after or similar.