jiebi/CodeConvo huggingface dataset:
Change:
{ lib, <ins> stdenv, </ins> attr, buildPythonPackage, <ins> fastapi, </ins> fetchFromGitHub, freezegun, <ins> httpx, </ins> orjson, poetry-core, pydantic,

Discussion:
In this was to the list of . As is linux-only this breaks the build for and all of it's dependencies on other platforms such as The package also seems to build without this dependency (at least on ) -- I hope I didn't overlook something here (also sadly can't check right now). Thank you! Steps to reproduce the behavior: or Add any other context about the problem here. <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. --Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
<del> { lib, stdenv, fetchFromGitHub, makeWrapper , shntool, cuetools , flac, faac, mp4v2, wavpack, mac , imagemagick, libiconv, enca, lame, mutagen, vorbis-tools , aacgain, mp3gain, vorbisgain }: let wrapSplit2flac =  format: '' makeWrapper $out/bin/.split2flac-wrapped $out/bin/split2${format}  --set SPLIT2FLAC_FORMAT ${format}  --prefix PATH : ${lib.makeBinPath [ shntool cuetools flac faac mp4v2 wavpack mac imagemagick libiconv enca lame mutagen vorbis-tools aacgain mp3gain vorbisgain ]} ''; in stdenv.mkDerivation rec { pname = "split2flac"; version = "122"; src = fetchFromGitHub { owner = "ftrvxmtrx"; repo = "split2flac"; rev = version; sha256 = "1a71amamip25hhqx7wwzfcl3d5snry9xsiha0kw73iq2m83r2k63"; }; dontBuild = true; nativeBuildInputs = [ makeWrapper ]; patchPhase = '' substituteInPlace split2flac  --replace 'FORMAT="''${0##*split2}"' 'FORMAT=''${SPLIT2FLAC_FORMAT:-flac}' ''; installPhase = '' mkdir -p $out/share/bash-completion/completions cp split2flac-bash-completion.sh  $out/share/bash-completion/completions/split2flac-bash-completion.sh mkdir -p $out/bin cp split2flac $out/bin/.split2flac-wrapped ${wrapSplit2flac "flac"} ${wrapSplit2flac "mp3"} ${wrapSplit2flac "ogg"} ${wrapSplit2flac "m4a"} ${wrapSplit2flac "wav"} ''; meta = with lib; { description = "Split flac/ape/wv/wav + cue sheet into separate tracks"; homepage = "https://github.com/ftrvxmtrx/split2flac"; license = licenses.mit; platforms = platforms.all; maintainers = with maintainers; [ jfrankenau ]; }; } </del>

Discussion:
fails to compile on because of a broken Python 2 dependency (). According to the github home page, is not anymore developed and replaced by , see ,
Thanks for reporting. I opened the above mentioned PR to address this issue. In the PR I made an alias for split2flac that throws a message ecouraging the use of or to help package the library. A review is welcomed ;)
Change:
--prefix XDG_DATA_DIRS : "${gnome.caribou}/share" ) <ins> buildPythonPath "$out ${python3.pkgs.xapp}" # https://github.com/NixOS/nixpkgs/issues/200397 patchPythonScript $out/bin/cinnamon-spice-updater </ins> # https://github.com/NixOS/nixpkgs/issues/129946 <del> buildPythonPath "${python3.pkgs.xapp}" </del> patchPythonScript $out/share/cinnamon/cinnamon-desktop-editor/cinnamon-desktop-editor.py <ins> # Called as `pkexec cinnamon-settings-users.py`. wrapGApp $out/share/cinnamon/cinnamon-settings-users/cinnamon-settings-users.py </ins> ''; passthru = {

