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 _______________________________________________ Grub-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/grub-devel
