Re: [PATCH] Re-enable grub-extras

2010-09-24 Thread Colin Watson
On Thu, Sep 23, 2010 at 04:36:33PM +0200, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > Go ahead for both trunk and extras Thanks. I've committed all this now. Note that this means that grub-extras is no longer compatible with 1.98; there were some modules where the contortions required to do

Re: [PATCH] Re-enable grub-extras

2010-09-23 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Go ahead for both trunk and extras On 09/23/2010 03:08 PM, Colin Watson wrote: > On Tue, Sep 21, 2010 at 07:35:30PM +0100, Colin Watson wrote: > >> Here's a patch to re-enable grub-extras by allowing extra modules to >> provide Autogen definitions files. A few things to note: >> > Vladimir

Re: [PATCH] Re-enable grub-extras

2010-09-23 Thread Colin Watson
On Thu, Sep 23, 2010 at 02:12:53PM +0100, Colin Watson wrote: > On Thu, Sep 23, 2010 at 02:08:20PM +0100, Colin Watson wrote: > > Once again, the required patches to grub-extras modules are attached. > > No, really! Here they are. At Keshav's request, I made sure 'make install' worked properly.

Re: [PATCH] Re-enable grub-extras

2010-09-23 Thread Colin Watson
On Thu, Sep 23, 2010 at 02:08:20PM +0100, Colin Watson wrote: > Once again, the required patches to grub-extras modules are attached. No, really! Here they are. -- Colin Watson [cjwat...@ubuntu.com] === added file '.bzrignore' --- .bzrignore 1970-01-01 00:0

Re: [PATCH] Re-enable grub-extras

2010-09-23 Thread Colin Watson
On Tue, Sep 21, 2010 at 07:35:30PM +0100, Colin Watson wrote: > Here's a patch to re-enable grub-extras by allowing extra modules to > provide Autogen definitions files. A few things to note: Vladimir commented on this on IRC, noting three problems. In reverse order of complexity: 1) The GRUB_C

[PATCH] Re-enable grub-extras

2010-09-21 Thread Colin Watson
Here's a patch to re-enable grub-extras by allowing extra modules to provide Autogen definitions files. A few things to note: * I had to have autogen.sh create 'contrib' symlinks in the source tree, as Automake got hopelessly confused when I had $(GRUB_CONTRIB) in source file names - au