* [bug#70293] [PATCH] gnu: diffoscope: Update to 263. @ 2024-04-08 22:27 Wilko Meyer 2024-04-09 16:36 ` Vagrant Cascadian 2024-04-09 17:49 ` bug#70293: " Christopher Baines 0 siblings, 2 replies; 7+ messages in thread From: Wilko Meyer @ 2024-04-08 22:27 UTC (permalink / raw) To: 70293; +Cc: Wilko Meyer, Vagrant Cascadian * gnu/packages/diffoscope.scm (diffoscope): Update to 263. Change-Id: I8c9fce87dcf78fbc0f5ed319fa9e8b6d2e37fee1 --- gnu/packages/diffoscope.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/diffoscope.scm b/gnu/packages/diffoscope.scm index 5bbb4f51e2..bceb465d5b 100644 --- a/gnu/packages/diffoscope.scm +++ b/gnu/packages/diffoscope.scm @@ -74,7 +74,7 @@ (define-module (gnu packages diffoscope) (define-public diffoscope (package (name "diffoscope") - (version "261") + (version "263") (source (origin (method git-fetch) @@ -83,7 +83,7 @@ (define-public diffoscope (commit version))) (file-name (git-file-name name version)) (sha256 - (base32 "0v56i2wyc4p843crl5b5w6j1awhp1a6xbfb92hm0pp8558f1h76i")))) + (base32 "1bq45gyn214hf9brnn5xlj9xvcg6p0yr8cc2p153f93pgzsyqlg4")))) (build-system python-build-system) (arguments (list base-commit: 3571e4752501fb0f7da7785a3834009f522eb24c -- 2.41.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [bug#70293] [PATCH] gnu: diffoscope: Update to 263. 2024-04-08 22:27 [bug#70293] [PATCH] gnu: diffoscope: Update to 263 Wilko Meyer @ 2024-04-09 16:36 ` Vagrant Cascadian 2024-04-09 17:40 ` Vagrant Cascadian 2024-04-09 17:49 ` bug#70293: " Christopher Baines 1 sibling, 1 reply; 7+ messages in thread From: Vagrant Cascadian @ 2024-04-09 16:36 UTC (permalink / raw) To: Wilko Meyer, 70293; +Cc: Christopher Baines [-- Attachment #1: Type: text/plain, Size: 1831 bytes --] On 2024-04-09, Wilko Meyer wrote: > * gnu/packages/diffoscope.scm (diffoscope): Update to 263. Thanks for the update! Not a *huge* deal, but this version should have perl added to native inputs, as some tests now use "zipdetails" from perl, and are skipped if it is not present. Or does something else pull in perl? In general, checking the diff between the old diffoscope version and the new version would reveal things like this, e.g. in tests/ you might look for things like: -@skip_unless_tools_exist("zipinfo") +@skip_unless_tools_exist("zipinfo", "zipdetails") Also, strange as it may sound, reading debian/changelog might also hint at changes like this (which would be included in the diff between versions)! live well, vagrant > Change-Id: I8c9fce87dcf78fbc0f5ed319fa9e8b6d2e37fee1 > --- > gnu/packages/diffoscope.scm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/gnu/packages/diffoscope.scm b/gnu/packages/diffoscope.scm > index 5bbb4f51e2..bceb465d5b 100644 > --- a/gnu/packages/diffoscope.scm > +++ b/gnu/packages/diffoscope.scm > @@ -74,7 +74,7 @@ (define-module (gnu packages diffoscope) > (define-public diffoscope > (package > (name "diffoscope") > - (version "261") > + (version "263") > (source > (origin > (method git-fetch) > @@ -83,7 +83,7 @@ (define-public diffoscope > (commit version))) > (file-name (git-file-name name version)) > (sha256 > - (base32 "0v56i2wyc4p843crl5b5w6j1awhp1a6xbfb92hm0pp8558f1h76i")))) > + (base32 "1bq45gyn214hf9brnn5xlj9xvcg6p0yr8cc2p153f93pgzsyqlg4")))) > (build-system python-build-system) > (arguments > (list > > base-commit: 3571e4752501fb0f7da7785a3834009f522eb24c > -- > 2.41.0 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 227 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* [bug#70293] [PATCH] gnu: diffoscope: Update to 263. 2024-04-09 16:36 ` Vagrant Cascadian @ 2024-04-09 17:40 ` Vagrant Cascadian 2024-04-09 18:03 ` Christopher Baines 0 siblings, 1 reply; 7+ messages in thread From: Vagrant Cascadian @ 2024-04-09 17:40 UTC (permalink / raw) To: Wilko Meyer, 70293; +Cc: Christopher Baines [-- Attachment #1: Type: text/plain, Size: 570 bytes --] On 2024-04-09, Vagrant Cascadian wrote: > On 2024-04-09, Wilko Meyer wrote: >> * gnu/packages/diffoscope.scm (diffoscope): Update to 263. > > Thanks for the update! > > Not a *huge* deal, but this version should have perl added to native > inputs, as some tests now use "zipdetails" from perl, and are skipped if > it is not present. Or does something else pull in perl? For a build without perl, the test coverage is: 638 passed, 72 skipped With perl added to native-inputs: 644 passed, 66 skipped That seems like a nice improvement. :) live well, vagrant [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 227 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* [bug#70293] [PATCH] gnu: diffoscope: Update to 263. 2024-04-09 17:40 ` Vagrant Cascadian @ 2024-04-09 18:03 ` Christopher Baines 2024-04-09 18:32 ` Vagrant Cascadian 0 siblings, 1 reply; 7+ messages in thread From: Christopher Baines @ 2024-04-09 18:03 UTC (permalink / raw) To: Vagrant Cascadian; +Cc: Wilko Meyer, 70293 [-- Attachment #1: Type: text/plain, Size: 756 bytes --] Vagrant Cascadian <vagrant@reproducible-builds.org> writes: > On 2024-04-09, Vagrant Cascadian wrote: >> On 2024-04-09, Wilko Meyer wrote: >>> * gnu/packages/diffoscope.scm (diffoscope): Update to 263. >> >> Thanks for the update! >> >> Not a *huge* deal, but this version should have perl added to native >> inputs, as some tests now use "zipdetails" from perl, and are skipped if >> it is not present. Or does something else pull in perl? > > For a build without perl, the test coverage is: > > 638 passed, 72 skipped > > With perl added to native-inputs: > > 644 passed, 66 skipped > > That seems like a nice improvement. :) Cool, I think it would be useful if we could get the test suite to log more as well, since 66 is still quite a few tests. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 987 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* [bug#70293] [PATCH] gnu: diffoscope: Update to 263. 2024-04-09 18:03 ` Christopher Baines @ 2024-04-09 18:32 ` Vagrant Cascadian 2024-04-10 20:50 ` Vagrant Cascadian 0 siblings, 1 reply; 7+ messages in thread From: Vagrant Cascadian @ 2024-04-09 18:32 UTC (permalink / raw) To: Christopher Baines [-- Attachment #1: Type: text/plain, Size: 2365 bytes --] On 2024-04-09, Christopher Baines wrote: > Vagrant Cascadian <vagrant@reproducible-builds.org> writes: > >> On 2024-04-09, Vagrant Cascadian wrote: >>> On 2024-04-09, Wilko Meyer wrote: >>>> * gnu/packages/diffoscope.scm (diffoscope): Update to 263. >>> >>> Thanks for the update! >>> >>> Not a *huge* deal, but this version should have perl added to native >>> inputs, as some tests now use "zipdetails" from perl, and are skipped if >>> it is not present. Or does something else pull in perl? >> >> For a build without perl, the test coverage is: >> >> 638 passed, 72 skipped >> >> With perl added to native-inputs: >> >> 644 passed, 66 skipped >> >> That seems like a nice improvement. :) Or at least, not a regression from diffoscope 261: https://ci.guix.gnu.org/build/3903616/log/raw 644 passed, 66 skipped :) > Cool, I think it would be useful if we could get the test suite to log > more as well, since 66 is still quite a few tests. You mean log why something was skipped? Yeah, that would be nice... maybe cranking up the verbosity on the tests suites would help, if it doesn't generate too much noise. For the most part there are a few things not yet packaged for guix. I sometimes use this to hunt for additional things in need of packaging: guix shell --pure diffoscope $(guix shell --pure diffoscope -- diffoscope --list-missing | awk -F packages: '/Guix/{print $2}' | tr -d , | tr ' ' '\n' | sed -e '/^$/d') python-argcomplete python-androguard python-defusedxml python-jsondiff -- diffoscope --list-missing Which currently outputs: Available-in-Arch Linux-packages: mono Available-in-Debian-packages: aapt, apksigcopier, apksigner, apktool, coreboot-utils, dexdump, fontforge-extras, mono-utils, oggvideotools, perl, procyon-decompiler, xmlbeans Available-in-FreeBSD-packages: mono Available-in-GNU Guix-packages: External-Tools-Required: aapt2, apksigcopier, apksigner, apktool, cbfstool, dexdump, dumpxsb, lipo, oggDump, otool, pedump, procyon, showttf, zipdetails Missing-Python-Modules: guestfs, r2pipe Notably, I just added the mapping to get zipdetails for guix: https://salsa.debian.org/reproducible-builds/diffoscope/-/commit/90dd18830d9330c8fb8f87dd2ca50972574b6926 In the past I have sometimes temporarily monkey-patched such updates until it lands in a new upstream version. live well, vagrant [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 227 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* [bug#70293] [PATCH] gnu: diffoscope: Update to 263. 2024-04-09 18:32 ` Vagrant Cascadian @ 2024-04-10 20:50 ` Vagrant Cascadian 0 siblings, 0 replies; 7+ messages in thread From: Vagrant Cascadian @ 2024-04-10 20:50 UTC (permalink / raw) To: Christopher Baines; +Cc: Wilko Meyer, 70293 [-- Attachment #1: Type: text/plain, Size: 976 bytes --] On 2024-04-09, Vagrant Cascadian wrote: > On 2024-04-09, Christopher Baines wrote: >> Vagrant Cascadian <vagrant@reproducible-builds.org> writes: >> >>> On 2024-04-09, Vagrant Cascadian wrote: >>>> On 2024-04-09, Wilko Meyer wrote: >>>>> * gnu/packages/diffoscope.scm (diffoscope): Update to 263. >>>> >>>> Thanks for the update! >>>> >>>> Not a *huge* deal, but this version should have perl added to native >>>> inputs, as some tests now use "zipdetails" from perl, and are skipped if >>>> it is not present. Or does something else pull in perl? >>> >>> For a build without perl, the test coverage is: >>> >>> 638 passed, 72 skipped >>> >>> With perl added to native-inputs: >>> >>> 644 passed, 66 skipped >>> >>> That seems like a nice improvement. :) > > Or at least, not a regression from diffoscope 261: > > https://ci.guix.gnu.org/build/3903616/log/raw > > 644 passed, 66 skipped I pushed a fix for this regression as c1c9d6b3cdf5955f1bf5fded2a0c496ce2e631f1. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 227 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#70293: [PATCH] gnu: diffoscope: Update to 263. 2024-04-08 22:27 [bug#70293] [PATCH] gnu: diffoscope: Update to 263 Wilko Meyer 2024-04-09 16:36 ` Vagrant Cascadian @ 2024-04-09 17:49 ` Christopher Baines 1 sibling, 0 replies; 7+ messages in thread From: Christopher Baines @ 2024-04-09 17:49 UTC (permalink / raw) To: Wilko Meyer; +Cc: Vagrant Cascadian, 70293-done [-- Attachment #1: Type: text/plain, Size: 338 bytes --] Wilko Meyer <w@wmeyer.eu> writes: > * gnu/packages/diffoscope.scm (diffoscope): Update to 263. > > Change-Id: I8c9fce87dcf78fbc0f5ed319fa9e8b6d2e37fee1 > --- > gnu/packages/diffoscope.scm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Thanks, I've pushed this to master as ff5ef67110b6b4cf9dd165876ed53ece5b8514a7. Chris [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 987 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-04-10 20:51 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-04-08 22:27 [bug#70293] [PATCH] gnu: diffoscope: Update to 263 Wilko Meyer 2024-04-09 16:36 ` Vagrant Cascadian 2024-04-09 17:40 ` Vagrant Cascadian 2024-04-09 18:03 ` Christopher Baines 2024-04-09 18:32 ` Vagrant Cascadian 2024-04-10 20:50 ` Vagrant Cascadian 2024-04-09 17:49 ` bug#70293: " Christopher Baines
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/guix.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.