[PATCH] gnu: genimage: Update to 11.

  • Done
  • quality assurance status badge
Details
3 participants
  • Danny Milosavljevic
  • Mathieu Othacehe
  • Vincent Legoll
Owner
unassigned
Submitted by
Vincent Legoll
Severity
normal
V
V
Vincent Legoll wrote on 30 Mar 2020 20:34
(address . guix-patches@gnu.org)
CAEwRq=rDU9xoL4mRmnoTcAZVBXCa9ddO_P9dWG-BYxTnnaFGYQ@mail.gmail.com
Hello,

here is an update to version 11 for genimage,
this is not the latest, as there is a test suite
failure for version 12 that I cannot understand.

The output for this test failure is in the attached
log file.

It looks like the created image uses 2 filesystem
block more than the expected, and so the bitmap
checksum changes, etc.

Danny, you close the issue:
what was the outcome ?

I want to have your word on this before creating an
issue upstream, if we can solve it on our side.

So, until this test suite failure is solved we can still
get the version 11 in...

--
Vincent Legoll
Attachment: genimage-v12-test-suite-failure.txt
From e64a6703fa1bbe79364df14848870fdc988745c9 Mon Sep 17 00:00:00 2001
From: Vincent Legoll <vincent.legoll@gmail.com>
Date: Fri, 27 Mar 2020 22:10:06 +0100
Subject: [PATCH] gnu: genimage: Update to 11.

* gnu/packages/genimage.scm (genimage): Update to 11.
[arguments]: add a new test dump file to the patched ones
because of missing /etc/passwd in build environment.
---
gnu/packages/genimage.scm | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)

Toggle diff (32 lines)
diff --git a/gnu/packages/genimage.scm b/gnu/packages/genimage.scm
index fdf686f9d7..27f55a398a 100644
--- a/gnu/packages/genimage.scm
+++ b/gnu/packages/genimage.scm
@@ -41,7 +41,7 @@
(define-public genimage
(package
(name "genimage")
- (version "10")
+ (version "11")
(source (origin
(method git-fetch)
(uri (git-reference
@@ -50,7 +50,7 @@
(file-name (string-append name "-" version "-checkout"))
(sha256
(base32
- "0had00p2why2l1fl14mq7nbhmmfbd3na4qnnpg36akdy05g67jbn"))))
+ "15jmh17lvm3jw9c92bjarly7iwhmnfl322d91mprfv10ppb9ip54"))))
(build-system gnu-build-system)
(arguments
`(#:phases
@@ -149,6 +149,7 @@
(substitute* '("test/ext2test.dump"
"test/ext3test.dump"
"test/ext4test.dump"
+ "test/ext2test-percent.dump"
"test/mke2fs.dump")
(("root") "unknown"))
#t))
--
2.25.2
D
D
Danny Milosavljevic wrote on 30 Mar 2020 22:47
(name . Vincent Legoll)(address . vincent.legoll@gmail.com)(address . 40331@debbugs.gnu.org)
20200330224713.2545a005@scratchpost.org
On Mon, 30 Mar 2020 20:34:42 +0200
Vincent Legoll <vincent.legoll@gmail.com> wrote:

Toggle quote (7 lines)
> Danny, you close the issue:
> https://github.com/pengutronix/genimage/issues/36
> what was the outcome ?
>
> I want to have your word on this before creating an
> issue upstream, if we can solve it on our side.

I can't remember. I *think* it was because one of our tools had xattr support
and another one of our tools had no xattr support, so the xattrs would get lost
and that would change the image or something.
-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6CWtEACgkQ5xo1VCww
uqWqSggAmEPVGRWq590GKl4ih6ooU7yYXlD5Y5kGJZq/KACrOzTKVayRZPKrut0L
0RpAB6C0aL6iipaq4r5MGFNKbLitER9B797MC5RJZSZ7Udy2nbD3ayTUTBg0bTtP
An5XwlfpQN9OmT7/6NW7fFF0Xtg5DwCB9ZM/b9+N9SakTM45FDvnRKgSj2X6wWJS
wY7uys4BRMJMMJfKjpfdGfDJCR1JjnEtA3Y91B+Fn46gp1YSM2+UsgNDTpC93NUS
MYdKa13j1JXUdagydnQPJqKw48IdMl8l39gudJvPbYAykU3Ga15zOjosvxY0SjDt
lDKKvNg/+isyrCWov13sC+biWJ3UPg==
=ZnH+
-----END PGP SIGNATURE-----


M
M
Mathieu Othacehe wrote on 8 Apr 2020 11:30
(name . Vincent Legoll)(address . vincent.legoll@gmail.com)(address . 40331@debbugs.gnu.org)
87lfn62ukz.fsf@gmail.com
Hello Vincent,

Toggle quote (4 lines)
> here is an update to version 11 for genimage,
> this is not the latest, as there is a test suite
> failure for version 12 that I cannot understand.

Till we find a solution for v12 test failures, I applied your update to
v11.

Thanks for this patch,

Mathieu
M
M
Mathieu Othacehe wrote on 8 Apr 2020 11:32
(name . Vincent Legoll)(address . vincent.legoll@gmail.com)(address . 40331-done@debbugs.gnu.org)
87h7xu2ui0.fsf@gmail.com
And closing!

Mathieu
Closed
D
D
Danny Milosavljevic wrote on 4 May 2020 19:30
Re: [PATCH] gnu: genimage: Update to 12.
(name . Vincent Legoll)(address . vincent.legoll@gmail.com)(address . 40331@debbugs.gnu.org)
20200504193013.0a5a0ccb@scratchpost.org
Aha, guix bug 41081 maybe? cp is built without xattr support.
-----BEGIN PGP SIGNATURE-----

iQEzBAEBCgAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6wUSUACgkQ5xo1VCww
uqUWYQf/UxZY6GhgchSLUdYBvX/d1Dg0N2QYDqUPtJ+q9ENb2jGuMFrxLcQudeT2
aCru5oZ5hiEE/bB8i71pxhueoa0VFAEE5+17+fXwk64kTOwtmcc5C/eyGcaUXP2N
i5qeSKfC1u6eV1Ph1sHkjvlDbgtP3l9H7GwPy2sICaLcQkvJqON+dOZ0oFQKxbto
rr4wMAY/A12ICvoqa4oR56VqMHZuVdOgTkFBAa6b2ZIPbdT6TVZTEuq4yvITRLvx
+NA4vOtyppDPWm5vjIfPifaQH2kdNLdMKQn8xbZJwyC9lXI8qPxI+mRd0RK57nii
7MYc5/ZYlavL8sm8CDcx+ilDlhN7yQ==
=ySkM
-----END PGP SIGNATURE-----


V
V
Vincent Legoll wrote on 21 May 2020 21:51
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40331@debbugs.gnu.org)
4f65ebff-7961-6d1e-77ac-68a7e93cff8b@gmail.com
Hello,

On 04/05/2020 19:30, Danny Milosavljevic wrote:
Toggle quote (2 lines)
> Aha, guix bug 41081 maybe? cp is built without xattr support.

I tried to build the new genimage with a coreutils-with-xattr,
but am still getting a test suite failure. I'll have to dig
deeper.

Thanks

--
Vincent Legoll
?
Your comment

This issue is archived.

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

To respond to this issue using the mumi CLI, first switch to it
mumi current 40331
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