jiebi/CodeConvo huggingface dataset:
Change:
<ins> diff --git a/service.py b/service.py index 024ad9a..6ef71dd 100644 --- a/service.py +++ b/service.py @@ -243,11 +243,8 @@ def playlistIndex(url, playlist): # Use the chosen resolver while forcing to use youtube_dl on legacy python 2 systems (dlp is python 3.6+) -if xbmcplugin.getSetting(int(sys.argv[1]),"resolver") == "0" or sys.version_info[0] == 2: -    from lib.youtube_dl import YoutubeDL -else: -    from lib.yt_dlp import YoutubeDL - +from yt_dlp import YoutubeDL + # patch broken strptime (see above) patch_strptime() </ins>

Discussion:
When trying to update nixos-24.05 from revision to , the check and build fail because is newly marked insecure (unstable PR: , backport PR: ). This breaks users of the package. I don't need the youtube-dl functionality and I don't wanna allow this insecure package on my system. Therefore, I'd appreciate an easier method to not depend on youtube-dl. This requires being able to build sendtokodi without youtube-dl altogether (and ensure sendtokodi does not use it). Either by default or as build option. Upstream seems to be mostly inactive, they won't help us here. Steps to reproduce the behavior: insecure packages (default behavior) to include sendtokodi on recent NixOS 24.05 ( or later) or unstable . Stable NixOS doesn't break on channel updates. Sendtokodi can be used without allowing any insecure packages. <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --(the relevant nixpkgs commits/PRs are already referenced, I cannot build the config on the broken revision) Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
sure code was never even used or executed in the entire life of this package in anyways, so all one has to do is remove from line and everything will be fine please go ahead and create a PR and it will be much appreciated :bow:
This isn't entirely true, as users can toggle between the two of them via a configuration. But regardless of that I agree that patching out the dependency is the right thing to do, even if it means that we will dishonor that configuration now. I've created to do so.
right, i sorta glossed over that part we should see if upstream wants to drop backend or not i suppose
Change:
]; buildInputs = [ <del> qtwebengine qtscript grantlee kxmlgui kwallet kparts </del> <ins> qtwebengine qtscript grantlee kxmlgui kwallet kparts qtxmlpatterns </ins> kjobwidgets kdesignerplugin kiconthemes knewstuff sqlcipher qca-qt5 kactivities karchive kguiaddons knotifyconfig krunner kwindowsystem libofx ];

Discussion:
Package name: skrooge Latest released version: 2.31.0 Current version on the unstable channel: 2.28.0 Current version on the stable/release channel: 2.28.0 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]:
Change:
# CMake install location "${_CMAKE_INSTALL_DIR}" ) <del> @@ -47,48 +44,48 @@ endif() </del> <ins> @@ -47,48 +44,49 @@ endif() </ins> # Non "standard" but common install prefixes list(APPEND CMAKE_SYSTEM_PREFIX_PATH

Discussion:
In this commit: , the code checks the env var without quotes, which means an unset or empty variables can cause build errors, eg this one for the Rust crate in a nix shell: I'm not sure it's missing quotes though, I don't know how CMake works in this situation, but since that looks like a bash-inspired syntax, I suppose it has bash-inspired bugs too Full error: Temporary workaround: Set to garbage, like , and it will build and run correctly
Ping the relevant people and link this issue in the original PR
probably it is duplicate of
Change:
google-cloud-sdk = callPackage ../tools/admin/google-cloud-sdk { python = python3; }; <del> google-cloud-sdk-gce = google-cloud-sdk.override { with-gce = true; }; </del> <ins> google-cloud-sdk-gce = google-cloud-sdk.override { python = python38; with-gce = true; }; </ins> google-fonts = callPackage ../data/fonts/google-fonts { };

