From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1TzVHa-0005fz-Hi for mharc-grub-devel@gnu.org; Sun, 27 Jan 2013 11:40:38 -0500 Received: from eggs.gnu.org ([208.118.235.92]:46061) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TzVHV-0005f2-9l for grub-devel@gnu.org; Sun, 27 Jan 2013 11:40:37 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TzVHS-0004Wi-CX for grub-devel@gnu.org; Sun, 27 Jan 2013 11:40:33 -0500 Received: from mail-bk0-f42.google.com ([209.85.214.42]:36536) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TzVHS-0004WY-4a for grub-devel@gnu.org; Sun, 27 Jan 2013 11:40:30 -0500 Received: by mail-bk0-f42.google.com with SMTP id jk7so157201bkc.29 for ; Sun, 27 Jan 2013 08:40:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:message-id:date:from:user-agent:mime-version:to:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=sNIXZ0ZbCmrg51R2qLoEX2iPTWv1+XOPP2z6Ucyq5PY=; b=B5zeHF+IGzX3qgLqeF5vCMx4LePVAeg13A1QMMxvgnkawpJNUS3AMDaJKLMdbgiTKg 4M+eGdyMLc3n9azsfEtZ+kIfoOJfYTgPc5n8KsC6t2IThkpVYaj51EqHN9mUmDuS1a4f MVVlTTBQCWAfLZ/cuQr34g6in8vxNnhRR8dUDWM2fWUvvszCLSAh0dVmgv9nx9nfDK2k rg1b55xpunmefE/bwvsBgeCYlZlIwVtXYzxgJEe2C1yRzmIbchYRHwKWS0swjHiE7vCx g/SM7A/wszGycjCbOp8O9yx/LdCd34iDUBZu8L4/Z9Y3TGilTGRC9IdbRlAao1pcrOol P8gw== X-Received: by 10.204.3.205 with SMTP id 13mr3275684bko.38.1359304829165; Sun, 27 Jan 2013 08:40:29 -0800 (PST) Received: from [192.168.56.2] ([151.36.152.7]) by mx.google.com with ESMTPS id fs20sm1877321bkc.8.2013.01.27.08.40.27 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sun, 27 Jan 2013 08:40:28 -0800 (PST) Message-ID: <5105589F.9090206@gmail.com> Date: Sun, 27 Jan 2013 17:41:03 +0100 From: Francesco Lavra User-Agent: Mozilla/5.0 (X11; Linux i686; rv:7.0.1) Gecko/20110929 Thunderbird/7.0.1 MIME-Version: 1.0 To: grub-devel@gnu.org Subject: Re: [PATCH]: Add EFI conditional References: <510431B2.9010409@gmail.com> <510545C3.3090308@gmail.com> In-Reply-To: <510545C3.3090308@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 209.85.214.42 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 27 Jan 2013 16:40:37 -0000 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