jiebi/CodeConvo huggingface dataset:
Change:
, fetchFromGitHub , cmake , pkg-config <ins> , wrapQtAppsHook </ins> , openscenegraph , mygui , bullet

Discussion:
Describe the bug It seems the package openmw (morrowind game engine) is missing dependencies, maybe Qt related. To Reproduce Install openmw and run or , you don't need Morrowind asset files to run them. Expected behavior A window is opened. Notify maintainers Metadata system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
Change:
{ stdenv , fetchFromGitHub <ins> , fetchpatch </ins> , zlib , expat , cmake

Discussion:
, CVSSv3=6.5 (nixos-19.09, nixos-unstable) Scanned versions: nixos-19.09: ; nixos-unstable: . May contain false positives.
See also: , ,
Patched on all three currently supported branches. See backlinks on MR .
Change:
{ stdenv, lib, buildEnv, writeShellScriptBin, fetchurl, vscode, unzip, jq }: let <del> extendedPkgVersion = lib.getVersion vscode; extendedPkgName = lib.removeSuffix "-${extendedPkgVersion}" vscode.name; </del> buildVscodeExtension = a@{ name, <del> namePrefix ? "${extendedPkgName}-extension-", </del> src, # Same as "Unique Identifier" on the extension's web page. # For the moment, only serve as unique extension dir.

Discussion:
Describe the bug A clear and concise description of what the bug is. the vscode-extensions directory has the structure so should be called with the argument instead of To Reproduce Steps to reproduce the behavior: ... ... ... Expected behavior A clear and concise description of what you expected to happen. Screenshots If applicable, add screenshots to help explain your problem. Additional context Add any other context about the problem here. Notify maintainers <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --Metadata Please run and paste the result. Maintainer information:
Change:
done ''; <ins> # TODO(@oxij): Stop referencing args here </ins> meta = { homepage = http://www.xen.org/; description = "Xen hypervisor and related components"

Discussion:
Security support for Xen 4.5 (our current default version) ended January 2018, so we should change the default to 4.8 in 18.03 (and remove 4.5 in the future).
what's your opinion on this?
See .
Change:
}; patches = [ (fetchpatch { <del> url = "https://git.alpinelinux.org/aports/plain/community/rapidjson/do-not-include-gtest-src-dir.patch"; </del> <ins> url = "https://git.alpinelinux.org/aports/plain/community/rapidjson/do-not-include-gtest-src-dir.patch?id=9e5eefc7a5fcf5938a8dc8a3be8c75e9e6809909"; </ins> hash = "sha256-BjSZEwfCXA/9V+kxQ/2JPWbc26jQn35CfN8+8NW24s4="; }) ];

Discussion:
Steps to reproduce the behavior: the and using an overlay, . Nix twice: (building Nix that targets new store, but lives in old store) (building Nix that targets and lives in new store) for error. I'm trying to build Nix with a different store path, so it's trying to build musl, but it fails due to a hash mismatch. Current hash in should actually be since that file was recently . However, I was told it would be better to include the commit ref in the URL instead of relying on the main branch. <!-- 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:
<ins> diff --git a/src/main/cpp/option_processor.cc b/src/main/cpp/option_processor.cc index 8f8f15685f..a7ae52d1e4 100644 --- a/src/main/cpp/option_processor.cc +++ b/src/main/cpp/option_processor.cc @@ -56,7 +56,7 @@ OptionProcessor::OptionProcessor( : workspace_layout_(workspace_layout), startup_options_(std::move(default_startup_options)), parse_options_called_(false), -      system_bazelrc_path_(BAZEL_SYSTEM_BAZELRC_PATH) {} +      system_bazelrc_path_("@bazelSystemBazelRCPath@") {} OptionProcessor::OptionProcessor( const WorkspaceLayout* workspace_layout, </ins>

Discussion:
Describe the bug By default and according to bazel documentation (), bazel is looking for a system-wide configuration is . In I decided to override this path and make it point in . It was the easiest solution to pass arguments to bazel in order to fix a few runtime issues. However it breaks the assumption that user can configure bazel using . To Reproduce Use bazel from nixpkgs with . Set some bazel settings in your file Observe that bazel ignores this file Expected behavior I'm opening this bug report to discuss what should be the expected behavior here? Should we respect the ? If yes, I'll open a pull request to restore the previous behavior.
I'll be in favour of keeping support for all paths specified in the documentation - having a system-wide is something people might want to make use of, so we shouldn't take away that possibility. We currently set the following options in bazelrc: We should either patch these as defaults into nixos, or introduce reading from a distro-provided location (should be coordinated with bazel upstream). I think adding a wrapper is a bit too much for all these options.
I do agree with your point of view. I'll try to propose a PR which restore the old behavior. In the meantime, I'm open to other comments on this issue. Regarding the options you quote, they are used in a local during build only. The global bazelrc that I provide is way simpler: Both were ad hoc solution for different problems, respectively issue and commit .
Change:
environment.etc."fish/foreign-env/interactiveShellInit".text = cfge.interactiveShellInit; environment.etc."fish/nixos-env-preinit.fish".text = '' <del> # avoid clobbering the environment if it's been set by a parent shell </del> # This happens before $__fish_datadir/config.fish sets fish_function_path, so it is currently # unset. We set it and then completely erase it, leaving its configuration to $__fish_datadir/config.fish set fish_function_path ${pkgs.fish-foreign-env}/share/fish-foreign-env/functions $__fish_datadir/functions

Discussion:
Fish usually has available, but the aliases are now missing, when the shell is not a login shell (presumably because of ). Have Then launch as a not-login shell. Try to use . System: 17.09pre107265. (Hummingbird)
Agh, what a mess. Thanks for raising this issue.
I think this issue is a little subtler than one would think: aliases are sourced, but only when the parent shell has not already sourced the interactive configuration. This is out of line with the behavior of the other shells on NixOS. I have a fix which I've tested with your example here. I will post a link to the relevant PR momentarily (after I put it up).
So apparently outgoing SSH is blocked from the WiFi on this cruise ship, so I can't push commits with the remotes I have set up, and my sister is nagging me to get off the computer so we can go out. If you need a quick fix on your own machine (I will be back later today, while I still have internet access, to submit the new PR), just change the lines in the NixOS module for fish () to and your aliases will get sourced every time the shell starts, so changes will be reflected without a reboot or manually unsetting the relevant variable before starting a new shell (). (The latter will do the trick if you want a quick fix without any changes to your Nixpkgs tree.)
there's a fix in this commit:
Thanks!

It looks like PR messed something up with fish environment as now connecting to a server with mosh fails giving fish: Unknown command 'mosh-server new -c 256 -s -l LANG=enUS.UTF-8 -l LCALL=enUS.UTF-8' fish: mosh-server 'new' '-c' '256' '-s' '-l' 'LANG=enUS.UTF-8' '-l' 'LCALL=enUS.UTF-8' I reverted the commit (it was mistakenly merged into 17.03) and mosh started working again. the system
I will likely not get a chance to really look into this until next week, but I will definitely figure out what went wrong here. Sorry about this! :-(
Don't worry too much, it's not something critical (hopefully it's the only thing affected). I'll try to sort this out though I have little experience here.
there's a fix in this commit:
Change:
<ins> { lib, stdenv, fetchFromGitHub, cmake, ninja, yasm , pkg-config, libjpeg, openssl, libopus, ffmpeg, alsaLib, libpulseaudio }: let rev = "c73a4718cbff7048373a63db32068482e5fd11ef"; sha256 = "0nr20mvvmmg8ii8f2rljd7iv2szplcfjn40rpy6llkmf705mwr1k"; in stdenv.mkDerivation { pname = "tg_owt"; version = "git-${rev}"; src = fetchFromGitHub { owner = "desktop-app"; repo = "tg_owt"; inherit rev sha256; }; patches = [ ./tg_owt-install.patch ]; nativeBuildInputs = [ pkg-config cmake ninja yasm ]; buildInputs = [ libjpeg openssl libopus ffmpeg alsaLib libpulseaudio ]; meta.license = lib.licenses.bsd3; } </ins>

Discussion:
, CVSSv3=2.4 (nixos-20.09) Scanned versions: nixos-20.09: . Cc Cc
Fixed by .
Change:
<ins> { lib , python3 , fetchFromGitHub }: python3.pkgs.buildPythonApplication rec { pname = "fritz-exporter"; version = "2.3.1"; pyproject = true; src = fetchFromGitHub { owner = "pdreker"; repo = "fritz_exporter"; rev = "fritzexporter-v${version}"; hash = "sha256-Dv/2Og1OJV7canZ8Y5Pai5gPRUvcRDYmSGoD2pnAkSs="; }; patches = [ # https://github.com/pdreker/fritz_exporter/pull/282 ./console-script.patch ]; postPatch = '' # don't test coverage sed -i "/^addopts/d" pyproject.toml ''; nativeBuildInputs = with python3.pkgs; [ poetry-core ]; propagatedBuildInputs = with python3.pkgs; [ attrs fritzconnection prometheus-client pyyaml requests ]; nativeCheckInputs = with python3.pkgs; [ pytestCheckHook ]; meta = { changelog = "https://github.com/pdreker/fritz_exporter/blob/${src.rev}/CHANGELOG.md"; description = "Prometheus exporter for Fritz!Box home routers"; homepage = "https://github.com/pdreker/fritz_exporter"; license = lib.licenses.asl20; mainProgram = "fritzexporter"; maintainers = with lib.maintainers; [ marie ]; }; } </ins>

Discussion:
Project description <!-- Describe the project a little: --Prometheus Exporter for AVM FritzBox Metadata homepage URL: source URL: license: AL2 platforms: linux, probably also darwin Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
There already is package called , but it seems abandoned. Does this new package have more/new metrics and/or is compatible with more FritzBox types?
I couldn't get to work on a machine where i was running the requested exporter before, so I would prefer to use this one. Not sure if its the models fault, but it's FRITZ!Box 5530 Fiber.
Change:
- [GoToSocial](https://gotosocial.org/), an ActivityPub social network server, written in Golang. Available as [services.gotosocial](#opt-services.gotosocial.enable). <ins> - [Typesense](https://github.com/typesense/typesense), a fast, typo-tolerant search engine for building delightful search experiences. Available as [services.typesense](#opt-services.typesense.enable). </ins> - [Anuko Time Tracker](https://github.com/anuko/timetracker), a simple, easy to use, open source time tracking system. Available as [services.anuko-time-tracker](#opt-services.anuko-time-tracker.enable). - [sitespeed-io](https://sitespeed.io), a tool that can generate metrics (timings, diagnostics) for websites. Available as [services.sitespeed-io](#opt-services.sitespeed-io.enable).

Discussion:
Project description Fast, typo tolerant, in-memory fuzzy Search Engine for building delightful search experiences. Metadata homepage URL: source URL: license: gpl3 platforms: linux, darwin
Change:
--prefix PATH : ${xdg-utils}/bin ''; <ins> passthru.updateScript = ./update.sh; </ins> meta = with lib; { description = "Wavebox messaging application"; homepage = "https://wavebox.io";

Discussion:
Package name: wavebox Latest released version: Current version on the unstable channel: Current version on the stable/release channel: (doesn't seem like the has anything to do with Wavebox 114 to 116?) Notify maintainers Note for maintainers: Please tag this issue in your PR.
Change:
<del> { stdenv, fetchurl, autoPatchelfHook, cmake, pkgconfig, libdrm, libpciaccess , libva , libX11, libXau, libXdmcp, libpthreadstubs }: </del> <ins> { stdenv, fetchurl, cmake, pkgconfig, gtest, libdrm, libpciaccess, libva, libX11 , libXau, libXdmcp, libpthreadstubs }: </ins> stdenv.mkDerivation rec { pname = "intel-media-sdk";

Discussion:
Describe the bug references because is in the inputs and it patches the samples at : To Reproduce Expected behavior No reference. Additional context I'm up to patchelf binaries that we ship, but: 1) depends on intel-media-sdk and users of it shouldn't get gcc in their closure for this. 2) We don't have even to ship the samples to everyone. Maybe split the outputs? Notify maintainers . Metadata Maintainer information:
I did a quick attempt at splitting the output, but it didn't work out very well. I'll have a further look this weekend.
Interesting... it also happens when is not in the inputs. As an alternative approach to split the outputs. This seems to work: But I wonder: Why do we ship samples? Perhaps:
BTW Why did you use ? It seems the samples are build from source... I think you've meant to them? That would have had to be done with: But this also fails with the same error: Which is super . Anyway I think we should leave off the samples:
I vaguely remember that there were linking problems with the sample binaries, mostly likely due to not being put in rpath.
I've had a further look into this problem, and it looks like changing the datadir from to would solve the problem with rpath, making patchelf unnecessary. Come to think of it, I think the samples belong to in the first place, but it's only a minor annoyance we can easily change. As of why I didn't exclude the samples, I'm rather hesitant at removing features unless there's a good reason for doing so. I'm all for removing it if it does turn out to cause problems, but I'd prefer taking a different approach if I can.
Actually, it looks like there no longer is a problem with rpath, so we can safely remove without changing the build flags at all.
Correction, there still were rpath problems present and changing datadir to didn't solve the problem. There are ways to deal with this besides using , but maybe you're right and it's easier to just remove the samples at this point. For reference, I did some quick search to see what other distros are doing: Debian: packages the samples in a separate package [1] Fedora: disables the samples all together [2] It's divided, but at least we're not alone if we do remove it. [1]: [2]:
Change:
# Prevent download/install of extensions touch "./install.lock" <del> # Mono runtimes from nix package (used by generated `OpenDebugAD7`). mv ./debugAdapters/bin/OpenDebugAD7 ./debugAdapters/bin/OpenDebugAD7_orig cp -p "${openDebugAD7Script}" "./debugAdapters/bin/OpenDebugAD7" </del> # Clang-format from nix package. <del> mv  ./LLVM/ ./LLVM_orig </del> <ins> mv ./LLVM/ ./LLVM_orig </ins> mkdir "./LLVM/" find "${clang-tools}" -mindepth 1 -maxdepth 1 | xargs ln -s -t "./LLVM" <del> # Patching  cpptools and cpptools-srv elfInterpreter="$(cat $NIX_CC/nix-support/dynamic-linker)" patchelf --set-interpreter "$elfInterpreter" ./bin/cpptools patchelf --set-interpreter "$elfInterpreter" ./bin/cpptools-srv chmod a+x ./bin/cpptools{-srv,} </del> <ins> # Patching binaries chmod +x bin/cpptools bin/cpptools-srv bin/cpptools-wordexp debugAdapters/bin/OpenDebugAD7 patchelf --replace-needed liblttng-ust.so.0 liblttng-ust.so.1 ./debugAdapters/bin/libcoreclrtraceptprovider.so </ins> ''; <del> meta = with lib; { license = licenses.unfree; maintainers = [ maintainers.jraygauthier ]; # A 32 bit linux would also be possible with some effort (specific download of binaries + # patching of the elf files with 32 bit interpreter). platforms = [ "x86_64-linux" ]; }; </del> <ins> postFixup = lib.optionalString gdbUseFixed '' wrapProgram $out/share/vscode/extensions/ms-vscode.cpptools/debugAdapters/bin/OpenDebugAD7 --prefix PATH : ${lib.makeBinPath [ gdb ]} ''; meta = with lib; { description = "The C/C++ extension adds language support for C/C++ to Visual Studio Code, including features such as IntelliSense and debugging."; homepage = "https://marketplace.visualstudio.com/items?itemName=ms-vscode.cpptools"; license = licenses.unfree; maintainers = with maintainers; [ jraygauthier stargate01 ]; platforms = [ "x86_64-linux" ]; }; </ins> }

Discussion:
OpenDebug does not work. vscode-cpptools working opendebug -not work !
correct path ) !
Change:
lib, stdenv, fetchFromGitHub, <del> python3, </del> <ins> python311, </ins> pkg-config, SDL2, libpng,

Discussion:
Package name: renpy Latest released version: 8.3.2 <!-- Search your package here: --- Current version on the unstable channel: 8.2.1. Current version on the stable/release channel: 8.2.1. <!-- 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:
owner = "KhronosGroup"; repo = "Vulkan-Tools"; rev = "sdk-${version}"; <del> sha256 = "0izmzyj6gb51d71vbdjcgd9qw34aidvbmz0mg4bkc13n48w8s9vj"; </del> <ins> sha256 = "sha256-oNJm9Gi41aA5krkpkQI0EYdIlMcQpdodv9yqXhnNURA="; </ins> }); nativeBuildInputs = [ cmake ];

Discussion:
vulkan-tools fails with an assertion error from vulkan-headers Steps to reproduce the behavior: the nixos-unstable-small channel vulkan-tools being installed # nix-shell -p vulkan-tools --show-trace error: assertion '(version == (vulkan-headers).version)' failed at /nix/store/cqbcz8k1wq604f8b8nc8khamyh2cl6dl-nixos-22.05pre348006./nixos/pkgs/tools/graphics/vulkan- 10| # the rest of nixpkgs. 11| src = (assert version == vulkan-headers.version; | ^ 12| fetchFromGitHub { … while evaluating the attribute 'src' of the derivation 'vulkan-tools-1.2.189.1' at /nix/store/cqbcz8k1wq604f8b8nc8khamyh2cl6dl-nixos-22.05pre348006./nixos/pkgs/stdenv/generic/make- 204 (attrs ? pname && attrs ? version)) { 205| name = | ^ 206| let … while evaluating the attribute 'buildInputs' of the derivation 'shell' at /nix/store/cqbcz8k1wq604f8b8nc8khamyh2cl6dl-nixos-22.05pre348006./nixos/pkgs/stdenv/generic/make- 204 (attrs ? pname && attrs ? version)) { 205| name = | ^ 206| let
Fix at , thanks for the report. I thought we had tests for this stuff...
fixed it for me. Thanks!
Change:
occ = pkgs.writeScriptBin "nextcloud-occ" '' #! ${pkgs.runtimeShell} <del> cd ${cfg.package} </del> <ins> cd ${webroot} </ins> sudo=exec if [[ "$USER" != nextcloud ]]; then sudo='exec /run/wrappers/bin/sudo -u nextcloud --preserve-env=NEXTCLOUD_CONFIG_DIR --preserve-env=OC_PASS'

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:
''; }; <ins> testToINIDuplicateKeys = { expr = generators.toINI { listsAsDuplicateKeys = true; } { foo.bar = true; baz.qux = [ 1 false ]; }; expected = '' [baz] qux=1 qux=false [foo] bar=true ''; }; </ins> testToINIDefaultEscapes = { expr = generators.toINI {} { "no [ and ] allowed unescaped" = {

Discussion:
Describe the bug I'd like the function to accept lists and produce duplicate keys, like so: which would yield:
ping to see if he has any interest in implementing this :heart:
Done in :)
Change:
++ lib.optionals stdenv.hostPlatform.isLinux [ linux-pam ] ++ lib.optional (stdenv.hostPlatform.isLinux && stdenv.hostPlatform.isx86_64) pmdk.dev ++ lib.optional (!stdenv.hostPlatform.isDarwin) mytopEnv <del> ++ lib.optionals withStorageMroonga [ kytea libsodium msgpack zeromq ] ++ lib.optionals (lib.versionAtLeast common.version "10.7") [ fmt_8 ]; </del> <ins> ++ lib.optionals withStorageMroonga [ kytea libsodium msgpack zeromq ]; </ins> patches = common.patches;

Discussion:
MariaDB 10.6.6 does not seem to exist at the given source.
Yeah, they unpublished those releases. I'd suggest just reverting the updates for now
Change:
aiorun = callPackage ../development/python-modules/aiorun { }; <ins> aiorwlock = callPackage ../development/python-modules/aiorwlock { }; </ins> aiosenseme = callPackage ../development/python-modules/aiosenseme { }; aiosenz = callPackage ../development/python-modules/aiosenz { };

Discussion:
Project description Ray is a fast and simple framework for building and running distributed applications. It is packaged with RLlib, a scalable reinforcement learning library. The project provides a wheel at , but according to NixOs is by default not compatible. Metadata homepage URL: source URL: license: apache2 platforms: unix, linux
I'm interested in this as well, when I try to use any sort of non-local mode with ray installed in a nix-shell environment via pip, it throws errors launching redis. Even though redis-server exists at the aforemention path. It fails to launch when I attempt to execute redis directly.
This issue has been mentioned on NixOS Discourse. There might be relevant details there:
I haven't tested, but since manylinux1 is available by default now (), does it work on your machine on the latest unstable?
I still have the issue with redis unfortunately
this seems to work
I see, the s fixed it.
I marked this as stale due to inactivity. &rarr;
Still important to me.
I marked this as stale due to inactivity. &rarr;
In case anyone is still looking for this - it is packaged in python-on-nix
Change:
<del> {stdenv, fetchurl, sendmailPath ? "/usr/sbin/sendmail"}: </del> <ins> {stdenv, fetchurl, vim, sendmailPath ? "/usr/sbin/sendmail"}: </ins> stdenv.mkDerivation { name = "cron-4.1";

Discussion:
When I run `sudo crontab -e", I get no crontab for root - using an empty one /bin/sh: /usr/bin/vi: No such file or directory crontab: "/usr/bin/vi" exited with status 127 Now I agree that I probably shouldn't do that (I should keep my crontab in ), but a better error message would probably be better. Please run and paste the results. system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
I also have this issue. Current work around is but this needs to be fixed. See
Please feel free to close as this issue has been fixed.
Change:
]; prePatch = '' <del> sed 's_shotcutPath, "qmelt"_"${mlt}/bin/melt"_' -i src/jobs/meltjob.cpp </del> <ins> sed 's_shotcutPath, "melt"_"${mlt}/bin/melt"_' -i src/jobs/meltjob.cpp </ins> sed 's_shotcutPath, "ffmpeg"_"${mlt.ffmpeg}/bin/ffmpeg"_' -i src/jobs/ffmpegjob.cpp sed 's_qApp->applicationDirPath(), "ffmpeg"_"${mlt.ffmpeg}/bin/ffmpeg"_' -i src/docks/encodedock.cpp NICE=$(type -P nice)

Discussion:
Describe the bug It appears that the binary is not included in the output of the derivation. To Reproduce Steps to reproduce the behavior: in NixOS 20.09. a video, e.g. by adding a solid color with “Open Other” → “Color” & dragging it to the timeline. to export with default settings. Expected behavior Video is exported successfully. Actual behavior Export fails with the following log message: Workaround separately. file after failed export. to create the exported video. Notify maintainers Metadata Maintainer information:
Thanks for your report, you are right, export is broken. But was also not included in $out/bin in older versions, so I guess there is a path to mlt missing somewhere, so shotcut does not find melt. We have to find out where.
found it, was changed to , just preparing a PR
I have to apologize for not testing the export after updating to shotcut-20.10.31
Thanks, !