2 commits - imgcreate/live.py

[Jeroen van Meeuwen]

imgcreate/live.py |    7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

New commits:
commit 2e04352bdf3d1770c242e12e2d3b8461660e48eb
Author: Jeroen van Meeuwen (Ergo Project) 
Date:   Mon Jan 31 10:10:27 2011 -0700

Bad karma commit reverted; The option to boot from a local
    drive *MUST* exist as 99.9% of our consumers have default
    desktop hardware configurations.

If you seek to fix and/or improve, perhaps look at chain.c32's
    append hd 0 0 instead of the localboot hexadecimal addressing.

Revert "Remove boot from local drive option"

This reverts commit 150ce6e01e6d99764132220dd4b3983d8df0f812.

diff --git a/imgcreate/live.py b/imgcreate/live.py
index c26ec2c..60cd1de 100644
--- a/imgcreate/live.py
+++ b/imgcreate/live.py
@@ -555,7 +555,10 @@ menu hiddenrow 5
 """

def __get_local_stanza(self, isodir):
-        return ""
+        return """label local
+  menu label Boot from local drive
+  localboot 0xffff
+"""

def _configure_syslinux_bootloader(self, isodir):
         """configure the boot loader"""

commit fe20e690f2f1a14201db8bc95056746f07e4e7e7
Author: Jeroen van Meeuwen (Ergo Project) 
Date:   Mon Jan 31 10:05:59 2011 -0700

Really switch the default compression type, not just the default cli option value

diff --git a/imgcreate/live.py b/imgcreate/live.py
index cc8c630..c26ec2c 100644
--- a/imgcreate/live.py
+++ b/imgcreate/live.py
@@ -46,7 +46,7 @@ class LiveImageCreatorBase(LoopImageCreator):
         """
         LoopImageCreator.__init__(self, ks, name, fslabel=fslabel, releasever=releasever)

-        self.compress_type = "gzip"
+        self.compress_type = "xz"
         """mksquashfs compressor to use."""

self.skip_compression = False

[Bruno Wolff III]

On Mon, Jan 31, 2011 at 17:12:12 +0000,
  Jeroen van Meeuwen  wrote:
> Really switch the default compression type, not just the default cli option value

Thanks for catching that.

[Brian C. Lane]

On Mon, Jan 31, 2011 at 05:12:12PM +0000, Jeroen van Meeuwen wrote:
> imgcreate/live.py |    7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> New commits:
> commit 2e04352bdf3d1770c242e12e2d3b8461660e48eb
> Author: Jeroen van Meeuwen (Ergo Project) 
> Date:   Mon Jan 31 10:10:27 2011 -0700
> Bad karma commit reverted; The option to boot from a local
> drive *MUST* exist as 99.9% of our consumers have default
> desktop hardware configurations.

So you are saying that this actually *does* work for a significant
number of users? It doesn't work on any of the hardware I have, nor for
a number of other people I've talked to.
> If you seek to fix and/or improve, perhaps look at chain.c32's
> append hd 0 0 instead of the localboot hexadecimal addressing.

If that will make it work for more people that's a good idea. I'll look
into it.

Thanks,

Brian