jiebi/CodeConvo huggingface dataset:
Code change:
Package name: tmuxinator Latest released version: 3.1.2Current version on the unstable channel: 3.0.5 Current version on the stable/release channel: 3.0.5 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]:

Related discussion:
text: inherit ruby; name = "${gemName}-${version}"; gemName = "tmuxinator"; <del> version = "3.0.5"; source.sha256 = "1ycsx9mvl0jsds4igi6avxclsyl5lndh1mpj2ykvzfz26wdddg6p"; </del> <ins> version = "3.1.2"; source.sha256 = "t96v3RwBfezB644RVbTmO8i/xh1F23WQC8PBxc/msjg="; </ins> erubis = buildRubyGem rec { inherit ruby;
Code change:
Package name: tmuxinator Latest released version: 3.1.2Current version on the unstable channel: 3.0.5 Current version on the stable/release channel: 3.0.5 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]:

Related discussion:
text: inherit ruby; name = "ruby${ruby.version}-${gemName}-${version}"; gemName = "thor"; <del> version = "1.2.1"; source.sha256 = "0inl77jh4ia03jw3iqm5ipr76ghal3hyjrd6r8zqsswwvi9j2xdi"; </del> <ins> version = "1.3.0"; source.sha256 = "sha256-Gtx/nls2VaaMcTk/7ovQrQiNFO6Og6C3NybyPLs8p8M="; </ins> }; xdg = buildRubyGem rec {
Code change:
Package name: lightgbm Latest released version: 4.3.0 <!-- Search your package here: --- Current version on the unstable channel: 4.1.0 Current version on the stable/release channel: 4.1.0 <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --- Version 4.1.0 gets broken by . Version 4.3.0 incorporates which builds tests with C++14. 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]:
Thanks. I'll have a look

Related discussion:
text: #   in  #   rWrapper.override{ packages = [ lgbm ]; }" pname = lib.optionalString rLibrary "r-" + pnameBase; <del> version = "4.1.0"; </del> <ins> version = "4.3.0"; </ins> src = fetchFromGitHub { owner = "microsoft"; repo = pnameBase; rev = "v${version}"; fetchSubmodules = true; <del> hash = "sha256-AhXe/Mlor/i0y84wI9jVPKSnyVbSyAV52Y4yiNm7yLQ="; </del> <ins> hash = "sha256-hEoGdzC6n8t14ZUFWFrdljEkQRo9qaDGYTamvIAgrbg="; </ins> }; nativeBuildInputs = [ cmake ]
Code change:
Package name: lightgbm Latest released version: 4.3.0 <!-- Search your package here: --- Current version on the unstable channel: 4.1.0 Current version on the stable/release channel: 4.1.0 <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --- Version 4.1.0 gets broken by . Version 4.3.0 incorporates which builds tests with C++14. 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]:
Thanks. I'll have a look

Related discussion:
text: propagatedBuildInputs = lib.optionals rLibrary [ rPackages.data_table <ins> rPackages.markdown </ins> rPackages.rmarkdown rPackages.jsonlite rPackages.Matrix
Code change:
Package name: lightgbm Latest released version: 4.3.0 <!-- Search your package here: --- Current version on the unstable channel: 4.1.0 Current version on the stable/release channel: 4.1.0 <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --- Version 4.1.0 gets broken by . Version 4.3.0 incorporates which builds tests with C++14. 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]:
Thanks. I'll have a look