Discussion:
To reproduce, here is an example: The related code is at Some nice to have: As suggested  should be something that can be accessible by everything and we are likely going to put our "" there and next we are going to import in basically every file. Adding to in is probably not a good idea since this is a desktop shell package you will end up having this variable everywhere, I imagine a treewide will be likely needed.
Change:
with subtest("Test default gateway"): router.wait_until_succeeds("ping -c 1 192.168.3.1") client.wait_until_succeeds("ping -c 1 192.168.3.1") <ins> router.wait_until_succeeds("ping -c 1 fd00:1234:5678:3::1") client.wait_until_succeeds("ping -c 1 fd00:1234:5678:3::1") </ins> ''; }; routeType = {

Discussion:
Setting the default IPV4 and IPV6 gateway using and respectively results in a system configuration in which only the IPV6 default gateway is configured. Looking at the generated network configuration files, this seems to be due to an order of precedence and non-merging behaviour within the section of the config. My system has only one network interface with both an ipv4 and ipv6 address configured. I expect both routes to be applied to this interface. Reverse-engineering the , I discovered that the gateways are configured differently to what would apply. As a result, I was able to test the alternative method and found that the following would correctly configure the network: With that, I also noticed that the entries were to the section rather than the section. This would imply to me that the handling of defaultGateway should do the same. both and with . the configuration. to view the active routes, and note the lack of an IPV4 default route. Both IPV4 and IPV6 default routes/gateways should be configured. The behaviour should match setting to the value of each gateway. Broken configuration example: Working configuration example:
looks like we should be generating one route per gateway instean of one route with multiple gateways. putting the gateways into would have the same effect, but emitting multiple routes seems clearer.
Change:
license = licenses.unfree; platforms = [ "x86_64-linux" "aarch64-linux" "loongarch64-linux" ]; sourceProvenance = with sourceTypes; [ binaryNativeCode ]; <del> maintainers = with maintainers; [ pokon548 ]; </del> <ins> maintainers = with maintainers; [ pokon548 xddxdd ]; </ins> mainProgram = "wechat-uos"; }; };

Discussion:
Segmentation fault prevents from launching. Steps to reproduce the behavior: wechat-uos from master branch command WeChat launches. works, so it is NOT that broke the package. <!-- Please @ people who are in the list of the offending package or module. If in doubt, check for whoever last touched something. -- system: host os: multi-user?: sandbox: version: channels(root): nixpkgs: Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
!$g614RgC4xlu3GtyIPmf1mCBe0VEZmQwt- Seems to be caused by missing
I can determine that is bad and is good. Since there is a Python package build failure between these two commits, bisect is almost impossible. result seems to not support missing coredump backtrace:
After building for days, I can now determine, it is that broken wechat-uos. The last commit that could run on master is ! Also CC
Change:
-            log.error("Error: cannot launch ldconfig -p to locate libfakeXinerama:") -            log.error(" %s", e) -    return find_lib("libfakeXinerama.so.1") <del> +    return "@libfakeXinerama@/lib/libfakeXinerama.so.1" </del> <ins> +    return "@libfakeXinerama@/lib/libfakeXinerama.so.1.0" </ins> current_xinerama_config = None

Discussion:
Several applications (including xterm) log errors relating to libfakeXinerama not being found. Note: this issue was phrased different, this error happened to be a red herring, but it's still an issue with corresponding fix. Steps to reproduce the behavior: ¹ ¹: my actual command line is this one, but the other options are merely for usability (i.e. closing the session properly instead of leaving dangling servers running): xterm runs normally. Many error messages like these are being logged: The file is used as follows: The file is created as follows: Note that is invalid while or would be valid. Please run and paste the result.
Regarding the extensive edit and the title change: I've confirmed that the PR works and silences the plethora of error messages, however it doesn't fix GLX, so I removed that from the issue and will continue my wild goose chase.
Change:
let commonTargetPkgs = pkgs: with pkgs; [ <del> steamPackages.steam-fonts </del> # Needed for operating system detection until # https://github.com/ValveSoftware/steam-for-linux/issues/5909 is resolved lsb-release

Discussion:
<!-- Note that these are hard requirements --<!-- You can use the "Go to file" functionality on GitHub to find the package Then you can go to the history for this package Find the latest "packagename: oldversion -newversion" commit The "newversion" is the current version of the package --- <!-- Type the name of your package and try to find an open pull request for the package If you find an open pull request, you can review it! There's a high chance that you'll have the new version right away while helping the community! --- name: steam-fonts <!-- The current version can be found easily with the same process as above for checking the master branch If an open PR is present for the package, take this version as the current one and link to the PR --current version: "1" desired version: <!-- 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. is no longer available under the given URL. current URL: File could be here:
Change:
<del> { stdenv, fetchurl, python, pkgconfig, readline, talloc , libxslt, docbook_xsl, docbook_xml_dtd_42 </del> <ins> { stdenv , fetchurl , python3 , pkg-config , readline , talloc , libxslt , docbook-xsl-nons , docbook_xml_dtd_42 , which , wafHook </ins> }: stdenv.mkDerivation rec { <del> name = "tevent-0.9.37"; </del> <ins> pname = "tevent"; version = "0.10.2"; </ins> src = fetchurl { <del> url = "mirror://samba/tevent/${name}.tar.gz"; sha256 = "1q77vbjic2bb79li2a54ffscnrnwwww55fbpry2kgh7acpnlb0qn"; </del> <ins> url = "mirror://samba/tevent/${pname}-${version}.tar.gz"; sha256 = "+EJ4IuWyh4+4so1vUNloSHNPPzEwYS+1dP3S0hSKZpY="; </ins> }; <del> nativeBuildInputs = [ pkgconfig ]; </del> <ins> nativeBuildInputs = [ pkg-config which python3 libxslt docbook-xsl-nons docbook_xml_dtd_42 wafHook ]; </ins> buildInputs = [ <del> python readline talloc libxslt docbook_xsl docbook_xml_dtd_42 </del> <ins> python3 readline # required to build python talloc </ins> ]; <del> preConfigure = '' sed -i 's,#!/usr/bin/env python,#!${python}/bin/python,g' buildtools/bin/waf ''; </del> <ins> wafPath = "buildtools/bin/waf"; </ins> <del> configureFlags = [ </del> <ins> wafConfigureFlags = [ </ins> "--bundled-libraries=NONE" "--builtin-libraries=replace" ]; meta = with stdenv.lib; { description = "An event system based on the talloc memory management library"; <del> homepage = https://tevent.samba.org/; </del> <ins> homepage = "https://tevent.samba.org/"; </ins> license = licenses.lgpl3Plus; platforms = platforms.all; };

Discussion:
Describe the bug build of failed. To Reproduce Metadata system: host os: multi-user?: sandbox: version: channels(root): channels(gtgteq): nixpkgs: Maintainer information:
Looks like regression in . We need to build talloc with Python 3 too.
It might also need
Change:
<ins> { lib, buildPythonPackage, fetchFromGitHub, pytestCheckHook, pythonOlder, setuptools, }: buildPythonPackage rec { pname = "pipe"; version = "2.2"; pyproject = true; disabled = pythonOlder "3.8"; src = fetchFromGitHub { owner = "JulienPalard"; repo = "Pipe"; rev = "refs/tags/v${version}"; hash = "sha256-/xMhh70g2KPOOivTjpAuyfu+Z44tBE5zAwpSIEKhK6M="; }; build-system = [ setuptools ]; nativeCheckInputs = [ pytestCheckHook ]; pythonImportsCheck = [ "pipe" ]; disabledTests = [ # Test require network access "test_netcat" ]; meta = with lib; { description = "Module to use infix notation"; homepage = "https://github.com/JulienPalard/Pipe"; changelog = "https://github.com/JulienPalard/Pipe/releases/tag/v${version}"; license = licenses.mit; maintainers = with maintainers; [ fab ]; }; } </ins>

Discussion:
Project description A Python library to use infix notation in Python Metadata homepage URL: source URL: license: MIT platforms: unix, linux, darwin Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
Change:
<del> { stdenv, fetchurl, dpkg, bash, python27Packages }: let py = python27Packages; in stdenv.mkDerivation rec { name = "cura-lulzbot-${version}"; version = "15.02.1-1.03-5064"; src = if stdenv.hostPlatform.system == "x86_64-linux" then fetchurl { url = "https://download.alephobjects.com/ao/aodeb/dists/jessie/main/binary-amd64/cura_${version}_amd64.deb"; sha256 = "1gsfidg3gim5pjbl82vkh0cw4ya253m4p7nirm8nr6yjrsirkzxg"; } else if stdenv.hostPlatform.system == "i686-linux" then fetchurl { url = "http://download.alephobjects.com/ao/aodeb/dists/jessie/main/binary-i386/cura_${version}_i386.deb"; sha256 = "0xd3df6bxq4rijgvsqvps454jkc1nzhxbdzzj6j2w317ppsbhyc1"; } else throw "${name} is not supported on ${stdenv.hostPlatform.system}"; python_deps = with py; [ pyopengl pyserial numpy wxPython30 power setuptools ]; pythonPath = python_deps; propagatedBuildInputs = python_deps; buildInputs = [ dpkg bash py.wrapPython ]; phases = [ "unpackPhase" "installPhase" ]; unpackPhase = "dpkg-deb -x ${src} ./"; installPhase = '' mkdir -p $out/bin cp -r usr/share $out/share find $out/share -type f -exec sed -i 's|/usr/share/cura|$out/share/cura|g' "{}" ; cat <<EOT > $out/bin/cura #!${bash}/bin/bash PYTHONPATH=$PYTHONPATH:$out/share/cura ${py.python}/bin/python $out/share/cura/cura.py "$@" EOT chmod 555 $out/bin/cura ''; meta = with stdenv.lib; { description = "3D printing host software for the Lulzbot"; longDescription = '' Cura LulzBot Edition is a fork of the 3D printing/slicing software from Ultimaker, with changes to support 3D printers from Aleph Objects. ''; homepage = https://www.lulzbot.com/cura/; license = licenses.agpl3; platforms = platforms.linux; maintainers = with maintainers; [ pjones ]; }; } </del>

Discussion:
Is there a way to install a more recent version of cura-lulzbot edition?
I guess the pool of people that could update and test this is very limited so feel free to tackle it yourself :) We can help. How old is the current nixpkgs version vs upstream ?
Can you test ?
Happy to, not 100% sure on the process for testing that pull request. do in build off a local version of nixpkgs with that pull applied. Can you point me in the right direction?
The one-line command to try this out would be: You could also manually clone the git repository of the PR and build it: But , so please wait a bit. EDIT: the cura binary is apparently called
you want to try the instructions gebner provided, I just updated the PR to roll the patch behind the now-broken link inline.
...FYI, the PR is now updated to 3.6.15. I'm using it happily myself, but reports (of success or failure) would be very welcome.
Change:
# Autogenerated by maintainers/scripts/gnome.sh update fetchurl: { <del> name = "gitg-3.18.0"; </del> <ins> name = "gitg-3.20.0"; </ins> src = fetchurl { <del> url = mirror://gnome/sources/gitg/3.18/gitg-3.18.0.tar.xz; sha256 = "fa4b7b9c492f13f5f1d864af1281ea377ac8c7619c856e05f533b18989edf421"; </del> <ins> url = mirror://gnome/sources/gitg/3.20/gitg-3.20.0.tar.xz; sha256 = "1f09f61208349d003f228e51dc9709bd3426960f5585c0e38197bd02b51f3346"; </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:
--replace "@CC@"     "${stdenv.cc}/bin/cc" ''; preBuild = '' <ins> export HOME="$TMP" </ins> local topDir=$out/lib/ghcjs-${version} mkdir -p $topDir

Discussion:
It appears to me that a number of cabal-install commands in cabal install 1.24 have a side effect of attempting to create which gets expanded to which fails. For example, installing fails with I am not entirely sure. Many Haskell libraries install just fine. I think the issue only occurs with things like ghcjs which attempt to invoke itself. This is the patch to cabal-install which I believe is responsible: In particular, is changed to call and is responsible for trying to create . A second data point is my helper: Previous I was able to use it in my to ensure that only files listed in the file were to the src But that also started failing with an error about not being able to create . I have no suggestions for a fix or a work around at this time.
I typically set to work around these kind of issues.
Does this issue still exist?
The issue still exists: If you write a Nix package that runs , you're going to have to be aware of it and apply the workaround. I opened a . If they're not interested in fixing it there, then I think we ought to reopen this as a documentation issue here.
Why would anyone call inside of a Nix build? Just run to achieve the same effect with way less moving parts involved.
Because one didn't know you could do that! Either way, documentation issue? :smiley:
This issue has been mentioned on NixOS Discourse. There might be relevant details there:
Change:
meta = with lib; { description = "A helper for screenshots within sway"; <del> homepage = "https://github.com/swaywm/sway/tree/master/contrib"; license = licenses.mit; platforms = platforms.all; maintainers = sway-unwrapped.meta.maintainers ++ (with maintainers; [ evils ]); </del> <ins> maintainers = with maintainers; [ evils ]; </ins> }; }; inactive-windows-transparency = python3Packages.buildPythonApplication rec { <ins> inherit version src; </ins> # long name is long lname = "inactive-windows-transparency"; pname = "sway-${lname}"; <del> version = sway-unwrapped.version; src = sway-unwrapped.src; </del> format = "other"; dontBuild = true;

Discussion:
User-contributed scripts are being moved over to this repository: I just adopted the use of and couldn't build grimshot: I could build my nixosconfiguration before when I was using . I guess nixpkgs-wayland is slightly ahead, and we'll have this problem when nixos-unstable updates sway.
i don't think github notifies issue subscribers of linked PRs so: i've opened to fix this
Yes, that's a nixpkgs-wayland problem/regression/bug. Nixpkgs is still fine. We just have to update the package before the next Sway update but it might indeed be best to do this in advance.
Change:
*  `metricProvider` (optional) *    this attribute contains additional machine config * <ins> *  `nodeName` (optional) *    override an incompatible testnode name * </ins> *  Example: *    exporterTests.<exporterName> = { *      exporterConfig = {

Discussion:
Project description Collect ALL UniFi Controller, Site, Device & Client Data Nixpkg currently has which is not developed anymore. Corresponding NixOS module may have to be updated. Metadata homepage URL: source URL: license: MIT platforms: unix, linux, darwin
Change:
<del> { stdenv, fetchFromGitHub, cmake, qt4, pythonPackages }: stdenv.mkDerivation rec { name = "open-pdf-presenter-git-2014-09-23"; src = fetchFromGitHub { owner  = "olabini"; repo   = "open-pdf-presenter"; rev    = "f14930871b60b6ba50298c27377605e0a5fdf124"; sha256 = "1lfqb60zmjmsvzpbz29m8yxlzs2fscingyk8jvisng1y921726rr"; }; buildInputs = [ cmake qt4 pythonPackages.poppler-qt4 ]; meta = { homepage = https://github.com/olabini/open-pdf-presenter; description = "A program for presenting PDFs on multi-monitor settings (typically a laptop connected to a overhead projector)"; license = stdenv.lib.licenses.gpl3; maintainers = [ ]; platforms = stdenv.lib.platforms.all; }; } </del>

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

Discussion:
On nixos-unstable, the package does not install: is installed No maintainer but pinging who last touched Please run and paste the result.
was updated to 4.0.1 a while back. is pinning most of it's dependencies thus the issue. Looks like that could need an update (available 2.2.18, upstream 2.2.27).
So I can build the upgraded package with the following: But it doesn't look right. I guess we shouldn't blindly replace pinned versions with incompatible versions. How does nixpkgs usually deals with this?
Change:
gst_all_1.gstreamer gst_all_1.gst-plugins-base gnome3.libgee which udev gnome3.gexiv2 libraw rest json_glib gettext desktop_file_utils glib lcms2 gdk_pixbuf librsvg makeWrapper gnome_doc_utils <del> gnome3.gnome_icon_theme gnome3.gnome_icon_theme_symbolic ]; </del> <ins> gnome3.gnome_icon_theme gnome3.gnome_icon_theme_symbolic hicolor_icon_theme ]; </ins> meta = with stdenv.lib; { description = "Popular photo organizer for the GNOME desktop";

Discussion:
Shotwell does not store its configuration when running under nix. Specifically, on startup, it says: Is there a standard way of fixing such issue in nix? In case it's relevant, I'm not running nixos, but the nixpkgs unstable channel on a Fedora box.
Try wrapping the binary with . Also on your non-nixos system you should have dconf installed at least.
Many thanks for the tip - I can confirm that this fixes the problem. I will submit a pull request later today.
Reopening, close when the fix is committed.
Sorry for closing prematurely; closing now. Thanks again for being so helpful!
Reopening, let's make sure this one is documented.
document what? This issue is closed, we can open another one for documentation. There's already an issue for documenting GNOME stuff iirc. Mostly because the commit refers to this issue.
"Documentation for Gnome3 packaging".
Change:
<ins> diff --git a/platform/gsutil/gslib/command_runner.py b/platform/gsutil/gslib/command_runner.py index 06ca5e5..4a4e225 100644 --- a/platform/gsutil/gslib/command_runner.py +++ b/platform/gsutil/gslib/command_runner.py @@ -61,7 +61,6 @@ from gslib.utils.text_util import InsistAsciiHeaderValue from gslib.utils.text_util import print_to_fd from gslib.utils.unit_util import SECONDS_PER_DAY from gslib.utils.update_util import LookUpGsutilVersion -from gslib.tests.util import HAS_NON_DEFAULT_GS_HOST def HandleHeaderCoding(headers): @@ -331,17 +330,6 @@ class CommandRunner(object): Returns: Return value(s) from Command that was run. """ -    command_changed_to_update = False -    if (not skip_update_check and -        self.MaybeCheckForAndOfferSoftwareUpdate(command_name, debug)): -      command_name = 'update' -      command_changed_to_update = True -      args = [_StringToSysArgType('-n')] - -      # Check for opt-in analytics. -      if system_util.IsRunningInteractively() and collect_analytics: -        metrics.CheckAndMaybePromptForAnalyticsEnabling() - if not args: args = [] @@ -414,18 +402,10 @@ class CommandRunner(object): ShutDownGsutil() if GetFailureCount() > 0: return_code = 1 -    if command_changed_to_update: -      # If the command changed to update, the user's original command was -      # not executed. -      return_code = 1 -      print('n'.join( -          textwrap.wrap( -              'Update was successful. Exiting with code 1 as the original command ' -              'issued prior to the update was not executed and should be re-run.' -          ))) return return_code def MaybeCheckForAndOfferSoftwareUpdate(self, command_name, debug): +    return False """Checks the last time we checked for an update and offers one if needed. Offer is made if the time since the last update check is longer </ins>

Discussion:
Describe the bug To Reproduce Steps to reproduce the behavior: Expected behavior Works. Metadata Maintainer information:
cc
I am getting same error
/cc . Maybe the closure size got reduced a bit too much
Hmm, this only seems to come up in . I think we plausibly just want to remove that function.