[PATCH 1/2] gnu: genimage: upgrade to v14

  • Done
  • quality assurance status badge
Details
2 participants
  • Mathieu Othacehe
  • Ryan Sundberg
Owner
unassigned
Submitted by
Ryan Sundberg
Severity
normal

Debbugs page

Ryan Sundberg wrote 3 years ago
(address . guix-patches@gnu.org)(name . Ryan Sundberg)(address . ryan@arctype.co)
20211104083511.11852-1-ryan@arctype.co
Upgrades genimage from version 11 to 14. A patch is required for the
tests due to differences in ext tooling.

* gnu/local.mk: Replace patch reference
* gnu/packages/genimage.scm: Update package source
* gnu/packages/patches/genimage-mke2fs-test.patch: New patch file
* gnu/packages/patches/genimage-signedness.patch: Delete obsolete patch
---
gnu/local.mk | 2 +-
gnu/packages/genimage.scm | 8 ++--
.../patches/genimage-mke2fs-test.patch | 40 +++++++++++++++++++
.../patches/genimage-signedness.patch | 24 -----------
4 files changed, 45 insertions(+), 29 deletions(-)
create mode 100644 gnu/packages/patches/genimage-mke2fs-test.patch
delete mode 100644 gnu/packages/patches/genimage-signedness.patch

