xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Jackson <ian.jackson@eu.citrix.com>
To: xen-devel@lists.xenproject.org
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>
Subject: [OSSTEST PATCH 10/35] backports handling: Refactor kernel deb handling
Date: Fri, 20 Jan 2017 12:18:59 +0000	[thread overview]
Message-ID: <1484914764-20861-11-git-send-email-ian.jackson@eu.citrix.com> (raw)
In-Reply-To: <1484914764-20861-1-git-send-email-ian.jackson@eu.citrix.com>

Introduce private function $kdeb->().  No functional change.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 Osstest/Debian.pm | 45 +++++++++++++++++++++++++++++----------------
 1 file changed, 29 insertions(+), 16 deletions(-)

diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm
index 52527bf..5a083c2 100644
--- a/Osstest/Debian.pm
+++ b/Osstest/Debian.pm
@@ -1114,32 +1114,45 @@ END
 	$kp = "need-kernel-deb-wheezy-backports" if $kp eq "need-kernel-deb-armmp";
 	$kp =~ s/need-kernel-deb-$ho->{Suite}-// or next;
 
-	my $kern = "$d_i/$kp.deb";
-	my $kurl = create_webfile($ho, "kernel", sub {
-	    copy($kern, $_[0])
-		or die "Copy kernel $kern failed: $!";
-        });
-
-	my $ird = "$d_i/initramfs-tools.deb";
-	my $iurl = create_webfile($ho, "initramfs-tools", sub {
-	    copy($ird, $_[0])
-		or die "Copy initramfs-tools $ird failed: $!";
-        });
-
-	preseed_hook_command($ho, 'late_command', $sfx, <<END);
+        my $cmds = <<END;
 #!/bin/sh
 set -ex
 
 r=/target
 
-$preseed_wget -O \$r/tmp/kern.deb $kurl
-$preseed_wget -O \$r/tmp/initramfs-tools.deb $iurl
+END
+
+        my @kdebs;
+        my $kdeb = sub {
+            my ($what, $file) = @_;
+
+            logm("for late_command using $what $file");
+            my $furl = create_webfile($ho, $what, sub {
+	        copy($file, $_[0]) or die "Copy $what $file failed: $!";
+            });
+            my $tmpfn = "/tmp/$what.deb";
+
+            $cmds .= <<END;
+$preseed_wget -O \$r$tmpfn $furl
+END
+            push @kdebs, $tmpfn;
+        };
+
+        my $kern = "$d_i/$kp.deb";
+        $kdeb->("kernel", $kern);
+
+        my $ird = "$d_i/initramfs-tools.deb";
+        $kdeb->("initramfs-tools", $ird);
+
+        $cmds .= <<END;
 
 # This will fail due to dependencies...
-in-target dpkg -i /tmp/kern.deb /tmp/initramfs-tools.deb || true
+in-target dpkg -i @kdebs || true
 # ... Now fix everything up...
 in-target apt-get install -f -y
 END
+
+	preseed_hook_command($ho, 'late_command', $sfx, $cmds);
     }
 
     if ( $ho->{Flags}{'need-uboot-bootscr'} ) {
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

  parent reply	other threads:[~2017-01-20 12:20 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-20 12:18 [OSSTEST PATCH 00/35] ARM64 softiron support Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 01/35] Serial console: honour LinuxSerialConsole in bootloader setup Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 02/35] ts-host-install: Expect "pool" in ntp.conf, not only "server" Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 03/35] ts-host-install: NTP not honoured bug remains Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 04/35] mg-debian-installer-update: Break out default setting of src= Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 05/35] mg-debian-installer-update: Print working directory to stderr Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 06/35] mg-debian-installer-update: Log our version into our output Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 07/35] mg-debian-installer-update: backports: Provide pps and ptp Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 08/35] mg-debian-installer-update: backports: Add modules for EFI Ian Jackson
2017-01-20 12:18 ` [OSSTEST PATCH 09/35] mg-debian-installer-update: backports: Provide crypto Ian Jackson
2017-01-20 12:18 ` Ian Jackson [this message]
2017-01-20 12:19 ` [OSSTEST PATCH 11/35] backports handling: Make backports initramfs-tools optional Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 12/35] backports handling: Cope with dtbs in subdirs Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 13/35] d-i: Cope if .ssh already exists (in /target/root/ and also /home/...) Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 14/35] d-i grub: Extend UEFI workaround to stretch Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 15/35] pxe templates: Do defaulting outside host_pxefile Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 16/35] netboot: Rework interface to host_netboot_file (was host_pxefile) Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 17/35] netboot: Rename pxeboot functions Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 18/35] netboot: Change pxe to netboot in comments, docs and messages Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 19/35] netboot: Change pxe to netboot in variable names Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 20/35] netboot: Rename mg-netgrub-loader-update Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 21/35] netboot: Rename ./mg-hosts mknetbootdir subcommand Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 22/35] netboot: Break out subst_netboot_template Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 23/35] netboot: Introduce TftpNetGrub* Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 24/35] netboot: Change stash file name to not be pxelinux-specific Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 25/35] mg-netgrub-loader-update: Introduce $grubbase Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 26/35] mg-netgrub-loader-update: Introduce $img Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 27/35] netboot: UEFI grub: Do it more like pxe Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 28/35] netboot: mg-netgrub-loader-update: Print sample dhcp config fragment Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 29/35] netboot: Rename TftpNetbootGroup (from TftpPxeGroup) Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 30/35] netboot: mg-netgrub-loader-update: Add some more docs Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 31/35] d-i: preseed_create_guest: Break out preseed_backports_packages Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 32/35] d-i: backports: When using backports kernel for a host, install linux-base Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 33/35] d-i: When using backports kernel, suppress kernel modules warning Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 34/35] d-i: Add comment about interrupting the install Ian Jackson
2017-01-20 12:19 ` [OSSTEST PATCH 35/35] ARM64: Create jobs, and provide the installer (with backports kernel) Ian Jackson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1484914764-20861-11-git-send-email-ian.jackson@eu.citrix.com \
    --to=ian.jackson@eu.citrix.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).