Related discussion:
text: --replace  "install_args <- c("CMD", "INSTALL", "--no-multiarch", "--with-keep.source", tarball)"  "install_args <- c("CMD", "INSTALL", "--no-multiarch", "--with-keep.source", "-l $out/library", tarball)" <ins> # Retry this test in next release. Something fails in the setup, so GTEST_FILTER is not enough rm tests/cpp_tests/test_arrow.cpp </ins> ''; cmakeFlags = lib.optionals doCheck [ "-DBUILD_CPP_TEST=ON" ]
Code change:
I want to see systemd services in netdata monitoring. For that "cgroup accounting" is needed. The question is if enabling it on all systems would have any negative effect. In that case i would activate it only when netdata service is enabled. Arch Linux, Ubuntu and Fedora seem to have it enabled by default. debian not. shows no resources for services. System: NixOS 17.03.1700. (Gorilla) Nix version: nix-env (Nix) 1.11.13 Nixpkgs version: "17.03.1700." Sandboxing enabled: build-use-sandbox = true
I will implement it as option, that is enabled by default and for netdata service.
I believe you can set this with: And maybe also: (ideas from )
sure, that is how i do it right now. But i want to propose it to be enabled by default, so you have it when you need it for debugging. But also a single option to disable it when you have reasons to.
Is this still open for the discussion about defaulting to or is this still open simply because it wasn't closed? Unless something else than the PR is required? (
I created this issue to have this enabled for netdata. So an additional PR needs to be created to enable the new option when netdata is enabled. In addition it could be discussed to enable the option by default when a desktopManager is enabled (for desktop users). It should not be enabled by default on servers because it could have a little impact on performance.
Nowadays, this might already be enabled per default half-way: from my `systemd- (latest master right now) Are there any recent figures about performance implications when enabling the rest? If Fedora enables it, we could probably do so too.
this might already be enabled per default half-way thanks for noticing! can you create an issue or PR to change ? Are there any recent figures about performance implications when enabling the rest? i don't know any. i have no issues on my servers where i set it manually

Related discussion:
text: }; }; <ins> systemd.enableCgroupAccounting = true; </ins> security.wrappers."apps.plugin" = { source = "${pkgs.netdata}/libexec/netdata/plugins.d/apps.plugin.org"; capabilities = "cap_dac_read_search,cap_sys_ptrace+ep";
Code change:
I want to see systemd services in netdata monitoring. For that "cgroup accounting" is needed. The question is if enabling it on all systems would have any negative effect. In that case i would activate it only when netdata service is enabled. Arch Linux, Ubuntu and Fedora seem to have it enabled by default. debian not. shows no resources for services. System: NixOS 17.03.1700. (Gorilla) Nix version: nix-env (Nix) 1.11.13 Nixpkgs version: "17.03.1700." Sandboxing enabled: build-use-sandbox = true
I will implement it as option, that is enabled by default and for netdata service.
I believe you can set this with: And maybe also: (ideas from )
sure, that is how i do it right now. But i want to propose it to be enabled by default, so you have it when you need it for debugging. But also a single option to disable it when you have reasons to.
Is this still open for the discussion about defaulting to or is this still open simply because it wasn't closed? Unless something else than the PR is required? (
I created this issue to have this enabled for netdata. So an additional PR needs to be created to enable the new option when netdata is enabled. In addition it could be discussed to enable the option by default when a desktopManager is enabled (for desktop users). It should not be enabled by default on servers because it could have a little impact on performance.
Nowadays, this might already be enabled per default half-way: from my `systemd- (latest master right now) Are there any recent figures about performance implications when enabling the rest? If Fedora enables it, we could probably do so too.
this might already be enabled per default half-way thanks for noticing! can you create an issue or PR to change ? Are there any recent figures about performance implications when enabling the rest? i don't know any. i have no issues on my servers where i set it manually

