[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#643585: Bug 643585



Daniel,

I've fixed the git tree (rebased) to address the packages list; you
should you re-fetch the tree.

I think the white space fun was caused by bad options in my .vimrc.
Sorry for the back and forth on that. Out of courosity,

I'm just going to submit the 
0001-Adding-Ubuntu-Cloud-Images-package-lists.patch today. 

I'm going to spend a bit of time thinking about the binary-hdd-raw and
binary-hdd-virt (to support qcow and vmdk's).

~Ben


On 10/01/2011 12:10 AM, Daniel Baumann wrote:
> On 10/01/2011 01:08 AM, Ben Howard wrote:
>> 0001-Adding-Ubuntu-Cloud-Images-package-lists.patch:
>>     - Adds Ubuntu Cloud images packages list
> i'm sorry..
>
>   * in ubuntu-cloud, you have a useless whitespace after landscape,
>     plus a useless empty line at the end of the file.
>
>     what editor do you use, and don't you highlight (or strip) those?
>
>   * the comment in at the top of the file is supposed to refer to the
>     actual file, not to the 'standard'.list where you probably copied
>     it from.
>
>   * #include <ubunut-cloud> looks like a typo and wouldn't work.
>
>   * ubuntu-minimal is not a task, but a package, therefore you should
>     list it in the packages section, not the task one in ubuntu-cloud.
>
> otherwise good.
>
>> 0002-Adding-check-for-valid-mtab-during-chrooted-binary-r.patch:
>>     - Installs and reverts /etc/mtab if missing
>
> applied, thanks.
>
>> 0003-Adding-foreign-bootstrap-via-qemu-for-cross-arch-bui.patch
>>     - Adds LB_BOOTSTRAP_QEMU_*
> looks mostly ok, i'll try it out (not sure i'll get arround to it during
> the weekend as i'm mostly away, at latest during monday) and then let
> you know.
>
>> Tested using lb_config commands of:
>> lb_config --package-lists ubuntu-cloud --mode ubuntu --initramfs none
>> --initsystem none --bootloader none -d oneiric -b rootfs
>> --build-with-chroot true  --chroot-filesystem ext3 -a armel
>> --bootstrap-qemu-arch armel --bootstrap-qemu-static /usr/bin/qemu-arm-static
> jftr: with lb >= 3.0~a34-1, the above lb config command can be reduced to:
>
>   lb_config --package-lists ubuntu-cloud --mode ubuntu --system normal
>   --bootloader none -b rootfs --chroot-filesystem ext3 -a armel
>   --bootstrap-qemu-arch armel --bootstrap-qemu-static /usr/bin/qemu-
>   arm-static
>

-- 


Ben Howard
ben.howard@canonical.com
Canonical USA, Inc
GPG ID 0x5406A866



>From a9c18f2c344a3226cb32740b4a345e4593324e43 Mon Sep 17 00:00:00 2001
From: Ben Howard <ben.howard@canonical.com>
Date: Fri, 30 Sep 2011 16:23:00 -0600
Subject: [PATCH 1/3] Adding Ubuntu Cloud images packages lists.

---
 package-lists/ubuntu-cloud         |    9 +++++++++
 package-lists/ubuntu-cloud-desktop |    5 +++++
 2 files changed, 14 insertions(+), 0 deletions(-)
 create mode 100644 package-lists/ubuntu-cloud
 create mode 100644 package-lists/ubuntu-cloud-desktop

diff --git a/package-lists/ubuntu-cloud b/package-lists/ubuntu-cloud
new file mode 100644
index 0000000..d890fd1
--- /dev/null
+++ b/package-lists/ubuntu-cloud
@@ -0,0 +1,9 @@
+# /usr/share/live/build/package-lists/ubuntu-cloud - package list for live-build(7)
+
+#include <minimal>
+
+# Tasks
+standard^ server^ cloud-image^
+
+# Packages
+landscape-client ubuntu-minimal
diff --git a/package-lists/ubuntu-cloud-desktop b/package-lists/ubuntu-cloud-desktop
new file mode 100644
index 0000000..f9a6231
--- /dev/null
+++ b/package-lists/ubuntu-cloud-desktop
@@ -0,0 +1,5 @@
+# /usr/share/live/build/package-lists/ubuntu-cloud-desktop - package list for live-build(7)
+
+#include <ubuntu-cloud>
+
+ubuntu-destkop^
--
1.7.5.4


Reply to: