jiebi/CodeConvo huggingface dataset:
Change:
}; listen.port = mkOption { <del> type = types.port; default = 4242; </del> <ins> type = types.nullOr types.port; default = null; defaultText = lib.literalExpression '' if (config.services.nebula.networks.''${name}.isLighthouse || config.services.nebula.networks.''${name}.isRelay) then 4242 else 0; ''; </ins> description = lib.mdDoc "Port number to listen on."; };

Discussion:
When a "lighthouse" and a regular "roaming" node are on the same network and the default value for () is used, nodes occasionally enter a consistent failure state while trying to connect. I suspect this occurs when Nebula tries to NAT-punch with the local router and always tries to connect to the lighthouse instead of the roaming node. Steps to reproduce the behavior: up a lighthouse node and roaming node -forward port 4242 for the lighthouse node to set a value for in the Nebula config intermittent failures The value of should be required, with a link to Nebula docs: I'm not entirely sure, but logs from the issue indicate the lighthouse-node handshakes are both sending packets to "192.168.1.1:4242", which is port-forwarded to the lighthouse (I'm in a dual-NAT setup unfortunately; this is the IP of the outer NAT). Explicitly setting the port to (to automatically select an ephemeral port) on the roaming nodes immediately fixes the issue. (wrote the initial module with the port default) Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Based on those docs, I think we can solve this without requiring the user to specify the port. Since we know whether a node is a lighthouse via , we could implement a behavior where defaults to for lighthouses and for non-lighthouses: I'm not sure how to get the for this, though. Maybe if the options were , then we could just refer to ? We'd probably also want to include . Thoughts?
My thought is that a change should meet the following requirements: Per set and to something like:Set the actual value appropriately in the module with an expression like the above if the input is null. If (isLighthouse or isRelay) and the final port evaluates to 0, we should warn, since this config will probably result in connectivity issues. There should be a test for this. I am split on whether to backport this. While this is a bug, people may have relied on the default port to NAT port forward from routers to a Nebula node that is not a lighthouse or a relay.

As such the new default may impede connectivity on restrictive NATs, so this is also a breaking change.
Also, good job diagnosing this. I think I've run into this before but didn't know what was causing it.
Per (captured in + ) Given this, it is probably a good idea to both fix and backport this.
Change:
$n++; $rootDir = $ARGV[$n]; die "$0: ‘--root’ requires an argumentn" unless defined $rootDir; <ins> die "$0: no need to specify `/` with `--root`, it is the defaultn" if $rootDir eq "/"; </ins> $rootDir =~ s//*$//; # remove trailing slashes <ins> $rootDir = File::Spec->rel2abs($rootDir); # resolve absolute path </ins> } elsif ($arg eq "--force") { $force = 1;

