Re: [Xen-devel] [PATCH OSSTEST v7 04/15] distros: Support pvgrub for Wheezy too.

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("[PATCH OSSTEST v7 04/15] distros: Support pvgrub for 
Wheezy too."):
> This requires us to install pv-grub-menu from backports, which we do
> using a late_command.

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

However:

> +            preseed_hook_command($ho, 'late_command', $sfx, <<END);
> +#!/bin/sh
> +set -ex
> +
> +(
> +    echo
> +    echo \\\# $suite backports
> +    echo deb http://$c{DebianMirrorHost}/$c{DebianMirrorSubpath} 
> $suite-backports main
> +) >> /target/etc/apt/sources.list
> +in-target apt-get update
> +in-target apt-get install -y -t wheezy-backports pv-grub-menu
> +END

This way of saying this requires quite a lot of quoting!  Have you
considered using a nested here document ?  In shell you can say
<<'END2' or something and the content will not be subject to
$-expansion.

Ian.

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