Re: [PATCH] enable emuusb and emupci on emu platform only

[Andrey Borzenkov <arvidjaar@gmail.com>]

В Thu, 25 Jul 2013 16:44:08 +0200
Vladimir 'φ-coder/phcoder' Serbinenko <phcoder@gmail.com> пишет:

> On 21.07.2013 08:53, Andrey Borzenkov wrote:
> > Is there any reason we may want to build them when not building
> > --with-platform=emu?
> >
> I think it's better to reset COND_EMU_PCI/COND_EMU_USB to false in 
> configure state when not compiling for emu as usage of both enable and 
> consition simultaneously is marginal and it's beter not to rely on it 
> (so to i.a. permit its removal if necessary)

I think it makes sense to use both. "enable = emu" makes it obvious
that module should not be built on other platform. It also makes
resulting Makefile.am slightly more readable.

Actually it is already used this way for SDL.

Like this?

diff --git a/configure.ac b/configure.ac
index 4fe20a3..acd37f2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -949,6 +949,13 @@ fi
 AC_SUBST([enable_grub_emu_sdl])
 AC_SUBST([enable_grub_emu_usb])
 AC_SUBST([enable_grub_emu_pci])
+
+else
+
+# Ignore --enable-emu-* if platform is not emu
+enable_grub_emu_sdl=no
+enable_grub_emu_usb=no
+enable_grub_emu_pci=no
 fi
 
 AC_ARG_ENABLE([grub-mkfont],
diff --git a/grub-core/Makefile.core.def b/grub-core/Makefile.core.def
index 06617d7..230bed5 100644
--- a/grub-core/Makefile.core.def
+++ b/grub-core/Makefile.core.def
@@ -477,6 +477,7 @@ module = {
 module = {
   name = emuusb;
   common = bus/usb/usb.c;
+  enable = emu;
   condition = COND_GRUB_EMU_USB;
 };
 
@@ -546,6 +547,7 @@ module = {
   common = bus/emu/pci.c;
   common = commands/lspci.c;
 
+  enable = emu;
   condition = COND_GRUB_EMU_PCI;
 };