Related discussion:
text: been removed. </para> </listitem> <del> <listitem> <para> The <literal>rmilter</literal> package was removed with associated module and options due deprecation by upstream developer. Use <literal>rspamd</literal> in proxy mode instead. </para> </listitem> </del> <ins> <listitem> <para> The <literal>rmilter</literal> package was removed with associated module and options due deprecation by upstream developer. Use <literal>rspamd</literal> in proxy mode instead. </para> </listitem> <listitem> <para> systemd cgroup accounting via the <link linkend="opt-systemd.enableCgroupAccounting">systemd.enableCgroupAccounting</link> option is now enabled by default. It now also enables the more recent Block IO and IP accounting features. </para> </listitem> </ins> </itemizedlist> </section> </section>
Code change:
I want to see systemd services in netdata monitoring. For that "cgroup accounting" is needed. The question is if enabling it on all systems would have any negative effect. In that case i would activate it only when netdata service is enabled. Arch Linux, Ubuntu and Fedora seem to have it enabled by default. debian not. shows no resources for services. System: NixOS 17.03.1700. (Gorilla) Nix version: nix-env (Nix) 1.11.13 Nixpkgs version: "17.03.1700." Sandboxing enabled: build-use-sandbox = true
I will implement it as option, that is enabled by default and for netdata service.
I believe you can set this with: And maybe also: (ideas from )
sure, that is how i do it right now. But i want to propose it to be enabled by default, so you have it when you need it for debugging. But also a single option to disable it when you have reasons to.
Is this still open for the discussion about defaulting to or is this still open simply because it wasn't closed? Unless something else than the PR is required? (
I created this issue to have this enabled for netdata. So an additional PR needs to be created to enable the new option when netdata is enabled. In addition it could be discussed to enable the option by default when a desktopManager is enabled (for desktop users). It should not be enabled by default on servers because it could have a little impact on performance.
Nowadays, this might already be enabled per default half-way: from my `systemd- (latest master right now) Are there any recent figures about performance implications when enabling the rest? If Fedora enables it, we could probably do so too.
this might already be enabled per default half-way thanks for noticing! can you create an issue or PR to change ? Are there any recent figures about performance implications when enabling the rest? i don't know any. i have no issues on my servers where i set it manually

Related discussion:
text: }; systemd.enableCgroupAccounting = mkOption { <del> default = false; </del> <ins> default = true; </ins> type = types.bool; description = '' Whether to enable cgroup accounting.
Code change:
I want to see systemd services in netdata monitoring. For that "cgroup accounting" is needed. The question is if enabling it on all systems would have any negative effect. In that case i would activate it only when netdata service is enabled. Arch Linux, Ubuntu and Fedora seem to have it enabled by default. debian not. shows no resources for services. System: NixOS 17.03.1700. (Gorilla) Nix version: nix-env (Nix) 1.11.13 Nixpkgs version: "17.03.1700." Sandboxing enabled: build-use-sandbox = true
I will implement it as option, that is enabled by default and for netdata service.
I believe you can set this with: And maybe also: (ideas from )
sure, that is how i do it right now. But i want to propose it to be enabled by default, so you have it when you need it for debugging. But also a single option to disable it when you have reasons to.
Is this still open for the discussion about defaulting to or is this still open simply because it wasn't closed? Unless something else than the PR is required? (
I created this issue to have this enabled for netdata. So an additional PR needs to be created to enable the new option when netdata is enabled. In addition it could be discussed to enable the option by default when a desktopManager is enabled (for desktop users). It should not be enabled by default on servers because it could have a little impact on performance.
Nowadays, this might already be enabled per default half-way: from my `systemd- (latest master right now) Are there any recent figures about performance implications when enabling the rest? If Fedora enables it, we could probably do so too.
this might already be enabled per default half-way thanks for noticing! can you create an issue or PR to change ? Are there any recent figures about performance implications when enabling the rest? i don't know any. i have no issues on my servers where i set it manually

