From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:4001:c06::244; helo=mail-io0-x244.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-io0-x244.google.com (mail-io0-x244.google.com [IPv6:2607:f8b0:4001:c06::244]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 2C135221660C7 for ; Tue, 28 Nov 2017 07:49:20 -0800 (PST) Received: by mail-io0-x244.google.com with SMTP id w127so279461iow.11 for ; Tue, 28 Nov 2017 07:53:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=zKsdO5EcVDpxbLgog/KHS//d0Kc5GgzdIPkmCruh+t8=; b=NyTU4yNmYH0ECBC1JuXwq6TezPa9PqAc4msRxO60R2Aj9r5HwzIPkBSXyAtbd6Vdx0 JPQvSOgz5yCaqIGAn39u23wL526N9nTHSotajosu8r9SuQJyTv1weZL2Pmp+dCDDBlo4 qmn/ubXCVeSQDaHtsNjUkv141yZPb3HKTKxU8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=zKsdO5EcVDpxbLgog/KHS//d0Kc5GgzdIPkmCruh+t8=; b=byD9KjGW/sf+6kkQCDHlNOXoUF50hnvHUvnEbo7zCVhJchGVB7WdAAPbWBEGrAW0u1 oA3kbGi7PUswZetx+zKEaR9FERRnS72B+7S4kl2uXG/JK878ZI2/d7dB8oQs+QJrMkGq luRIvWP5VYuZ7v6zXmmtO9OzfYMfXMCIFdJIr6QmibpjTq4Hv164AFxHZHP1jK4QCYHM v0C7buyXJSJuEEnqXXmgb1OwuSbzF+y8rF13YftKEOXmxvnw/aQ4YcwvGlfVvJ55g7Gp TBTIsgYq/uotPYCJ/hf6jiwjZjkr34y0T/ujr0vFIex1AJWoa/j11QGzXWRgOds0642R qxqg== X-Gm-Message-State: AJaThX4kXjHaqEM9cKjnGZVjtES4/9PmQvIDQCdnMk4tAOHuUUvjCO1K pzBy6ULnsE1eyS51lpRd3yOFdOlabSXK+ZVpRXSHPQ== X-Google-Smtp-Source: AGs4zMZHdzVCaRlU7V10KR0p5hlfl+l3Q9PTarp03JIEXd+vMOJGqtlZnxL/2DVZDOnQHkNizHS+jM4UhMDQkSpvqJI= X-Received: by 10.107.59.85 with SMTP id i82mr672864ioa.253.1511884423480; Tue, 28 Nov 2017 07:53:43 -0800 (PST) MIME-Version: 1.0 Received: by 10.107.104.16 with HTTP; Tue, 28 Nov 2017 07:53:43 -0800 (PST) In-Reply-To: <20171128120009.176316-5-ruiyu.ni@intel.com> References: <20171128120009.176316-1-ruiyu.ni@intel.com> <20171128120009.176316-5-ruiyu.ni@intel.com> From: Ard Biesheuvel Date: Tue, 28 Nov 2017 15:53:43 +0000 Message-ID: To: Ruiyu Ni Cc: "edk2-devel@lists.01.org" , Leif Lindholm Subject: Re: [PATCH v2 4/6] BeagleBoardPkg: Fix build failure due to Tftp library removal X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Nov 2017 15:49:21 -0000 Content-Type: text/plain; charset="UTF-8" On 28 November 2017 at 12:00, Ruiyu Ni wrote: > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ruiyu Ni > Cc: Leif Lindholm > Cc: Ard Biesheuvel > --- > BeagleBoardPkg/BeagleBoardPkg.dsc | 17 +++++++++-------- > BeagleBoardPkg/BeagleBoardPkg.fdf | 3 ++- > 2 files changed, 11 insertions(+), 9 deletions(-) > > diff --git a/BeagleBoardPkg/BeagleBoardPkg.dsc b/BeagleBoardPkg/BeagleBoardPkg.dsc > index 4f7c0bd645..b6fdb90035 100644 > --- a/BeagleBoardPkg/BeagleBoardPkg.dsc > +++ b/BeagleBoardPkg/BeagleBoardPkg.dsc > @@ -2,7 +2,7 @@ > # Beagle board package. > # > # Copyright (c) 2009 - 2010, Apple Inc. All rights reserved.
> -# Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.
> +# Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved.
> # Copyright (c) 2016, Linaro Ltd. All rights reserved.
> # > # This program and the accompanying materials > @@ -355,11 +355,6 @@ [PcdsFixedAtBuild.common] > # OMAP Interrupt Controller > gEmbeddedTokenSpaceGuid.PcdInterruptBaseAddress|0x48200000 > > - # We want to use the Shell Libraries but don't want it to initialise > - # automatically. We initialise the libraries when the command is called by the > - # Shell. > - gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE > - > gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|10 > > # GUID of the UEFI Shell > @@ -487,6 +482,14 @@ [Components.common] > NULL|MdeModulePkg/Library/BootManagerUiLib/BootManagerUiLib.inf > NULL|MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootMaintenanceManagerUiLib.inf > } > + > + # > + # Shell > + # > + ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf { > + > + gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE > + } > ShellPkg/Application/Shell/Shell.inf { > > ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf > @@ -496,8 +499,6 @@ [Components.common] > NULL|ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.inf > NULL|ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.inf > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1CommandsLib.inf > - NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf Why is this line ^^^ being removed as well? > - NULL|ShellPkg/Library/UefiShellTftpCommandLib/UefiShellTftpCommandLib.inf > HandleParsingLib|ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.inf > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf > BcfgCommandLib|ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.inf > diff --git a/BeagleBoardPkg/BeagleBoardPkg.fdf b/BeagleBoardPkg/BeagleBoardPkg.fdf > index c9c6afd714..71249c7eb3 100644 > --- a/BeagleBoardPkg/BeagleBoardPkg.fdf > +++ b/BeagleBoardPkg/BeagleBoardPkg.fdf > @@ -1,7 +1,7 @@ > # FLASH layout file for Beagle board. > # > # Copyright (c) 2009, Apple Inc. All rights reserved.
> -# Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.
> +# Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved.
> # Copyright (c) 2016, Linaro, Ltd. All rights reserved.
> # > # This program and the accompanying materials > @@ -178,6 +178,7 @@ [FV.FvMain] > # UEFI application (Shell Embedded Boot Loader) > # > INF ShellPkg/Application/Shell/Shell.inf > + INF ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf > > # > # Bds > -- > 2.15.0.gvfs.1.preview.4 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel