Re: [PATCH]: Add EFI conditional

[Francesco Lavra]

Hi,

On 01/27/2013 04:20 PM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> On 26.01.2013 20:42, Francesco Lavra wrote:
> 
>> This patch adds the automake conditional COND_efi, which is true for
>> every configuration using the EFI platform. This conditional is used in
>> grub-core/Makefile.am to remove some duplicated entries in
>> KERNEL_HEADER_FILES.
>> As the EFI platform is expected to be used in more architectures (work
>> is in progress to port GRUB to ARM using EFI), this patch will allow
>> not to have even more duplicated code in the future.
>>
> 
> ARM U-Boot port should be a priority, not EFI flavours.
> And this patch just moves the problem around. Rather than having to keep
> different clauses in Makefile.am in sync you have to keep gentpl.py and
> configure.ac in sync. Doesn't seem to be worth it.

Well, this patch adds to configure.ac what is already there in gentpl.py
(the "efi" group). Group definitions are already used throughout
Makefile.core.def to conditionally enable modules and source files, and
any new configuration using EFI would already need to be added to the
efi group.
In our specific case, if a configuration belongs to a efi group, this
enables compilation in the kernel of, among others, disk/efi/efidisk.c
and kern/efi/efi.c, which define the global symbols exported by the
kernel headers. It just seems easier and more intuitive to add the
respective header files in Makefile.am under the same logical condition
which enables source file compilation in Makefile.core.def.
This patch would avoid having to re-declare the kernel header files for
any new configurations using EFI, and wouldn't require any additional
boilerplate elsewhere.
Anyway, if you don't feel like it's worth the trouble, no problem for
me, I won't insist any further :)

Regards,
Francesco