Related discussion:
text: [Manager] ${optionalString config.systemd.enableCgroupAccounting '' DefaultCPUAccounting=yes <ins> DefaultBlockIOAccounting=yes </ins> DefaultIOAccounting=yes DefaultBlockIOAccounting=yes <del> DefaultMemoryAccounting=yes DefaultTasksAccounting=yes </del> <ins> DefaultIPAccounting=yes </ins> ''} DefaultLimitCORE=infinity ${config.systemd.extraConfig}
Code change:
I want to see systemd services in netdata monitoring. For that "cgroup accounting" is needed. The question is if enabling it on all systems would have any negative effect. In that case i would activate it only when netdata service is enabled. Arch Linux, Ubuntu and Fedora seem to have it enabled by default. debian not. shows no resources for services. System: NixOS 17.03.1700. (Gorilla) Nix version: nix-env (Nix) 1.11.13 Nixpkgs version: "17.03.1700." Sandboxing enabled: build-use-sandbox = true
I will implement it as option, that is enabled by default and for netdata service.
I believe you can set this with: And maybe also: (ideas from )
sure, that is how i do it right now. But i want to propose it to be enabled by default, so you have it when you need it for debugging. But also a single option to disable it when you have reasons to.
Is this still open for the discussion about defaulting to or is this still open simply because it wasn't closed? Unless something else than the PR is required? (
I created this issue to have this enabled for netdata. So an additional PR needs to be created to enable the new option when netdata is enabled. In addition it could be discussed to enable the option by default when a desktopManager is enabled (for desktop users). It should not be enabled by default on servers because it could have a little impact on performance.
Nowadays, this might already be enabled per default half-way: from my `systemd- (latest master right now) Are there any recent figures about performance implications when enabling the rest? If Fedora enables it, we could probably do so too.
this might already be enabled per default half-way thanks for noticing! can you create an issue or PR to change ? Are there any recent figures about performance implications when enabling the rest? i don't know any. i have no issues on my servers where i set it manually

Related discussion:
text: $machine->waitForUnit('multi-user.target'); $machine->succeed('sysctl net.core.default_qdisc | grep -q "fq_codel"'); }; <ins> # Test cgroup accounting is enabled subtest "systemd cgroup accounting is enabled", sub { $machine->waitForUnit('multi-user.target'); $machine->succeed('systemctl show testservice1.service -p IOAccounting | grep -q "yes"'); $machine->succeed('systemctl status testservice1.service | grep -q "CPU:"'); }; </ins> ''; })
Code change:
Project description A fast, reliable and cross-platform command-line Minecraft launcher Metadata homepage URL: source URL: license: GPL-3.0 platforms: linux, windows (I can't test darwin) Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: <ins> { lib, stdenv, python3Packages, fetchFromGitHub, installShellFiles, jre, libX11, libXext, libXcursor, libXrandr, libXxf86vm, libpulseaudio, libGL, glfw, openal, udev, textToSpeechSupport ? stdenv.isLinux, flite, }: let # Copied from the `prismlauncher` package runtimeLibs = [ libX11 libXext libXcursor libXrandr libXxf86vm # lwjgl libpulseaudio libGL glfw openal stdenv.cc.cc.lib # oshi udev ] ++ lib.optional textToSpeechSupport flite; in python3Packages.buildPythonApplication rec { pname = "portablemc"; version = "4.3.0"; pyproject = true; disabled = python3Packages.pythonOlder "3.8"; src = fetchFromGitHub { owner = "mindstorm38"; repo = "portablemc"; rev = "v${version}"; hash = "sha256-jCv4ncXUWbkWlBZr3P1hNeVpdQzY9HtrFz+pmKknL0I="; }; patches = [ # Use the jre package provided by nixpkgs by default ./use-builtin-java.patch ]; nativeBuildInputs = [ installShellFiles ]; build-system = [ python3Packages.poetry-core ]; dependencies = [ python3Packages.certifi ]; # Note: Tests use networking, so we don't run them postInstall = '' installShellCompletion --cmd portablemc  --bash <($out/bin/portablemc show completion bash)  --zsh <($out/bin/portablemc show completion zsh) ''; preFixup = '' makeWrapperArgs+=( --prefix LD_LIBRARY_PATH : ${lib.makeLibraryPath runtimeLibs} --prefix PATH : ${lib.makeBinPath [ jre ]} ) ''; meta = { homepage = "https://github.com/mindstorm38/portablemc"; description = "A fast, reliable and cross-platform command-line Minecraft launcher and API for developers"; longDescription = '' A fast, reliable and cross-platform command-line Minecraft launcher and API for developers. Including fast and easy installation of common mod loaders such as Fabric, Forge, NeoForge and Quilt. This launcher is compatible with the standard Minecraft directories. ''; changelog = "https://github.com/mindstorm38/portablemc/releases/tag/${src.rev}"; license = lib.licenses.gpl3Only; mainProgram = "portablemc"; maintainers = with lib.maintainers; [ tomasajt ]; }; } </ins>
Code change:
Project description A fast, reliable and cross-platform command-line Minecraft launcher Metadata homepage URL: source URL: license: GPL-3.0 platforms: linux, windows (I can't test darwin) Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:

Related discussion:
text: <ins> diff --git a/portablemc/standard.py b/portablemc/standard.py index f59c55d..0f017e1 100644 --- a/portablemc/standard.py +++ b/portablemc/standard.py @@ -843,6 +843,8 @@ class Version: if jvm_major_version is not None and not isinstance(jvm_major_version, int): raise ValueError("metadata: /javaVersion/majorVersion must be an integer") +        return self._resolve_builtin_jvm(watcher, JvmNotFoundError.UNSUPPORTED_ARCH, jvm_major_version) + if platform.system() == "Linux" and platform.libc_ver()[0] != "glibc": return self._resolve_builtin_jvm(watcher, JvmNotFoundError.UNSUPPORTED_LIBC, jvm_major_version) @@ -926,31 +928,10 @@ class Version: builtin_path = shutil.which(jvm_bin_filename) if builtin_path is None: raise JvmNotFoundError(reason) - -        try: - -            # Get version of the JVM. -            process = Popen([builtin_path, "-version"], bufsize=1, stdout=PIPE, stderr=STDOUT, universal_newlines=True) -            stdout, _stderr = process.communicate(timeout=1) - -            version_start = stdout.index(f"1.{major_version}" if major_version <= 8 else str(major_version)) -            version = None - -            # Parse version by getting all character that are numeric or '.'. -            for i, ch in enumerate(stdout[version_start:]): -                if not ch.isnumeric() and ch not in (".", "_"): -                    version = stdout[version_start:i] -                    break - -            if version is None: -                raise ValueError() - -        except (TimeoutExpired, ValueError): -            raise JvmNotFoundError(JvmNotFoundError.BUILTIN_INVALID_VERSION) self._jvm_path = Path(builtin_path) -        self._jvm_version = version -        watcher.handle(JvmLoadedEvent(version, JvmLoadedEvent.BUILTIN)) +        self._jvm_version = "nixpkgs" +        watcher.handle(JvmLoadedEvent("nixpkgs", JvmLoadedEvent.BUILTIN)) def _download(self, watcher: Watcher) -> None: </ins>
Code change:
I noticed that in this line code actually checks true value of not that is declared at the top. Because of this, dlib doesn’t find the cuDNN libraries and compile without CUDA support. Steps to reproduce the behavior: with and without logs of both dlib compilation. One with will tell that it found cuda and cuDNN and dlib will use cuda, other will fail to find cuDNN and will tell that CUDA installation maybe incomplete and disables CUDA support. When the cudaSupport option defined in Dlib is enabled, dlib can be compiled with CUDA support without setting to true Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Ah good spot, I'll try and put in a PR to fix that tonight. Thanks
Also, I believe this issue can be related to this issue. For people affected by this they can enable = true; globally (which causes a bit of mass rebuilds on system). Also, I’m wondering why options defined in are not searchable in which could save me a bit of time.
Thank you too!
Leaving this here for history:

Related discussion:
text: lapack ] ++ lib.optionals guiSupport [ libX11 ] <del> ++ lib.optionals config.cudaSupport (with cudaPackages; [ </del> <ins> ++ lib.optionals cudaSupport (with cudaPackages; [ </ins> cuda_cudart.dev cuda_cudart.lib cuda_cudart.static
Code change:
Project description UI for building, configuring and running Monado, the open source OpenXR runtime. Metadata homepage URL: source URL: license: agpl3 platforms: unix, linux, darwin Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
To my information, software without a stable release isnt accepted yet.
Not even in unstable? Fair enough, I guess.
unstable automatically becomes stable, so yes :) I think once a release is done (even 0.x), it can be packaged, but don't take my word as the ultimate truth, please :sweat_smile:
This appears to be incorrect. was merged targeting a specific commit (despite actually having stable releases).
This is indeed incorrect; there are even specific rules for the attribute of unreleased versions.

Related discussion:
text: - [wg-access-server](https://github.com/freifunkMUC/wg-access-server/), an all-in-one WireGuard VPN solution with a web ui for connecting devices. Available at [services.wg-access-server](#opt-services.wg-access-server.enable). <ins> - [Envision](https://gitlab.com/gabmus/envision), a UI for building, configuring and running Monado, the open source OpenXR runtime. Available as [programs.envision](#opt-programs.envision.enable). </ins> - [Playerctld](https://github.com/altdesktop/playerctl), a daemon to track media player activity. Available as [services.playerctld](option.html#opt-services.playerctld). ## Backward Incompatibilities {#sec-release-24.11-incompatibilities}