jiebi/CodeConvo huggingface dataset:
Change:
using the EDITOR environment variable. ''; }; <ins> startWithGraphical = mkOption { type = types.bool; default = config.services.xserver.enable; defaultText = literalExpression "config.services.xserver.enable"; description = lib.mdDoc '' Start emacs with the graphical session instead of any session. Without this, emacs clients will not be able to create frames in the graphical session. ''; }; </ins> }; config = mkIf (cfg.enable || cfg.install) {

Discussion:
I have the Emacs daemon enabled on my system (). When I log in, I can see that the daemon is running but I am unable to start : Restarting the daemon fixes the issue. Is the daemon starting too early and missing some variables from Wayland? Steps to reproduce the behavior: emacs daemon () in and try to run starts: Please run and paste the result.
I'm not the maintainer of this service and I don't use it. I use the Emacs service from home-manager which I recommend:
My apologies. I saw that you were the original author, so I figured you were my best bet. It looks like home-manager might be avoiding this by (optionally) using socket activation. Something similar might work here, but I'm sure there's a more robust solution. I'll have to dig into it a bit when I have time.
Does helps
Yeah, ordering after seems to have worked. Thanks for the tip. I'm trying to work up a nice way to incorporate that into the service definition. Unfortunately, simply adding the "after" directive doesn't seem to be enough. I'm guessing that's because is reached, or at least initiated, before . I'm going to keep poking at it over the coming days.
Wanted to confirm made this issue go away.
Same here, "graphical" solution works. It would be nice to see pr merged.
Change:
libsForQt512 = recurseIntoAttrs (lib.makeScope qt512.newScope mkLibsForQt5); <del> # TODO bump to 5.12 on darwin once it's not broken qt5 = if stdenv.isDarwin then qt511 else qt512; </del> <ins> qt5 = qt512; </ins> libsForQt5 = if stdenv.isDarwin then libsForQt511 else libsForQt512; qt5ct = libsForQt5.callPackage ../tools/misc/qt5ct { };

Discussion:
We are currently stuck on qt 5.11 on darwin/macOS. To get things consistent, this should update to qt 5.12 (or even 5.13) soon. Please run and paste the results.
If qt512 builds on darwin we should change the default version:
is currently broken:
Fixed in

When running the provided by the package, it complains about not finding the tool, which is provided by . I have not found a suitable workaround, since it seems like the location of the scanner is hardcoded inside Invoke on a project using Qt. How to use this tool is explained here: Please run and paste the results. system: host os: multi-user?: sandbox: version: channels(raskolnikov): nixpkgs:
The error is as follows:
Hi I tried withyour latestchanges from . The fact that Qt 5.12 now works on Darwin again is great news! However, this issue seems to not be resolved. Now the does not bail out inmmediately as it did before. It does indeed do something for a while, but then later I get: Maybe there is some other part of the code where it is missing the path correction? Thanks a lot!
My bad, I was actually using the old one by mistake... I am getting new errors though: I will investigate...
I think I found the issue. The problem is this line: is returning , which is incorrect. I think lines 1258-1260 can be discarded altogether since without the tool is still correctly using , which is a better way of passing the import paths, since inside Nix we actually need multiple import paths anyways due to the modularized Qt installation. Would you be happy to extend the patch with such a modification?
Ok... I have further investigated and I was wrong... One must indeed pass to to really make it work. One can indeed pass multiple times, and probably we shall do it for every entry in . Probably this can be patched... but it seems like we are getting into non trivial patching territory...
Thanks Mathew for reopening the issue. I opened a corresponding issue on Qt:
I marked this as stale due to inactivity. &rarr;
Actually this still stands and it'd be nice to have it fixed some day. There are other issues that are blocking me on Mac these days though, like .
I marked this as stale due to inactivity. &rarr;
Change:
<del> { lib, stdenv, fetchFromGitHub }: </del> <ins> { lib, stdenv, fetchFromGitHub, fetchpatch }: </ins> stdenv.mkDerivation rec { version = "0.6.8";

Discussion:
bash tab completion for command is not working for letters m - z. Steps to reproduce the behavior: -shell -p l <tab<tabDisplay all 1728 possibilities? (y or n) l2md ... lzwolf -shell -p m <tab<tabDisplay all 9413 possibilities? (y or n) m17-cxx-demod ... zzuf -shell -p n <tab<tabempty tab completion for letters m to z work correctly Running shows the suggestion list for packages which contains: I am pretty sure the customization function (ends with single quote) in messes up when evaluating the list of packages. Running in the shell with all packages except the string and everything works as expected. Please run and paste the result. system: host os: multi-user?: sandbox: version: channels(root): nixpkgs: Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
I build an overlay to delete all strings in ending with ' via before feeding them to . with patchfile /etc/nixos/overlays- Now bash-completion for works again for letters a-z.
Thanks for reporting! Fix is in
Change:
"agpl" ] }, <del> "news": { "sha256": "", "url": "", "version": "", "description": "📰 A RSS/Atom Feed reader App for Nextcloudnn- 📲 Synchronize your feeds with multiple mobile or desktop [clients](https://nextcloud.github.io/news/clients/)n- 🔄 Automatic updates of your news feedsn- 🆓 Free and open source under AGPLv3, no ads or premium functionsnn**System Cron is currently required for this app to work**nnRequirements can be found [here](https://nextcloud.github.io/news/install/#dependencies)nnThe Changelog is available [here](https://github.com/nextcloud/news/blob/master/CHANGELOG.md)nnCreate a [bug report](https://github.com/nextcloud/news/issues/new/choose)nnCreate a [feature request](https://github.com/nextcloud/news/discussions/new)nnReport a [feed issue](https://github.com/nextcloud/news/discussions/new)", "homepage": "https://github.com/nextcloud/news", "licenses": null }, </del> "notes": { "sha256": "1g4ibrymsfd2bcvmyfyrl23z2kh4bgkwrgyacvdx1glk44di6sgc", "url": "https://github.com/nextcloud-releases/notes/releases/download/v4.8.1/notes.tar.gz",

Discussion:
Out-of-bounds list index when upgrading from Nextcloud 26 to 27. Steps to reproduce the behavior: Change Nextcloud.package version from 26 to 27 and change extraApps versions Compilable expression. <details<summaryentire stack trace:</summary</detailsHere's my NC expression I had to manually remove some which Nix would tell me don't exist for NC27. Furthermore, entirely removing solves the issue.
as long as there's no release from that supports Nextcloud 27 we cannot do much to fix that unfortunately, i.e. for now you'll either need to stick with v26 or uninstall this app temporarily. However, this weird error message is in fact a bug (the lock entry for this app for v27 was malformed in nixpkgs because the app was attempted to be even though there's no support) and I opened to make sure that you'll get a slightly better error-message.
Thanks! For sure, a better indication that an app isn't available for the chosen NC version would be useful
Change:
buildGoPackage rec { pname = "wireguard-go"; <del> version = "0.0.20190517"; </del> <ins> version = "0.0.20190908"; </ins> <del> goPackagePath = "git.zx2c4.com/wireguard-go"; </del> <ins> goPackagePath = "golang.zx2c4.com/wireguard"; </ins> src = fetchzip { url = "https://git.zx2c4.com/wireguard-go/snapshot/wireguard-go-${version}.tar.xz"; <del> sha256 = "0ir3dp13vkkqr76q0jvw610qw40053ngk51psqhqxfaw3jicdqgr"; </del> <ins> sha256 = "1jsch0157nk86krdknij7dsvg6i7ar0ydhy07r40drhxqyp3q0hx"; </ins> }; goDeps = ./deps.nix;

Discussion:
I can't compile on macOS: Metadata system: host os: multi-user?: sandbox: version: channels(eridius): nixpkgs: Maintainer information:
Change:
asciidocFull = asciidoc-full;  # added 2014-06-22 at_spi2_atk = at-spi2-atk; # added 2018-02-25 at_spi2_core = at-spi2-core; # added 2018-02-25 <ins> avldrums-lv2 = x42-avldrums; # added 2020-03-29 </ins> bar-xft = lemonbar-xft;  # added 2015-01-16 bashCompletion = bash-completion; # Added 2016-09-28 batti = throw "batti has been removed from nixpkgs, as it was unmaintained"; # added 2019-12-10

Discussion:
Recently a second version of avldrums got merged: We already had
cc
How many of these kind of plugins do we have? Does it make sense to put them in a package set?
What do you mean by these kind of plugins? LV2 in general? Might make sense. Not sure what the advantages and disadvantages are. avldrums? There is only one, but it's packaged twice. (just in case you missed that). :)
Yes, lv2 in general. We should do this
So by package set you mean just move them next to each other in all-packages? BTW, not all of the above are plugins: lv2: the library lv2bm: an lv2 benchmarker faust2lv2: a compiler to make lv2s with There are also more lv2 plugins in nixpkgs.
Unfortunately x42 stuff has two names, sometimes distantly related (x42-autotune is aka fat1.lv2). It seems that x42 names are better because they are more recognizable (as belonging to x42-), they are used in URLs (https://x42-) and as the names of packages (https://x42-). It would be useful to be able to find all packaged lv2 plugins at a glance. However, since some lv2 plugins come with a standalone application, sometimes it will be difficult to decide whether a package should be classified as a plugin or not. (I would classify x42-plugins as plugins despite their programs in bin/, but qmidiarp as a non-plugin despite its lib/lv2/.)
I propose the same approach as in Python. In case its plugins, put them in the plugin set. In case its only application, put them in top-level, if both, put them in plugins and offer a top-level reference.
I agree it would be useful to be able to find all packaged lv2 plugins at a glance.
OK. I would propose to name the package set audioPlugins and put the definitions in applications/audio/plugins/ . They should comprise DAW plugins (lv2, ladspa, dssi, vst), but not e.g. media player plugins (gst-plugins-good).

It does not seem useful to subdivide plugins further, because audio hosts support many types of plugins (Ardour and Carla support lv2, ladspa, vst) and some packages provide many types of plugins at once (zam-plugins has lv2, ladspa, vst; wolf-shaper has lv2, dssi, vst).
I'm not familiar with DAWs but that sounds reasonable! Maybe then so its clear its not about and such?
Change:
tremor = callPackage ../development/libraries/tremor { }; <ins> triehash = callPackage ../tools/misc/triehash { }; </ins> trillian = callPackage ../tools/misc/trillian { buildGoModule = buildGo115Module; };

Discussion:
Describe the bug Between and , apt somehow got reintroduced into the repository () but without an entry in all- It could be bumped to 2.1.18 - Out-of-date package report It doesn't compile for the same reason for which it was originally removed from Nixpkgs and I don't know how to fix it - bug report I need apt for another package to work properly (apt -python-apt -python-debian -click: ) - packaging request? To Reproduce for apt package -not packaged in Nixpkgs repo -but package code exists? commit/PR history - (staging) removed all apt code, (master) bumped apt Expected behavior apt being either in all- or missing from the repository completely. Preferably, a working apt derivation. Additional context Notify maintainers package maintainer for apt last bump to package originally removed the package Metadata system: host os: multi-user?: sandbox: version: channels(root): channels(bt1cn): nixpkgs: Maintainer information:
FWIW, I briefly looked into this: Switching to gcc9 hides the detailed error message, but compilation still fails.
I looked into this a little. fixes the build, but the binary that can be produced is far from functional. reveals that apt expects several other files relative to its install location to exist. As the package is currently defunct and unreachable, I think it is best to remove it.
I marked this as stale due to inactivity. &rarr;
Bump, still exists.
Change:
@contextmanager <del> def audio_recording(machine: Machine) -> None: </del> <ins> def record_audio(machine: Machine): </ins> """ Perform actions while recording the machine audio output.

Discussion:
Likely caused by build failure of . : I am really sorry I didn't test this in and seems like this is one of channel blockers left. Firefox maintainer also cc
FWIW I think you can just remove the and it'll typecheck fine.
Agreed, the yield does not yield anything and is only used to interrupt the function calls.
Change:
<ins> diff -Naur source-old/pyproject.toml source-new/pyproject.toml --- source-old/pyproject.toml	1980-01-02 00:00:00.000000000 -0300 +++ source-new/pyproject.toml	2023-02-04 10:09:48.087418202 -0300 @@ -29,11 +29,11 @@ "Topic :: Security", ] dependencies = [ -    "pydantic~=1.10.2", -    "python-dotenv~=0.21.0", -    "requests~=2.28.1", -    "rich~=12.6.0", -    "shodan~=1.28.0", +    "pydantic", +    "python-dotenv", +    "requests", +    "rich", +    "shodan", ] dynamic = ["version"] @@ -63,7 +63,7 @@ "mypy", "pytest", "pytest-cov", -    "types-requests~=2.28.1", +    "types-requests", ] [tool.hatch.envs.default.scripts] typecheck = "mypy -p {args:wtfis}" </ins>

Discussion:
Project description As stated by the developer: Metadata homepage URL: source URL: license: MIT platforms: unix, linux, darwin
Change:
inherit (gnome3) libcroco; <ins> libcangjie = callPackage ../development/libraries/libcangjie { }; </ins> libctemplate = callPackage ../development/libraries/libctemplate { }; libcue = callPackage ../development/libraries/libcue { };

Discussion:
Describe the bug Python 3.7 pycurl is failing to build in the latest nix unstable To Reproduce Steps to reproduce the behavior: python3.7-pycurl- fails with an assertion failure in the test, i.e. Expected behavior The build works without issues Build Log Metadata Maintainer information:
I tried building the same derivation three times, having 2 failures and 1 successful build. The failures are different from the one in the given build log. Command: on latest Success logs: Failure logs: I think that tests are a bit... "flaky", so to say. (read "non-deterministic")
I tried running it 3 times and didn't get it to work, let me try a few more times. Might it be the case that on some machines its failing reliably?
This is the latest run with even more tests failing Something seems to be quite wrong the tests, i.e. they seem to be seriously flaky.
Update on this, I did manage to get it to eventually pass the tests but it took 20 times. Maybe we should report these flaky tests to upstream?
I opened to disable the tests, which will fix this
This issue still exists despite your fix I got it today on 20.03. The reason is probably that is not the only one that can trigger it, there is also a : Full build errors: <details</details
PR at
And another one : <details</detailsPR:
Change:
<del> { lib, stdenv, fetchurl, ncurses, zlib , openssl ? null , sslSupport ? true </del> <ins> { lib, stdenv, fetchurl, ncurses, zlib, openssl, sslSupport ? true, </ins> }: assert sslSupport -> openssl != null;

Discussion:
Steps to reproduce the behavior: tinyfugue on aarch64-darwin command: full log: <!-- 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]:
Willing to contribute, but I don't know exactly how tinyfugue is built. I can also help with testing on MacOS if needed.
I think it's fixed, works on my , if you could have a look it would be great: or if you don't have checked out:
Change:
<ins> diff --git a/dlls/crypt32/rootstore.c b/dlls/crypt32/rootstore.c index f795181..fb4926a 100644 --- a/dlls/crypt32/rootstore.c +++ b/dlls/crypt32/rootstore.c @@ -18,6 +18,7 @@ #include "config.h" #include <stdarg.h> #include <stdio.h> +#include <stdlib.h> /* getenv */ #include <sys/types.h> #ifdef HAVE_SYS_STAT_H #include <sys/stat.h> @@ -916,6 +917,11 @@ static void read_trusted_roots_from_known_locations(HCERTSTORE store) for (i = 0; !ret && i < ARRAY_SIZE(CRYPT_knownLocations); i++) ret = import_certs_from_path(CRYPT_knownLocations[i], from, TRUE); + +        char *nix_cert_file = getenv("NIX_SSL_CERT_FILE"); +        if (nix_cert_file != NULL) +            ret = import_certs_from_path(nix_cert_file, from, TRUE); + check_and_store_certs(from, store); } CertCloseStore(from, 0); </ins>

Discussion:
Describe the bug Oof, this might be a very specific issue, but I wanted to report it anyway so it might save some time for other people troubleshooting the same issue. I run a downloader in a nix build using wine. However, it seems like the . This means I cannot establish any TLS connections from wine inside the nix sandbox. To Reproduce Steps to reproduce the behavior: a TLS connection from wine inside a nix build Expected behavior There should be a way to specify the ca certificates. Additional context This does fix the issue for me: If you're all okay with that, I can create a nixpkgs PR for this. Maintainer information: cc
Hm. I wonder if we should just apply this to the Wine package.
on first glance offered patch looks obivious
Yeah, I think adding this patch makes sense
I opened
Change:
, rustPlatform , cmake , pkg-config <del> , python3 </del> , perl <del> , freetype </del> , fontconfig <del> , libxkbcommon , xcbutil , libX11 , libXcursor , libXrandr , libXi , vulkan-loader </del> , copyDesktopItems , makeDesktopItem <ins> , glib , gtk3 </ins> , openssl , libobjc , Security

Discussion:
lapce: current version: 0.1.0 desired version: 0.1.1 maintainers: The window creation is switched from winit to druid, which fixed lots of keyboard bugs. so maybe require new dependencies and removal of many dependencies. arrow keys in terminal should be working now. vulkan part should be removed as it switched from Wgpu to OpenGL. Please tag this issue in your PR.
Change:
sed -i -e 's|thunar_dialogs_show_insecure_program (parent, _(".*"), file, exec)|1|' thunar/thunar-file.c ''; <ins> preFixup = '' gappsWrapperArgs+=( # https://github.com/NixOS/nixpkgs/issues/329688 --prefix PATH : ${lib.makeBinPath [ exo ]} ) ''; </ins> meta = with lib; { description = "Xfce file manager"; mainProgram = "thunar";

Discussion:
Missing dependency on breaks "Open Terminal Here" functionality Steps to reproduce the behavior:  without in that right clicking and selecting "Open Terminal Here" is broken Package should handle so users don't have to manually install Possible to fix by wrapping lines occur in <!-- 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:
inherit (pkgs) sentencepiece; }; <ins> sentence-transformers = callPackage ../development/python-modules/sentence-transformers { }; </ins> sentinel = callPackage ../development/python-modules/sentinel { }; sentinels = callPackage ../development/python-modules/sentinels { };

Discussion:
Project description Natural Language Search Engine for your Org-Mode and Markdown notes, Beancount transactions and Photos Metadata homepage URL: source URL: license: gpl3 , ...
Change:
<ins> { fetchFromGitHub , lib , gettext , pkgs , python3 , umockdev , writeScript }: let # We need these simple wrapper shell scripts because Inkscape extensions with # interpreter="shell" always get invoked with the `sh` command [0], regardless of # the shebang at the top of the script. # [0]: https://gitlab.com/inkscape/inkscape/-/blob/d61d917afb94721c92a650b2c4b116b0a4826f41/src/extension/implementation/script.cpp#L93 launch-sendto_silhouette = writeScript "sendto_silhouette.sh" '' cd $(dirname $0) ./sendto_silhouette.py "$@" ''; launch-silhouette_multi = writeScript "silhouette_multi.sh" '' cd $(dirname $0) ./silhouette_multi.py "$@" ''; in python3.pkgs.buildPythonApplication rec { pname = "inkscape-silhouette"; version = "1.28"; format = "setuptools"; src = fetchFromGitHub { owner = "fablabnbg"; repo = pname; rev = "v${version}"; sha256 = "sha256-uNVhdkZFadL7QNlCsXq51TbhzRKH9KYDPDNCFhw3cQs="; }; patches = [ ./interpreter.patch ./use-prefix-for-udev.patch ]; propagatedBuildInputs = [ python3.pkgs.pyusb python3.pkgs.lxml python3.pkgs.inkex python3.pkgs.matplotlib python3.pkgs.wxPython_4_2 python3.pkgs.xmltodict ]; nativeBuildInputs = [ gettext # msgfmt ]; nativeCheckInputs = [ python3.pkgs.pytestCheckHook umockdev ]; pytestFlagsArray = [ "test" ]; doCheck = true; installPhase = '' runHook preInstall make install PREFIX=$out runHook postInstall ''; postInstall = '' # Unmark read_dump.py as executable so wrapPythonProgramsIn won't turn it # into a shell script (thereby making it impossible to import as a Python # module). chmod -x $out/share/inkscape/extensions/silhouette/read_dump.py cp ${launch-sendto_silhouette} $out/share/inkscape/extensions/sendto_silhouette.sh cp ${launch-silhouette_multi} $out/share/inkscape/extensions/silhouette_multi.sh ''; postFixup = '' wrapPythonProgramsIn "$out/share/inkscape/extensions/" "$out $pythonPath" ''; meta = with lib; { description = "An extension to drive Silhouette vinyl cutters (e.g. Cameo, Portrait, Curio series) from within Inkscape."; homepage = "https://github.com/fablabnbg/inkscape-silhouette"; license = licenses.gpl2Only; maintainers = with maintainers; [ jfly ]; platforms = platforms.all; }; } </ins>

Discussion:
Project description An extension to drive Silhouette vinyl cutters (e.g. Cameo, Portrait, Curio series) from within inkscape Metadata homepage URL: source URL: license: gpl2 platforms: unix
Change:
The nextcloud-occ program preconfigured to target this Nextcloud instance. ''; }; <ins> nginx.recommendedHttpHeaders = mkOption { type = types.bool; default = true; description = "Enable additional recommended HTTP response headers"; }; </ins> }; config = mkIf cfg.enable (mkMerge [

Discussion:
Describe the bug headers are set in with appendHttpConfig or extraConfig you only can add the same header again with another value but you can not replace the headers set in Expected behavior have an option to enable default headers and disable them to set them yourself Additional context specially for cache or content-security-policy headers annoying Notify maintainers
What about ?
might work but creates the need to check this options on every update.
What needs to be checked every update? Why?
i mean if there is a need for changes in this section.
Ah sorry, yes I understand.
Perhaps we can set headers in an RFC42-style way such as: This would be an addition that should be in the -module itself though. Unfortunately I already have way too much other todos right now, so I don't really want to start another (IMHO) non-trivial task here, so feel free to take over :)