Discussion:
Boto seems to be broken at least when brought in via the google-compute-image package. Steps to reproduce the behavior: the package in a derivation. to build the derivation. the error: I expect the build of the derivation to succeed. output forthe error: I'm not sure who the maintainers are. Please run and paste the result.
There's a PR to fix it by specifying a different Python version:
Change:
stdenv.mkDerivation rec { pname = "libopenshot"; <del> version = "0.2.7"; </del> <ins> version = "0.3.0"; </ins> src = fetchFromGitHub { owner = "OpenShot"; repo = "libopenshot"; rev = "v${version}"; <del> sha256 = "sha256-aF8wrPxFIjCy5gw72e/WyL/Wcx9tUGDkrqHS+ZDVK0U="; </del> <ins> sha256 = "sha256-qe866gFhcbd7yCNXDiZ9Aj0TAiWoJ+r7C5BjtWBKSGA="; </ins> }; postPatch = ''

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: openshot-qt <!-- 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: 2.6.1 desired version: 3.0.0 <!-- Search your package here: If no maintainer is listed for your package, tag the person that last updated the package -- Please tag this issue in your PR.
Change:
<ins> { fetchFromGitHub, buildMaven, maven, jdk, makeWrapper, stdenv, ... }: stdenv.mkDerivation rec { name = "exhibitor-${version}"; version = "1.5.6"; src = fetchFromGitHub { owner = "soabase"; repo = "exhibitor"; sha256 = "07vikhkldxy51jbpy3jgva6wz75jksch6bjd6dqkagfgqd6baw45"; rev = "5fcdb411d06e8638c2380f7acb72a8a6909739cd"; }; mavenDependenciesSha256 = "00r69n9hwvrn5cbhxklx7w00sjmqvcxs7gvhbm150ggy7bc865qv"; # This is adapted from https://github.com/volth/nixpkgs/blob/6aa470dfd57cae46758b62010a93c5ff115215d7/pkgs/applications/networking/cluster/hadoop/default.nix#L20-L32 fetchedMavenDeps = stdenv.mkDerivation { name = "exhibitor-${version}-maven-deps"; inherit src nativeBuildInputs; buildPhase = '' cd $pomFileDir; while timeout --kill-after=21m 20m mvn package -Dmaven.repo.local=$out/.m2; [ $? = 124 ]; do echo "maven hangs while downloading :(" done ''; installPhase = ''find $out/.m2 -type f ! -regex '.+(pom|jar|xml|sha1)' -delete''; # delete files with lastModified timestamps inside outputHashAlgo = "sha256"; outputHashMode = "recursive"; outputHash = mavenDependenciesSha256; }; # The purpose of this is to fetch the jar file out of public Maven and use Maven # to build a monolithic, standalone jar, rather than build everything from source # (given the state of Maven support in Nix). We're not actually building any java # source here. pomFileDir = "exhibitor-standalone/src/main/resources/buildscripts/standalone/maven"; nativeBuildInputs = [ maven ]; buildInputs = [ makeWrapper ]; buildPhase = '' cd $pomFileDir mvn package --offline -Dmaven.repo.local=$(cp -dpR ${fetchedMavenDeps}/.m2 ./ && chmod +w -R .m2 && pwd)/.m2 ''; meta = with stdenv.lib; { homepage = "https://github.com/soabase/exhibitor"; description = "ZooKeeper co-process for instance monitoring, backup/recovery, cleanup and visualization"; license = licenses.asl20; platforms = platforms.unix; }; installPhase = '' mkdir -p $out/bin mkdir -p $out/share/java mv target/$name.jar $out/share/java/ makeWrapper ${jdk}/bin/java $out/bin/startExhibitor.sh --add-flags "-jar $out/share/java/$name.jar" --suffix PATH : ${stdenv.lib.makeBinPath [ jdk ]} ''; } </ins>

Discussion:
We currently have a nice ZooKeper module and package, but apparently all the cool kids run ZK via , which we don't have packaged or any module around. Anyone feel like packaging and/or writing a module for it? cc who have dealt with ZK
I would like to own this. Thank you for bringing this up.
Let's assign it to you then &ndash; informally, as GitHub only allows that for organization members (or some similar condition).
Thanks
is this happening?
Sorry for the delay. Yes I am working on this.
any progress?
FYI, I took a crack at this in
Change:
lxd-image-server = handleTest ./lxd-image-server.nix {}; #logstash = handleTest ./logstash.nix {}; lorri = handleTest ./lorri/default.nix {}; <ins> maddy = handleTest ./maddy.nix {}; </ins> magic-wormhole-mailbox-server = handleTest ./magic-wormhole-mailbox-server.nix {}; magnetico = handleTest ./magnetico.nix {}; mailcatcher = handleTest ./mailcatcher.nix {};

Discussion:
Maddy, a small and compact mail server written in Go, was merged as a package with by . Having a NixOS module to enable and configure it would be really great! If someone is interested in creating a PR, I will be happy to help and test it :+1: Best regards Jonas
Hi, I dont have any sort of way to properly test this within the actual nixos repo but I've got a personal working module/service you can try
I did something very similar to the above, but based my serviceConfig on the example . It contains quite a bit more of the systemd security features, so if you want to make something to include in nixpkgs I think all these things should be on! I also hooked it up to Dovecot, hence the waiting script, not sure if that's the best way of doing it.
How about this: Nice and simple.
Have you tried this? Somehow it doesn't work for me:
- it looks like the units aren't included in the nixpkgs package :man_shrugging:, but I created a PR to fix this in Please review, test, and approve PR. I have never used so I wouldn't want to merge that PR without you (or someone else) testing. Thanks!
Change:
, lxqt-build-tools , wrapQtAppsHook , gitUpdater <ins> , version ? "2.0.0" </ins> }: stdenv.mkDerivation rec { pname = "qtermwidget"; <del> version = "2.0.0"; </del> <ins> inherit version; </ins> src = fetchFromGitHub { owner = "lxqt"; repo = pname; rev = version; <del> hash = "sha256-kZS6D/wSJFRt/+Afq0zCCmNnJPpFT+1hd4zVPc+rJsE="; </del> <ins> hash = { "1.4.0" = "sha256-wYUOqAiBjnupX1ITbFMw7sAk42V37yDz9SrjVhE4FgU="; "2.0.0" = "sha256-kZS6D/wSJFRt/+Afq0zCCmNnJPpFT+1hd4zVPc+rJsE="; }."${version}"; </ins> }; nativeBuildInputs = [

Discussion:
Steps to reproduce the behavior: virt-manager-qt on 24.05 (or, I assume, unstable) Full log: Bisecting the build yields from as the first bad commit. It updates from to . <!-- 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]:
Relevant upstream commit:
Change:
stdenv.mkDerivation rec { name = "spirv-tools-${version}"; <del> version = "2017-09-01"; </del> <ins> version = "2018-02-05"; </ins> src = spirv_sources.tools; patchPhase = ''ln -sv ${spirv_sources.headers} external/spirv-headers'';

Discussion:
Tried to make a system upgrade, got this: Obviously, upgrade can't be completed. have in . run Please run and paste the results. system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
This looks like the offending line: This could be related to what file system you are using?
Actually this looks like unrelated to glslang. Works on master:
Change:
platform = platforms.${stdenv.system} or (throw "unsupported system ${stdenv.hostPlatform.system}"); urlVersion = lib.replaceStrings [ "." ] [ "_" ] version; in fetchurl { <del> url = "https://files.renoise.com/demo/Renoise_${urlVersion}_Demo_Linux_${platform.archSuffix}.tar.gz"; </del> <ins> urls = [ "https://files.renoise.com/demo/Renoise_${urlVersion}_Demo_Linux_${platform.archSuffix}.tar.gz" "https://files.renoise.com/demo/archive/Renoise_${urlVersion}_Demo_Linux_${platform.archSuffix}.tar.gz" ]; </ins> hash = platform.hash; };

Discussion:
404 when trying to install. Version is out of date on both stable / unstable. Current version is 1.3.5 aka <

<] Trying to install this and get failure due to bad URL. Probably due to a stale version number, this should target the most recent version or redux
Change:
nativeBuildInputs = [ pkgs.nodePackages.node-pre-gyp pkgs.which <ins> pkgs.xcbuild </ins> ]; buildInputs = [

Discussion:
Steps to reproduce the behavior: n8n (from ) Bisected to . Would be fixed by , but that's now closed due to marsam's departure from the community; filing this issue to avoid the breakage getting lost. <!-- 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:
<ins> { lib, buildDunePackage, ocaml_gettext, ounit }: buildDunePackage rec { pname = "gettext-stub"; inherit (ocaml_gettext) src version meta; propagatedBuildInputs = [ ocaml_gettext ]; doCheck = true; checkInputs = lib.optional doCheck ounit; } </ins>

Discussion:
Describe the bug To Reproduce Steps to reproduce the behavior: vagrant on latest master error Expected behavior Builds Screenshots If applicable, add screenshots to help explain your problem. Additional context Add any other context about the problem here. Metadata system: host os: multi-user?: sandbox: version: channels(maciej): nixpkgs: Maintainer information:
Broken in . I don't know anything about ocaml, so I'm not well qualified to fix this. CC
Apologies for breaking . Thanks for reporting this issue; PR should fix it.
Change:
propagatedBuildInputs = [ six requests <ins> infotagger </ins> inputstreamhelper ];

Discussion:
When using Kodi with the YouTube plugin , I get the following error: Steps to reproduce the behavior: The YouTube plugin used to work perfectly. shows that the dependency was but that the dependency is an old one. The wrapper is here:
Thanks for catching this. Looks like a pretty straight forward addon to package. Is this something you'd be interested in tackling
I can, if I find the time! One thing I do not understand is that the dependency is an old one. I could see that the YouTube plugin is failing because the new dependency is missing, but it seems to fail because of (which is also not provided by Nixpkgs, but why didn't it fail before?).
I have tested this and it works for me. Thanks
Change:
, buildPythonPackage , pythonOlder , fetchFromGitHub <ins> , fetchpatch </ins> , substituteAll , gdb , django

Discussion:
Steps to reproduce the behavior: on latest Nixpkgs master (See ). Tracked upstream:
Oh nice catch! I'll try a git bisect on the to try to narrow down the cause.
Forgot I was running this in the background :sweat_smile: - tracked it down to the :
debugpy doesn't depend on etils
Oh yep, it was probably just a bad automatic bisect then, I'll try manually.
Ok, I just ran it again and it just looks like there was just an unrelated error in one of the commits it sampled before: If this wasn't going to work I was thinking of adding failing inputDrvs as a skip condition. Didn't need to, but it'll be helpful for next time!
Change:
rustypaste-cli = callPackage ../tools/misc/rustypaste-cli { }; <ins> rustywind = callPackage ../development/tools/misc/rustywind { }; </ins> rw = callPackage ../tools/misc/rw { }; rwc = callPackage ../tools/system/rwc { };

Discussion:
Project description CLI for organizing Tailwind CSS classes Metadata homepage URL: source URL: license: Apache License 2.0 platforms: unix, linux, darwin, ...
Change:
<del> { wxGTK, lib, stdenv, fetchurl, cmake, libGLU_combined, zlib </del> <ins> { wxGTK, lib, stdenv, fetchurl, fetchFromGitHub, cmake, libGLU_combined, zlib </ins> , libX11, gettext, glew, glm, cairo, curl, openssl, boost, pkgconfig , doxygen, pcre, libpthreadstubs, libXdmcp , wrapGAppsHook , oceSupport ? true, opencascade , ngspiceSupport ? true, libngspice , swig, python, pythonPackages <ins> , lndir </ins> }: assert ngspiceSupport -> libngspice != null; with lib; <del> stdenv.mkDerivation rec { </del> <ins> let mkLib = version: name: sha256: attrs: stdenv.mkDerivation ({ name = "kicad-${name}-${version}"; src = fetchFromGitHub { owner = "KiCad"; repo = "kicad-${name}"; rev = "${version}"; inherit sha256 name; }; nativeBuildInputs = [ cmake ]; } // attrs); in stdenv.mkDerivation rec { </ins> name = "kicad-${version}"; series = "5.0"; version = "5.0.2";

Discussion:
KISYSMOD and KISYS3DMOD point to missing directories. On my system they are and , respectively. Neither of these paths exist though. I do see a few other kicad-5.0.0 directories in the Nix store, but I'm not sure if I should use those (and if so, which one). the environment variables (Preferences Configure Paths...)
The library is also supposed to be at which does not exist while points to it.
Mostly fixed by . Symbols and footprints are now included, and the variables point to the correct location. We don't have the 3D models by default as they're five gigabytes, for that you need to use the derivation (which unfortunately means recompiling kicad for now).
Change:
stdenv lib bin tlpdb tlpdbxz tl installShellFiles coreutils findutils gawk getopt ghostscript_headless gnugrep <del> gnumake gnupg gnused gzip ncurses perl python3 ruby zip; </del> <ins> gnumake gnupg gnused gzip html-tidy ncurses perl python3 ruby zip; </ins> }; in overrides tlpdb;

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:
update-resolv-conf = callPackage ../tools/networking/openvpn/update-resolv-conf.nix { }; <del> open-pdf-presenter = callPackage ../applications/misc/open-pdf-presenter { }; </del> openvswitch = callPackage ../os-specific/linux/openvswitch { }; optipng = callPackage ../tools/graphics/optipng {

Discussion:
Recently released removes the QT4 support. For now, we will split the package and keep on 0.61.
It is still used so it should remain at an older version.
~It's used just in two packages, apparently ( and ).~ Hopefully those can be soon migrated to Qt5. Otherwise we can keep old version for only and keep updating the rest...
Those are only python dependents. It is also referenced from calligra, texstudio, texmaker, qpdfview, diffpdf,
Thanks, I grepped for a wrong string.
Yeah. (I meant that should remain at an older version, while can be decoupled and upgraded.)
moves diffpdf away from poppler_qt4, FWIW.
currently working on a fix for package as its latest version uses qt5 now, so poppler_qt4 isn't needed anymore
I opened a updating qpdfview, which makes open-pdf-presenter the last thing depending on (via the python library). Since it is abandoned, it is not likely to be updated. Do we remove it?
almost done, need some rest for now, PR for will come tomorrow :)
Finally getting rid of this in .
Change:
unittest-xml-reporting = callPackage ../development/python-modules/unittest-xml-reporting { }; <ins> univers = callPackage ../development/python-modules/univers { }; </ins> unpaddedbase64 = callPackage ../development/python-modules/unpaddedbase64 { }; unrardll = callPackage ../development/python-modules/unrardll { };

Discussion:
Project description Python project to parse and compare package versions and ranges. Metadata homepage URL: source URL: license: Apache-2.0 platforms: linux
Change:
changelog = "https://github.com/executablebooks/markdown-it-py/blob/${src.rev}/CHANGELOG.md"; license = licenses.mit; maintainers = with maintainers; [ bhipple ]; <ins> mainProgram = "markdown-it"; </ins> }; }

Discussion:
at calls 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. at Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
}; }; <del> meta.maintainers = with maintainers; [ bbigras ]; </del> <ins> meta.maintainers = with maintainers; [ bbigras anpin ]; </ins> }

Discussion:
Hello, Recently started using on my homelab server, and when running it from the commandline, with my own supplied config file, works perfectly, but then when trying and setting to true, the setting is not reflected in the file that is generated in the derivaton, and passed to the service at startup. If you look at the attribute, it doesn't seem like is mentioned at all. This is the code I used to enable . Tunnel is reported in the cloudflare dashboard, but when looking at logs when trying to access a private network, it explicitly says that warp routing is not enabled. Steps to reproduce the behavior: cloudflared, and enable the nix store for the generated file that is created by the derivation that the yaml does not include any setting For to be included in the service configuration. If applicable, add screenshots to help explain your problem. Add any other context about the problem here. Please run and paste the result.