grub-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] fix building bug with --enable-grub-emu-usb


From: Pavel Roskin
Subject: Re: [PATCH] fix building bug with --enable-grub-emu-usb
Date: Fri, 19 Jun 2009 11:46:06 -0400

On Wed, 2009-06-17 at 15:27 +0200, Vladimir 'phcoder' Serbinenko wrote:
> Hello. This is a fix for problem in subject line. I believe this fix
> to be correct and legitimate but as I don't know ruby well could
> someone review it? If nobody knows ruby in depth I think we can commit
> this even without deep review

I would prefer that we take a different approach with grub-emu.  I think
the USB support should be enabled whenever libusb is enabled.  If libusb
is missing, then the USB related code should be omitted from the sources
using preprocessor directives.  The build system doesn't need to know
anything about it.

Generally, please think how to reduce complexity, not how to add to it.
Complex systems are harder to maintain.

-- 
Regards,
Pavel Roskin




reply via email to

[Prev in Thread] Current Thread [Next in Thread]