New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
diffoscope: 77 -> 85 #28785
diffoscope: 77 -> 85 #28785
Conversation
@@ -1,25 +1,24 @@ | |||
{ lib, stdenv, fetchgit, fetchpatch, python3, docutils | |||
{ lib, stdenv, fetchgit, fetchpatch, python3Packages, docutils | |||
, acl, binutils, bzip2, cbfstool, cdrkit, colord, cpio, diffutils, e2fsprogs, file, fpc, gettext, ghc | |||
, gnupg1, gzip, jdk, libcaca, mono, pdftk, poppler_utils, sng, sqlite, squashfsTools, unzip, vim, xz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you meant to replace vim
-> xxd
(otherwise the reference to xxd
below doesn't work).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, fixed!
I was running diffoscope for a long time and was sad there was no progress reporting... After some wrangling the python dependencies, ended up with this: https://github.com/dtzWill/nixpkgs/tree/feature/diffoscope-progressbar Which also bumps to 86 which among other things adds support for If this sounds good, feel free to grab the relevant commits or ping me and I'll do a PR in a day or two. |
If you could do a PR that would be great! Please ping me when you do the PR. |
I've added these commits to #29009. |
Thanks! |
Motivation for this change
diffoscope is failing on
release-17.09
due to a failing test forlibarchive-c
. The new version oflibarchive-c
fails the same test so I disabled it, but diffoscope seems to work.Also includes an update for libarchive-c but diffoscope is the only user of that library.
If @dezgeg is cool with this, I will push it to 17.09 as well.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)