Discussion:
When you are installing NixOS and running , if you specify i.e. when current directory is , the script does not resolve the absolute path to the specified directory, so it does not generate file systems paths in . Steps to reproduce the behavior: to install NixOS how it's described in the guide the step with , proceed to directory and run instead Script resolves the path of argument and generates proper <img width="933" alt="Снимок экрана 2022-02-20 в 17 14 50" src="https://user-"Not applicable
Change:
githubId = 8211181; name = "Kevin Kandlbinder"; }; <ins> kfears = { email = "kfearsoff@gmail.com"; github = "KFearsoff"; githubId = 66781795; matrix = "@kfears:matrix.org"; name = "KFears"; }; </ins> kfollesdal = { email = "kfollesdal@gmail.com"; github = "kfollesdal";

Discussion:
Project description This is the next generation OpenVPN client for Linux. I am having issues connecting to my work OpenVPN server, everyone else that is connecting from linux succesfully is using this package homepage source URL: license: AGPL-3.0 License
I marked this as stale due to inactivity. &rarr;
Not stale.
Here is a draft at packaging, it only seems to work using latest unstable at the time of this comment: It's amazing how quickly these packagings accumulate hacks and seds and shebang patches
Change:
# Autogenerated by maintainers/scripts/gnome.sh update fetchurl: { <del> name = "libgit2-glib-0.23.6"; </del> <ins> name = "libgit2-glib-0.24.0"; </ins> src = fetchurl { <del> url = mirror://gnome/sources/libgit2-glib/0.23/libgit2-glib-0.23.6.tar.xz; sha256 = "5c8d6b5cb81ab64b96cb0c52ef37463b6d8998a40ce77a08eda9db66fb781144"; </del> <ins> url = mirror://gnome/sources/libgit2-glib/0.24/libgit2-glib-0.24.0.tar.xz; sha256 = "d616c268821c28ff8dc1a6419dbf8555fa48e31dc6509c10f5151be7690f4845"; </ins> }; }

Discussion:
The update to libgit2 in broke compilation of the version of libgit2-glib, and thus gitg. Try to build it from . NixOS on
Change:
deployAndroidPackage { inherit package os; buildInputs = [ autoPatchelfHook ] <del> ++ lib.optional (os == "linux") [ pkgs.glibc pkgs.stdenv.cc.cc pkgs.zlib pkgs.openssl.out pkgs.ncurses5 ]; </del> <ins> ++ lib.optional (os == "linux") [ pkgs.glibc pkgs.stdenv.cc.cc pkgs.zlib pkgs.openssl_1_0_2.out pkgs.ncurses5 ]; </ins> patchInstructions = lib.optionalString (os == "linux") '' addAutoPatchelfSearchPath $packageBaseDir/lib autoPatchelf $packageBaseDir/lib

Discussion:
Describe the bug LLDB is not provided by default in NixOS's Android SDK. On enabling this package, I found out that the command failed to discover and , resulting of the following trace. To Reproduce Steps to reproduce the behavior: a file with the following content, using . Expected behavior Result in installed derivation. Metadata Please run and paste the result. Maintainer information:
Replacing openssl with openssl10_2 should fix it here:
Change:
# libuuid, slowing down program startup a lot). ./no-ldconfig.patch <ins> # Fix ctypes.util.find_library with gcc10. ./find_library-gcc10.patch </ins> ] ++ optionals stdenv.hostPlatform.isCygwin [ ./2.5.2-ctypes-util-find_library.patch ./2.5.2-tkinter-x11.patch

Discussion:
Describe the bug Can be noticed when building . Since 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 Nixpkgs at Maintainer information:
is this still relevant with ?
nope fixed with
Change:
owner = "xonsh"; repo = "xonsh"; rev = "refs/tags/${version}"; <del> hash = "sha256-7qOEc4RSdOO059LietKnrxY7cy9MfgmfJjdqZZ5ENLU="; </del> <ins> hash = "sha256-mHOCkUGiSSPmkIQ4tgRZIaCTLgnx39SMwug5EIx/jrU="; </ins> }; nativeBuildInputs = with python3.pkgs; [

Discussion:
Using xonsh in a environment can override the Python interpreter, which is a problem if the environment is supposed to use a specific one. Then create containing Then inside xonsh, a different Python interpreter is used: Running xonsh should not modify the such that a different executable is used. In the above, when we run we should get the library. Running removes the problematic PATH entry but this is pretty hacky. Please run and paste the result.
I think the recommended way to do this is to add an override to xonsh. See
That's a help but if I launch a nested xonsh by running in xonsh, the extra packages are gone. I've removed these unwanted entries by doing, e.g. in : Everything works fine so far.
since your PR was is this issue still fixed? ETA: looks ok but if anyone still has in their config they should probably remove it before upgrading xonsh or they may get
To make it clear, was reapplied in , while reverting
Oops, it seems we have re-introduced this issue in
Change:
<ins> { lib , stdenv , autoreconfHook , fetchFromGitHub , bzip2 , xz , zlib }: stdenv.mkDerivation (finalAttrs: { # Same name as the Debian library pname = "libstaden-read"; version = "1.15.0"; src = fetchFromGitHub { owner = "jkbonfield"; repo = "io_lib"; rev = "io_lib-" + builtins.replaceStrings ["."] ["-"] finalAttrs.version; fetchSubmodules = true; hash = "sha256-2Dlx+MXmqar81/Xmf0oE+6lWX461EDYijiZsZf/VD28="; }; patches = [ # Needed so that the lib can be detected ./libstaden-install-config-header.patch ]; buildInputs = [ bzip2 xz zlib ]; nativeBuildInputs = [ autoreconfHook ]; # autoreconfHook does not descend into htscodecs folder preAutoreconf = '' pushd ./htscodecs autoreconf --install --force --verbose pushd ''; meta = { description = "C library for reading/writing various DNA sequence formats"; homepage = "https://staden.sourceforge.net"; downloadPage = "https://github.com/jkbonfield/io_lib/releases"; changelog = "https://github.com/jkbonfield/io_lib/blob/${finalAttrs.src.rev}/CHANGES"; license = with lib.licenses; [ bsd3 free ]; platforms = lib.platforms.all; maintainers = [ lib.maintainers.kupac ]; }; }) </ins>

Discussion:
Salmon is a bioinformatics tool to produce a highly accurate, transcript level quantification estimates from short-read RNA-seq data. It's mostly written in C++. There's a similar tool already in nixpkgs (Kallisto), but they both have a few unique features, so packaging Salmon is still useful. It's also quite a popular tool. homepage URL: source URL: license: GLP v3.0 platforms: linux, others?
I started packaging it, and made good progress. It's a way for me to learn Nix, but may need a few pointers to get it done
Open a draft PR, then.
Change:
wmutils-opt = callPackage ../tools/X11/wmutils-opt { }; <del> inherit (callPackage ../servers/web-apps/wordpress {}) wordpress wordpress6_1; </del> <ins> inherit (callPackage ../servers/web-apps/wordpress {}) wordpress wordpress6_1 wordpress6_2; </ins> wordpressPackages = ( callPackage ../servers/web-apps/wordpress/packages { plugins = lib.importJSON ../servers/web-apps/wordpress/packages/plugins.json;

Discussion:
Package name: wordpress Latest released version: 6.2 <!-- Search your package here: --- Current version on the unstable channel: 6.1.1 Current version on the stable/release channel: 6.1.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.
Change:
<ins> { lib, fetchgit, SDL, stdenv, libogg, libvorbis, libmad, xdelta }: stdenv.mkDerivation rec { name = "uhexen2"; version = "1.5.9"; src = fetchgit { url = "https://git.code.sf.net/p/uhexen2/uhexen2"; sha256 = "0crdihbnb92awkikn15mzdpkj1x9s34xixf1r7fxxf762m60niks"; rev = "4ef664bc41e3998b0d2a55ff1166dadf34c936be"; }; buildInputs = [ SDL libogg libvorbis libmad xdelta ]; preBuild = '' makeFiles=( "engine/hexen2 glh2" "engine/hexen2 clean" "engine/hexen2 h2" "engine/hexen2/server" "engine/hexenworld/client glhw" "engine/hexenworld/client clean" "engine/hexenworld/client hw" "engine/hexenworld/server" "h2patch" ) ''; buildPhase = '' runHook preBuild for makefile in "''${makeFiles[@]}"; do local flagsArray=( -j$NIX_BUILD_CORES -l$NIX_BUILD_CORES SHELL=$SHELL $makeFlags "''${makeFlagsArray[@]}" $buildFlags "''${buildFlagsArray[@]}" ) echoCmd 'build flags' ""''${flagsArray[@]}"" make  -C $makefile ""''${flagsArray[@]}"" unset flagsArray done runHook postBuild ''; installPhase = '' runHook preInstall install -Dm755 engine/hexen2/{glhexen2,hexen2,server/h2ded} -t $out/bin install -Dm755 engine/hexenworld/{client/glhwcl,client/hwcl,server/hwsv} -t $out/bin install -Dm755 h2patch/h2patch -t $out/bin runHook postInstall ''; meta = with lib; { description = "A cross-platform port of Hexen II game"; longDescription = '' Hammer of Thyrion (uHexen2) is a cross-platform port of Raven Software's Hexen II source. It is based on an older linux port, Anvil of Thyrion. HoT includes countless bug fixes, improved music, sound and video modes, opengl improvements, support for many operating systems and architectures, and documentation among many others. ''; homepage = "http://uhexen2.sourceforge.net/"; license = licenses.gpl2Plus; maintainers = with maintainers; [ xdhampus ]; platforms = platforms.all; }; } </ins>

Discussion:
Project description Engine for Hexen 2 Metadata homepage URL: source URL: license: gpl2 platforms: unix, linux, darwin, windows, dos, aros, morphos, amigaos
Change:
protozero = callPackage ../development/libraries/protozero { }; <del> flatbuffers = flatbuffers_2_0; flatbuffers_2_0 = callPackage ../development/libraries/flatbuffers/2.0.nix { }; flatbuffers_1_12 = callPackage ../development/libraries/flatbuffers/1.12.nix { }; </del> <ins> flatbuffers = callPackage ../development/libraries/flatbuffers { }; </ins> nanopb = callPackage ../development/libraries/nanopb { };

Discussion:
We define two packages, one for version 1.12 and one for version 2.0. The package for version 1.12 . Upstream has the fixes needed but only for version 2.0. The version 1.12 branch does not appear maintained, and there to it since March of 2020. We can continue to maintain this (e.g. we can disable warnings, or we can adapt those fixes to 1.12 and patch them), but perhaps we could drop support for the old version altogether? ZHF: /cc
Disabling -Werror is always reasonable, but if there's no users left in nixpkgs I think it's ok to drop it. Unless the maintainer or someone else still has a use for this (do a scream test by removing it, I guess)
is unused and can be safely removed.
Change:
{ stdenv, fetchzip, expat, libiconv, libjpeg, libpng, libtiff, zlib # darwin only attributes , derez, rez, setfile <del> , AGL, Cocoa, Kernel </del> <ins> , AGL, Cocoa, Kernel, WebKit </ins> }: stdenv.mkDerivation rec { <del> version = "3.0.4"; </del> <ins> version = "3.0.5.1"; </ins> pname = "wxmac"; src = fetchzip {

Discussion:
Describe the bug wxPython40, , does not build there likely because it uses wxGTK rather than this prevents kicad from being built on darwin To Reproduce Steps to reproduce the behavior: darwin, Expected behavior wxPython40 to build on darwin and be usable by kicad there Additional context i maintain the kicad package and Notify maintainers while there are no listed maintainers for this package it was initially contributed by (if you already investigated after the kicad PR ping, nothing relevant changed since) and has a fair amount of contributions by Maintainer information:
Hi I don't have/use mac but happy to answer any wxPython questions as I can
I'm experiencing the same issue with Linux, using .
likely a different issue. Stacktrace? What nixpkgs version / commit?
The as of this writing. Here's the output I'm seeing. <details<summarybuild log</summary</detailsIt looks like pkg-config is giving an old version of GTK+?
please use your failure seems the same as which i closed because it's fixed in the branch by and i can confirm at least starts building there (it's still going...) i'm not sure when this fix will get to ...
nice! didn't know about the details tag. Ok yeah that looks like my issue, thanks for pointing me in that direction.
I can reproduce this on 10.14.6. My best take so far was to use and adapt other Darwin requirements. <details<summarydiff against master</summary</detailsUnfortunately it does not build with the same errors as for the .
Bumped to 3.0.5.1, to its , and it builds! was set, but no dependency specified. That's what caused the missing symbols. <details<summaryworking diff</summary</detailsI'm not even sure why this works at all. claims they build from a submodule, but obviously in this setup we take whatever is supplied through . To be pedantic, 4.0.7 is supposed use some specific instead of a release, as you can . That commit is after* 3.0.4 and shortly before 3.0.5. I only skimmed the , but it sounds and looks like backwards-compatible bug fixes. Opinions on how to proceed?

If we follow upstream and choose the intermediate revision, what to specify as the package version?
i've kinda given up on stuff, at least until the next release which i hope will either be built based on a release or be compatible with the latest release (i have no reason to believe this would be the case though) if you have a diff that fixes a package, i recommend you make a PR 4.0.7 seems to work with the 3.0.4, it seems reasonable to assume the same would be true for 3.0.4
Change:
stdenv.mkDerivation rec { pname = "clamav"; <del> version = "1.3.1"; </del> <ins> version = "1.4.0"; </ins> src = fetchurl { url = "https://www.clamav.net/downloads/production/${pname}-${version}.tar.gz"; <del> hash = "sha256-EqMDW/JvVfceMQalGl+o17dEVy35imOSCpz/h2p9zOQ="; </del> <ins> hash = "sha256-1nqymeXKBdrT2imaXqc9YCCTcqW+zX8TuaM8KQM4pOY="; </ins> }; patches = [

Discussion:
Steps to reproduce the behavior: clamav Happened during last system update. <!-- 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]:
I confirm it's not lix bug
I'm guessing it is related to this:
Upgrading to version 1.4.0 works:
Change:
hash = "sha256-rMbf09pCHvVYI9g/aq74+PcsuU2LezpmDz4b/w9vRyc="; }; <ins> postPatch = lib.optionalString stdenv.isDarwin '' substituteInPlace setup.py  --replace-fail "-flto" "" ''; </ins> nativeBuildInputs = [ pybind11 ]; # tests are removed from fork

Discussion:
The package is broken, , because it fails during on Darwin. I am uncertain if this is something that has to be fixed in , or can it be worked around in nixpkgs? Steps to reproduce the behavior: Build on macOS <!-- 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]:
can you help out here?
Change:
]; postInstall = '' <ins> # our mimeapps patched from upstream to exclude: # * pantheon-mail -> geary # * evince.desktop -> org.gnome.Evince.desktop </ins> mkdir -p $out/share/applications cp -av ${./pantheon-mimeapps.list} $out/share/applications/pantheon-mimeapps.list <ins> # instantiates pantheon's dockitems </ins> cp "${dockitemAutostart}" $out/etc/xdg/autostart/default-elementary-dockitems.desktop <ins> # script `Exec` to start pantheon </ins> mkdir -p $out/libexec substitute ${executable} $out/libexec/pantheon --subst-var out chmod +x $out/libexec/pantheon <del> ''; postFixup = '' substituteInPlace $out/share/xsessions/pantheon.desktop  --replace "gnome-session --session=pantheon" "$out/libexec/pantheon"  --replace "wingpanel" "${wingpanel}/bin/wingpanel" </del> <ins> # absolute path patched xsession substitute ${xsession} $out/share/xsessions/pantheon.desktop --subst-var out </ins> ''; passthru = {

Discussion:
Describe the bug Open indicator and see the following in journal The error at the top comes from now depending on settings schema (provided by ibus). This situation is annoyingly complex, because we cannot wrap . So what we try to do is create a wrapper with all the transitive dependencies of the plugins and call it day. We used to do patches to hardcode the gsettings, but this was far to much work to maintain and could be prone to developer error sometimes. I'm thinking the easiest solution would be to add all the transitive dependencies of to , and then wingpanel's can pick up all the proper dependencies. I think the only ideal request I could make to Pantheon upstream is to make use of SearchProviders from GNOME I believe this basically should be the very thing they want. They currently have a forked version of synapse's plugins system which IMHO appears to be inferior. To Reproduce Steps to reproduce the behavior: wingpanel-with-indicators the switchboard-plugin process get killed Expected behavior The switchboard-plugin process doesn't die. Metadata system: host os: multi-user?: sandbox: version: channels(root): nixpkgs: Maintainer information:
Change:
changelog = "https://code.videolan.org/videolan/dav1d/-/tags/${version}"; # More technical: https://code.videolan.org/videolan/dav1d/blob/${version}/NEWS license = licenses.bsd2; <del> platforms = platforms.unix; </del> <ins> platforms = platforms.unix ++ platforms.windows; </ins> maintainers = with maintainers; [ primeos ]; }; }

Discussion:
Steps to reproduce the behavior: qt6.qtmultimedia for mingw (windows) hostPlatform I am slowly adding mingw support for qt6.qtmultimedia. Will reference this issue when I open a PR for a cross build dependency. May open some drafts in a effort to get help from the community. Here is an image of the dependency tree from msys2. ! Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Upstreaming my changes to nixpkgs is taking longer than I expected. Here is the publication of my overlays. Hopefully other members of the community can help me upstream these packages to get qtbase and qtmultimedia cross compiling.
Builds qt6.qtmultimedia for mingw32. Thank you and anyone else I forgot to mention, for helping with reviews. My experience of contributing to nixpkgs was very positive, and it's thanks to all you incredibly bright and generous people. Closing as resolved.
Thank you for putting in all the hard work, very much appreciated!

Steps to reproduce the behavior: qt6 for mingw (windows) hostPlatform I have this building on my local machine using overlays. Here is the dependency tree. ! My plan is to open PR's for each dependency until I can add mingw support for qt6. If you are a maintainer of a package that I am making a PR for, please keep in mind a may be blocked upstreaming mingw support for qt6 if PRs linger for a prolonged period of time. <!-- 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. system: host os: multi-user?: sandbox: version: nixpkgs: Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Upstreaming my changes to nixpkgs is taking longer than I expected. Here is the publication of my overlays. Hopefully other members of the community can help me upstream these packages to get qtbase and qtmultimedia cross compiling.
Change:
python_magic ]; <ins> checkInputs = [ xvfb-run nose flake8 ] ++ requiredTools; preCheck = "export HOME=$PWD"; checkPhase = '' runHook preCheck xvfb-run -s '-screen 0 800x600x24' make test runHook postCheck ''; # unhardcodes xrandr and fixes nosetests # upstream in progress: https://github.com/lutris/lutris/pull/3754 patches = [ ./fixes.patch ]; </ins> # avoid double wrapping dontWrapGApps = true; makeWrapperArgs = [

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 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: lutris-unwrapped <!-- The current version can be found easily with the same process than 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: 0.5.8.4 desired version: 0.5.9 <!-- Search your package here: If no maintainer is listed for your package, tag the person that last updated the package --maintainers: Please tag this issue in your PR.