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

[Vladimir 'φ-coder/phcoder' Serbinenko]

Go ahead. I mistagged this one in my inbox and hence missed it, sorry
for this
On 25.07.2013 19:06, Andrey Borzenkov wrote:
> В Thu, 25 Jul 2013 16:44:08 +0200
> Vladimir 'φ-coder/phcoder' Serbinenko <address@hidden> пишет:
> 
>> 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;
>  };
>  
> 
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/grub-devel
>

signature.asc
Description: OpenPGP digital signature