jiebi/CodeConvo huggingface dataset:
Change:
kestrel = callPackage ../tools/security/kestrel { }; <del> kexec-tools = callPackage ../os-specific/linux/kexec-tools { # clangStdenv fails with # purgatory/arch/i386/entry32-16.S:23:2: error: unknown directive #  .arch i386 #  ^ # purgatory/arch/i386/entry32-16.S:115:11: error: unknown token in expression #  ljmp %cs:*(realdest - entry16) #           ^ # make: *** [Makefile:128: purgatory/arch/i386/entry32-16.o] Error 1 stdenv = gccStdenv; }; </del> <ins> kexec-tools = callPackage ../os-specific/linux/kexec-tools { }; </ins> keepkey-agent = with python3Packages; toPythonApplication keepkey-agent;

Discussion:
Steps to reproduce the behavior: 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. --No maintainers, pinging from . Please run and paste the result. Add a :+1: [reaction] to [issues you find important]. [reaction]: [issues you find important]:
has:
need to edit to unpin gccStdenv will get past configure but fails at link step for a different reason
Looks like clang doesn't understand the assembly syntax kexec-tools uses which appears to be the GNU/GCC style. I see pixman has had a similar problem before which the solution for them is to disable assembly. We don't have that luxury here. We have two solutions but both requires patches: the assembly files to use a syntax clang accepts a different assembler With the first one, it sounds like ARM has something called UAL (Unified Assembly Language) so we might be able to use that. With the second one, it looks like kexec-tools just uses the CC for the assembler so we would have to patch the Makefiles. Personally, I think the 2nd option is the best here but it looks like this is a part of the Linux kernel git server I think inquiring upstream to look into this is not a bad idea.
on x64 linux with removed i get a link error: though exists not sure why it is trying to use if it is trying to statically link.
Yeah, I see that error as well but on aarch64-linux. Not sure why it failed to link things right.
I was able to use and run the failed command with :
I have discovered that adding makes the command work.
i guess the clang linker is not happy. i can build fine using which uses gnu binutils ld i believe.
Doesn't work for me. I still get the same error during the build phase.
?
Oh no, I'm doing: . Using the non-LLVM package set works but this issue is with .
I made a patch for one of the Makefiles and applying the , works with . Will PR it now.
seems like an lld problem. it is searching for '.so' files when building a relocatable image but should be searching for files. this patch fixes it.

error is coming from: filed also, i'm not sure that linking with unconditionally is correct but i'm a bit foggy on when unwind is required
Change:
]; patches = [ <del> (substituteAll { src = ./fix-paths.patch; inherit flashrom efibootmgr bubblewrap; tpm2_tools = "${tpm2-tools}"; }) </del> <ins> ./fix-paths.patch </ins> ./add-option-for-installation-sysconfdir.patch # installed tests are installed to different output

Discussion:
Describe the bug As I am trying to upgrade my XPS 13, fwupd can't find To Reproduce Expected behavior Update completes Additional context I am using nixos-unstable Metadata system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
Looking back at this patch that's probably not right. Probably all calls to need to be removed and it should just be a string to the executable.
If we're already wrapping the executables with , why not set the instead of patching?
Change:
) => fetch(`https://github.com/${owner}/${repo}/raw/${version}/core/Cargo.toml`) .then((res) => res.text()) <del> .then((txt) => mod.parse(txt).dependencies.v8.version); </del> <ins> .then((txt) => toml.parse(txt).dependencies.v8.version); </ins> const fetchArchShaTasks = (version: string, arches: Architecture[]) => arches.map(

Discussion:
Deno no longer working on darwin due to tinycc being broken. error: Package ‘tcc-0.9.27+date=2022-01-11’ in /nix/store/4la4fc5wfn4p0784i12h0a046yqzln16- is marked as broken, refusing to evaluate. It works. Looks like this happened with the move to Deno 1.23.4: Issue related to tinycc being broken: deno: tinycc:
Thanks for raising this. I shouldn't have missed this. I'll try fix tinycc and if we can't Mac might be able to fall back to the same non-tinycc method (even if it has some performance penalties) until they migrate to asm
Thanks for being on this
if you could try out that'd be helpful
i'm on it!
succeeded for me on x86_64-darwin
Nice, it worked for me in my x8664-darwin vm but it's always good to get it checked on real hardware. Would be nice to have someone try it on aarch64-darwin As a note if you want to do a "full" test of the FFI features (the one that requires tcc now) git clone /tmp/deno cd /tmp/deno mkdir -p ./target/result cp /location/of/the/nixpkgs/result/bin/deno ./target/result/ target/release/deno bench --allow-ffi --allow-read --unstable
Ran it on my aarch64-darwin machine:
fantastic thanks
also worked for me on aarch64 (i used an EC2 m1 macos instance). Unfortunately running the FFI tests requires a to be in the directory, which I noticed is output when running outside of nix. However, on aarch64, I can't get a direct to succeed...
sorry I missed a step there try if you still have the instance
cool, that worked.

the benchmarks ran without error...
Reopening as does not resolve this issue
That PR does resolve the issue, see the comments towards the bottom to pull in the fix early The unstable channel hasn't progressed for and not in scope for this issue ~I will open a backport ticket for 22.05 though~ actually is on which is unaffected by this issue
Yes, this is confirmed by ofborg.
OMG Sorry that I didn't follow the PR correctly. Not only I missed the part you fixed on the derivation of deno, but I saw the tinycc's derivation flagged as and mistook that it is the same on deno's. The old message of the beginning of the PR ("tinycc darwin issues were a pain, there's no point holding up this update since it's already broken") also made me confused and gave an idea that this issue was closed by mistake. Sorry for confusing everyone here, and thank you very much for fixing the problem!
I'd pretty much given up on trying to make tinycc build properly on darwin but as a last-ditch effort I tried just building the static lib on it's own and to my surprise it worked just fine
Change:
<ins> { lib , stdenv , fetchhg , qt5 }: let inherit (qt5) qtbase qtmultimedia wrapQtAppsHook; in stdenv.mkDerivation (self: { pname = "rpcemu"; version = "0.9.4"; src = fetchhg { url = "http://www.home.marutan.net/hg/rpcemu"; rev = "release_${self.version}"; sha256 = "sha256-UyjfTfUpSvJNFPkQWPKppxp/kO0hVGo5cE9RuCU8GJI="; }; nativeBuildInputs = [ wrapQtAppsHook ]; buildInputs = [ qtbase qtmultimedia ]; configurePhase = '' runHook preConfigure cd src/qt5 qmake runHook postConfigure ''; installPhase = '' runHook preInstall cd ../.. install -Dm755 rpcemu-interpreter -t $out/bin runHook postInstall ''; meta = { homepage = "https://www.marutan.net/rpcemu/index.php"; description = "Risc PC Emulator"; longDescription = '' RPCEmu is an emulator of classic Acorn computer systems, such as the Risc PC and A7000. It runs on multiple platforms including Windows, Linux and Mac OS X. RPCEmu should be considered Alpha Quality code. It has many known and unknown bugs, and all files used with it should be well backed up before using them with RPCEmu. ''; license = lib.licenses.gpl2Plus; maintainers =  builtins.attrValues { inherit (lib.maintainers) AndersonTorres; }; platforms = lib.platforms.linux; }; }) </ins>

Discussion:
Project description RiscPC Emulator Metadata homepage URL: source URL: license: gpl2 platforms: linux, darwin This also includes ROM images which could also be downloaded, as ROOL maintains these on an open source licence. Not sure the best idea for these... /var/lib? Thanks very much for all your hard work.
Builds and compiles. I will upload it tomorrow
The roms are mostly optional, in the sense the software will seek them in . There is no such a need for include them now.
Thanks very much.
how did you get it to work? I can't get it to recognize the existence of the hostfs folder.
Change:
} ]; systemd.services.netdata = { <del> path = with pkgs; [ gawk curl ]; </del> description = "Real time performance monitoring"; after = [ "network.target" ]; wantedBy = [ "multi-user.target" ]; <ins> path = (with pkgs; [ gawk curl ]) ++ lib.optional cfg.python.enable (pkgs.python3.withPackages cfg.python.extraPackages); </ins> preStart = concatStringsSep "n" (map (dir: '' mkdir -vp ${dir} chmod 750 ${dir}

Discussion:
In NixOS, netdata fails to start the python script that runs all of its python plugins. This completely breaks all netdata provides. See also: Enable the netdata service and rebuild the NixOS configuration. Check netdata's log for the failure Notice a line similar to: A visual inspection of the netdata Web UI confirms there are no functioning python plugins. Note: I have already tried adding to the netdata . This has not seemed to help. Please run and paste the results. system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
You might want to add it to . Or even better, patch the to use python:
Thanks for the advice. I'll give these a shot later today.
fixed in
I also backported it.
I just noticed I also forgot to add
This one makes it finally work:
Change:
src = fetchPypi { pname = "langchain_core"; inherit version; <del> hash = "sha256-SZEz+two7899JDBiNlIVGAgLsQ/Yv290Jt5KK78q67U="; </del> <ins> hash = "sha256-4xOXXZrikmNC5vKtdgM40x8YsSI+m4tNxAja7q3kaoM="; </ins> }; pythonRelaxDeps = [

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:
themes = [];  # extra themes, etc. }; <ins> sddmPlasma5 = sddm.override { themes = [ plasma5.plasma-workspace pkgs.breeze-icons ]; }; </ins> skrooge = libsForQt5.callPackage ../applications/office/skrooge {}; slim = callPackage ../applications/display-managers/slim {

Discussion:
On NixOS 17.03, the in the system configuration with Plasma 5 enabled, has and in its runtime closure, which causes unnecessary bloat. This is because has: then causes the directories of and to be to the wrapper script. (Automatically adding everything with a directory to the wrapper seems pretty drastic BTW.) System: NixOS Nix version: 1.12pre5073_1cf4801
KDE upstream is slightly careless about implicit runtime dependencies. I spent hours trying to figure out which runtime dependencies needed to be in . By the way, you have to guess at this because SDDM doesn't log error messages. Could we drop ? Probably. Can I answer that question for any of the other dependencies? Not a chance. So I nuked it from orbit. Guessing the correct setting would be a good project for a first-time contributor with a lot of free time. At the moment, I'm trying to get rid of entirely. This problem may go away as a happy accident, but that won't help this release. I'm open to other suggestions.
Change:
scanners, see http://www.sane-project.org/sane-backends.html. ''; homepage = "http://www.sane-project.org/"; <del> license = licenses.gpl2Plus; platforms = platforms.linux ++ platforms.darwin; maintainers = [ maintainers.symphorien ]; </del> <ins> license = lib.licenses.gpl2Plus; platforms = lib.platforms.linux ++ lib.platforms.darwin; maintainers = [ lib.maintainers.symphorien ]; </ins> }; }

Discussion:
My Canon CanoScan LiDE 20 does not work on NixOS. I wrote a minimal SANE config dir to reproduce: Invocation: ` Reveals: <details</detailsImportant part: It's failing because it's trying to write a lock file into .
I suspect the code involved is this: Looks like the packaging overrode the string, putting the nix output store path prefix in front; that seems wrong.
Grepping further: Indeed, the build logs show the wrong path:
Probably the nix build sets to a store path. Fixing it via: Other nixpkgs packages also set . Maybe nixpkgs should do this automatically for all packages? Probably is what puts the nix store path in. However, trying the workaround fails the build at step:
In PR I made an option to set the package without mass rebuilds. With that, the following workarounds make the scanner work: Now I can scan.
I'd be happy if somebody could pick up the task to pass a proper lock dir and ensure it exists in NixOS.
Another option is to disable locking entirely. Putting this in my nixos config is working for me:
Yes, that's what's mentioned in the comment just above:
It seems only scanners covered by SANE's driver are affected by this, as only those use locks as per : Note this driver, even though called , handles also many other vendors' scanners, including some Canon, Epson, HP, and others.
I think that's the reason it isn't being done by default, because upstream build systems try to create directories inside the runtime state directory at install time. So we're trading build time successes for runtime failures. A similar situation exists with /etc (sysconfdir), where I think the default should be /etc, but nixpkgs uses $prefix/etc because it has a higher success rate at build time.
With the workaround mentioned in I'm able to scan on a CanonScan Lide 25 using the sane-plustek backend. Would be great to have this properly fixed.
Using a Canon N670U (also covered by ) I ran into the same issue and resolved it with workaround Proper fix would be highly appreciated though.
Please give a try, it should fix the issue
Change:
{ lib, bundlerApp, makeWrapper, # Optional dependencies, can be null <del> epubcheck, kindlegen, </del> <ins> epubcheck, </ins> bundlerUpdateScript }:

Discussion:
. Consequently, the is broken. I suppose we need to choose whether to remove the package or find a mirror? CC maintainer: Side notes: Sadly, its replacement, , has no Linux support. But on a more positive note, there are other programs that can produce mobi files, some mentioned in .
I suggest we just drop it. Considering that kindlegen was under an unfree license, we're probably not allowed to distribute it, so let's just blow it away with a release note entry that mentions some alternatives.
I marked this as stale due to inactivity. &rarr;
My current recommendation for NixOS is to use calibre for conversion. It also has commandline tools for scripting.
Change:
rustPlatform.buildRustPackage rec { pname = "lapce"; <del> version = "0.0.12"; </del> <ins> version = "0.1.0"; </ins> src = fetchFromGitHub { owner = "lapce"; repo = pname; rev = "v${version}"; <del> sha256 = "sha256-ZFQjQ5+G0b0Fgg3+du/drt+62rC/TCNR5MIdJXAkTrE="; </del> <ins> sha256 = "sha256-KSumy7M7VNUib4CZ0ikBboEFMzDQt4xW+aUFHOi+0pA="; </ins> }; <del> cargoSha256 = "sha256-sMTootPsenaWzLLFImo6HWC1pcm2uFupPhVWsUJp1Ak="; </del> <ins> cargoSha256 = "sha256-7SVTcH9/Ilq8HcpJJI0KFiQA076lR2CAIBwmTVgmnjE="; </ins> nativeBuildInputs = [ cmake

Discussion:
lapce: current version:0.0.12 desired version:0.1.0 maintainers: Please tag this issue in your PR.
Change:
'' + lib.concatStrings (builtins.map installFirmware compatFirmware); <ins> # parallel install creates a bad symlink at $out/lib/sane/libsane.so.1 which prevents finding plugins # https://github.com/NixOS/nixpkgs/issues/224569 enableParallelInstalling = false; </ins> meta = with lib; { description = "SANE (Scanner Access Now Easy) backends"; longDescription = ''

Discussion:
Simple Scan, Skanlite, and fail to find any scanners immediately on nixos-unstable. Steps to reproduce the behavior: a WiFi enabled printer/scanner. avahi w/ nssmdns and sane and observe SANE should be able to find printers and webcams (v4l). N/A Reverting to sane-backends 1.1.1 allows to search for scanners again. 1.2.1 might be bugged or require more configuration.
I can reproduce: does not list anything. What is weird is that it works on top of but not on your commit anymore I tried to bisect it, but I end up in a staging iteration: So the culprit is likely to be another dep, maybe curl because I suspect you use the escl backend?
I feel like there's more to it than that. 1.2.1 fails to find even my webcam and the failure is immediate with no signs of actually having doing anything to search.
I have this issue as well. doesn't find anything, but find it like this:
sane is actually not loading any plugins anymore: running LD_DEBUG=files scanimage -L does not show messages like
Bisected down to and weirdly disabling parallel installs (ie undoing ) does fix the issue, I don't know why. Please test to confirm it is the right solution.
I overlayed your PR into my configuration and now works!
Change:
patchPhase = if isPy3k then '' rm tests/test_crypto.py tests/test_paging.py '' else null; <ins> # Disable test broken with aniso > 3.0.0, can be removed when updating to a flask-restful version with commit 54979f0 preCheck = '' substituteInPlace "./tests/test_inputs.py" --replace "test_bad_isointervals" "disabled_bad_isointervals" ''; </ins> buildInputs = [ nose mock blinker ]; propagatedBuildInputs = [ flask six pytz aniso8601 pycrypto ]; PYTHON_EGG_CACHE = "`pwd`/.egg-cache";

Discussion:
Commit updated aniso from 3.0.0 to 3.0.2. flask-restful 0.3.6 has a broken test when using . Build flask-restful after commit test fails. Please run and paste the results. system: host os: multi-user?: sandbox: version: channels(root): nixpkgs:
cc
Disabling the failing check with: works, but this depends on injecting exactly on the line above the test declaration, and will fail in the future if line number changes. Alternatively, we can remove the entire file or update flask-restful to use a version with commit , which fixes this bug (no version with this commit is avaliable on pypy).
Use substituteInPlace to rename the test, removing test from the name. That way it won't be recognized as a test.
Lol, perfect, i had no idea. I'm testing with nox-review, will PR soon.
Change:
<del> { lib, stdenv, fetchFromGitHub }: </del> <ins> { lib, stdenvNoCC, fetchFromGitHub }: </ins> /* This derivation is impure: it relies on an Xcode toolchain being installed

Discussion:
The iterm2 package () does not build on aarch64-darwin. I'm getting the following error message: Steps to reproduce the behavior: installing/building . the build failure. The package to build. n/a Complete logs are available 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. Maintainer information:
Hi Unfortunately, it was/is not possible to build item in a pure and reproducible way from source because it depends on your installed Xcode toolchain. It is therefore not an ideal candidate for nixpkgs and I didn't update the version of it either. Do you see any benefit in having iterm2 in nixpkgs compared to just downloading it from their website?
Hi thanks for packaging this! I generally have a slight preference to installing things via nix so that I can put them in my home-manager config, but at the end of the day it doesn't make a huge difference to me. It sounds like iterm2 is kind of a pain to package in general... Do you think it should be marked as broken or removed from nixpkgs?
Change:
systemd.timers.snapper-cleanup = { description = "Cleanup of Snapper Snapshots"; inherit documentation; <del> wantedBy = [ "basic.target" ]; </del> <ins> wantedBy = [ "timers.target" ]; requires = [ "local-fs.target" ]; </ins> timerConfig.OnBootSec = "10m"; timerConfig.OnUnitActiveSec = cfg.cleanupInterval; }; <ins> systemd.services.snapper-boot = lib.optionalAttrs cfg.snapshotRootOnBoot { description = "Take snapper snapshot of root on boot"; inherit documentation; serviceConfig.ExecStart = "${pkgs.snapper}/bin/snapper --config root create --cleanup-algorithm number --description boot"; serviceConfig.type = "oneshot"; requires = [ "local-fs.target" ]; wantedBy = [ "multi-user.target" ]; unitConfig.ConditionPathExists = "/etc/snapper/configs/root"; }; </ins> }); }

Discussion:
Snapper stops to take snapshots after a system update and reboot. snapper in autoUpgrade and autoUpgrade.allowReboot in for system upgrade and reboot. .service succeds, but snapper- does not start (see below). will start to take snapshots after or snapper- should start without intervention after a reboot. journalctl -u snapper- systemctl status snapper- snapper- is not listed in journalctl -u snapperd.service snapper list <!-- 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. Maintainer information:
correctly starts snapper- after a (manual) reboot. The new snapshotOnBoot option is working as well. Reboot after a system upgrade has not been testet yet.
snapper- fails to restart after a system upgrade, but snapshotOnBoot works as expected. systemctl status snapper- journalctl -u snapper- snapper list
Change:
input-fonts = callPackage ../data/fonts/input-fonts { }; <del> inziu-iosevka = callPackage ../data/fonts/inziu-iosevka { }; </del> iosevka = callPackage ../data/fonts/iosevka { nodejs = nodejs-8_x; };

Discussion:
Hi Could you please note that Inziu is already deprecated and it is replaced by Sarasa Gothic/Sarasa Mono? Yours Sincerely, Belleve
This has been discussed here: Due to stability reasons, I need Inziu Iosevka for myself at least for now, and it may also be helpful to others. Just out of curiosity, why do you want a font to be deprecated? I suppose different fonts have different styles, and could be useful in different cases.
Because I lost its building system (after an PC replace). If you want to keep that please mirror its files to GitHub.
I am sorry to hear that. I'll do the mirroring work tomorrow.
I changed my mind to remove this font due to license issues. ()
closing after font was removed
Change:
} (fix: rec { name = "vim_configurable-${version}"; <del> version = "7.4.826"; </del> <ins> version = "8.0.0005"; </ins> enableParallelBuilding = true; # test this

Discussion:
Currently vimconfigurable is version 7.4.826 where as vim is now 8.0005. $ nix-env -f /path/to/nixpkgs -i vimconfigurable $ vim --version VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Jan 1 1970 00:00:01) Included patches: 1-826 ... Updated version and sha256 in to be identical to that in System: (NixOS: 16.09beta264. (Flounder)) Nix version: (nix-env (Nix) 1.11.4) Nixpkgs version: (17.03pre-git)
Change:
# provide desktop item and icon cp -r usr/share $out substituteInPlace $out/share/applications/franz.desktop  <del> --replace Exec="/opt/Franz/franz" Exec=franz </del> <ins> --replace /opt/Franz/franz franz </ins> ''; dontWrapGApps = true;

Discussion:
Describe the bug Franz is pointing to an incorrect executable so launching it doesn't work. To Reproduce Steps to reproduce the behavior: franz franz via the file an error saying Expected behavior Franz launches as expected Additional context According to the nix package is designed to replace the which just however in the latest version of Franz it doesn't seem to be working Metadata Maintainer information:
cc