Toggle diff (117 lines)
diff --git a/gnu/local.mk b/gnu/local.mk
index 17d8cb60e0..d550cff648 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1112,7 +1112,7 @@ dist_patch_DATA = \
%D%/packages/patches/gdk-pixbuf-CVE-2020-29385.patch \
%D%/packages/patches/gdm-default-session.patch \
%D%/packages/patches/geary-CVE-2020-24661.patch \
- %D%/packages/patches/genimage-signedness.patch \
+ %D%/packages/patches/genimage-mke2fs-test.patch \
%D%/packages/patches/geoclue-config.patch \
%D%/packages/patches/ghc-8.0-fall-back-to-madv_dontneed.patch \
%D%/packages/patches/ghc-testsuite-dlopen-pie.patch \
diff --git a/gnu/packages/genimage.scm b/gnu/packages/genimage.scm
index bd77e4dc33..366ff62ab4 100644
--- a/gnu/packages/genimage.scm
+++ b/gnu/packages/genimage.scm
@@ -42,7 +42,7 @@
(define-public genimage
(package
(name "genimage")
- (version "11")
+ (version "14")
(source (origin
(method git-fetch)
(uri (git-reference
@@ -51,9 +51,9 @@
(file-name (git-file-name name version))
(sha256
(base32
- "15jmh17lvm3jw9c92bjarly7iwhmnfl322d91mprfv10ppb9ip54"))
- ;; will be shipped with release 14
- (patches (search-patches "genimage-signedness.patch"))))
+ "1l45djpbaffhyw0allq3mgzwrdilk05iyj0nvp8l3s47vnp5bnbs"))
+ ;; Different version of ext4
+ (patches (search-patches "genimage-mke2fs-test.patch"))))
(build-system gnu-build-system)
(arguments
`(#:phases
diff --git a/gnu/packages/patches/genimage-mke2fs-test.patch b/gnu/packages/patches/genimage-mke2fs-test.patch
new file mode 100644
index 0000000000..c4a17590f2
--- /dev/null
+++ b/gnu/packages/patches/genimage-mke2fs-test.patch
@@ -0,0 +1,40 @@
+diff --git a/test/mke2fs.dump b/test/mke2fs.dump
+index 8e63662..0c25798 100644
+--- a/test/mke2fs.dump
++++ b/test/mke2fs.dump
+@@ -11,7 +11,7 @@ Filesystem OS type: Linux
+ Inode count: 8192
+ Block count: 32768
+ Reserved block count: 1638
+-Free blocks: 26568
++Free blocks: 26566
+ Free inodes: 8141
+ First block: 1
+ Block size: 1024
+@@ -29,7 +29,7 @@ Mount count: 0
+ Maximum mount count: -1
+ Last checked: Sat Jan 1 00:00:00 2000
+ Check interval: 0 (<none>)
+-Lifetime writes: 141 kB
++Lifetime writes: 143 kB
+ Reserved blocks uid: 0 (user root)
+ Reserved blocks gid: 0 (group root)
+ First inode: 11
+@@ -49,13 +49,13 @@ Journal sequence: 0x00000001
+ Journal start: 0
+
+
+-Group 0: (Blocks 1-8192) csum 0x74a0 [ITABLE_ZEROED]
++Group 0: (Blocks 1-8192) csum 0x3f30 [ITABLE_ZEROED]
+ Primary superblock at 1, Group descriptors at 2-2
+- Block bitmap at 3 (+2), csum 0x16cec4db
++ Block bitmap at 3 (+2), csum 0x8d67cdef
+ Inode bitmap at 7 (+6), csum 0xb1052088
+ Inode table at 11-522 (+10)
+- 6093 free blocks, 1997 free inodes, 18 directories, 1997 unused inodes
+- Free blocks: 2100-8192
++ 6091 free blocks, 1997 free inodes, 18 directories, 1997 unused inodes
++ Free blocks: 2102-8192
+ Free inodes: 52-2048
+ Group 1: (Blocks 8193-16384) csum 0x8fde [INODE_UNINIT, BLOCK_UNINIT, ITABLE_ZEROED]
+ Backup superblock at 8193, Group descriptors at 8194-8194
diff --git a/gnu/packages/patches/genimage-signedness.patch b/gnu/packages/patches/genimage-signedness.patch
deleted file mode 100644
index 2204dfb545..0000000000
--- a/gnu/packages/patches/genimage-signedness.patch
+++ /dev/null
@@ -1,24 +0,0 @@
-commit 6574665f9d6c0757e8f55ccb465afbcaa90bf019
-Author: Timotej Lazar <timotej.lazar@araneo.si>
-Date: Wed Aug 19 19:36:02 2020 +0200
-
- test: normalize flags reported by dumpe2fs
-
- Filesystem flags depend on the system – usually signed_directory_hash,
- but unsigned_directory_hash at least on arm64.
-
- Signed-off-by: Timotej Lazar <timotej.lazar@araneo.si>
-
-diff --git a/test/basic-images.test b/test/basic-images.test
-index f6685e1..cb104c3 100755
---- a/test/basic-images.test
-+++ b/test/basic-images.test
-@@ -158,6 +158,8 @@ check_ext() {
- # format change
- csum2="Group 0: (Blocks 1-4095) \\[ITABLE_ZEROED\\]\| Checksum .*, unused inodes 205"
- dumpe2fs "${1}" | grep -v "^\($uuid\|$seed\|$csum1\|$csum2\)" > "dump" &&
-+ # some architectures (including arm64) use unsigned char
-+ sed -i 's/un\(signed_directory_hash\)/\1/' "dump" &&
- # fixup for ext3 journal size with old tune2fs
- sed -i 's/^\(Journal size: \)1029k$/\11024k/' "dump" &&
- # output format changed with some version
--
2.31.1
Mathieu Othacehe wrote 3 years ago
(name . Ryan Sundberg)(address . ryan@arctype.co)(address . 51599-done@debbugs.gnu.org)
871r3lebza.fsf@gnu.org
Hello Ryan,

Toggle quote (5 lines)
> * gnu/local.mk: Replace patch reference
> * gnu/packages/genimage.scm: Update package source
> * gnu/packages/patches/genimage-mke2fs-test.patch: New patch file
> * gnu/packages/patches/genimage-signedness.patch: Delete obsolete patch

Pushed with an edited commit message.

Thanks,

Mathieu
Closed
?
Your comment

This issue is archived.

To comment on this conversation send an email to 51599@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 51599
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help