Re: [PATCH 2/2] Disable lzo compression, lzma is doing its job just fine

2009-07-19 Thread Pavel Roskin
On Sun, 2009-07-19 at 20:24 +0200, Vladimir 'phcoder' Serbinenko wrote: > Now I agree with you. No further oppositions from me Committed. -- Regards, Pavel Roskin ___ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/gr

Re: [PATCH 2/2] Disable lzo compression, lzma is doing its job just fine

2009-07-19 Thread Vladimir 'phcoder' Serbinenko
> There then there is an issue of user choice.  I think we are offering > too many choices without explaining what's behind it. > > I could say lzo is "old and proven" and lzma is "new and more > effective", but since lzo is disabled by default and there are no > complaints about it, keeping lzo be

Re: [PATCH 2/2] Disable lzo compression, lzma is doing its job just fine

2009-07-19 Thread Pavel Roskin
On Sat, 2009-07-18 at 20:18 +0200, Robert Millan wrote: > It doesn't break things, since it's barely modified, and doesn't interact with > the rest of the code, but simply having more code means an added work to > maintain it when we restructure things, etc. It needs to pay off in some way. Exac

Re: [PATCH 2/2] Disable lzo compression, lzma is doing its job just fine

2009-07-18 Thread Robert Millan
On Thu, Jul 16, 2009 at 06:24:14PM +0200, Vladimir 'phcoder' Serbinenko wrote: > On Thu, Jul 16, 2009 at 1:40 AM, Pavel Roskin wrote: > > ChangeLog: > > > >        * Makefile.in: Remove LIBLZO and enable_lzo. > >        * conf/i386-pc.rmk: Remove lzo support. > >        * configure.ac: Remove check

Re: [PATCH 2/2] Disable lzo compression, lzma is doing its job just fine

2009-07-16 Thread Vladimir 'phcoder' Serbinenko
On Thu, Jul 16, 2009 at 1:40 AM, Pavel Roskin wrote: > ChangeLog: > >        * Makefile.in: Remove LIBLZO and enable_lzo. >        * conf/i386-pc.rmk: Remove lzo support. >        * configure.ac: Remove checks for lzo, don't define ENABLE_LZMA. >        * include/grub/i386/pc/kernel.h: Define ENABL

[PATCH 2/2] Disable lzo compression, lzma is doing its job just fine

2009-07-15 Thread Pavel Roskin
ChangeLog: * Makefile.in: Remove LIBLZO and enable_lzo. * conf/i386-pc.rmk: Remove lzo support. * configure.ac: Remove checks for lzo, don't define ENABLE_LZMA. * include/grub/i386/pc/kernel.h: Define ENABLE_LZMA. Remove lzo support. * kern/i386/pc/