From nobody Tue May 7 04:00:10 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+42521+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+42521+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1560822225; cv=none; d=zoho.com; s=zohoarc; b=HQnup9Cqs8DZobODKVuE5J+4nRiIdWySCVvTQoOFXdpERssr13/cblUobZhC6sIVqfmGvDRcJpa43kCsFEMt4U8OSBhds4FRFouWnbH+ke9xS1m6AhCic3fh3kAmYGTTrGLS7rbTHRic/XD0psuxsmTp3AGIDQ2ZyY3W/wh0vxs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1560822225; h=Cc:Date:From:List-Id:List-Unsubscribe:Message-ID:Reply-To:Sender:Subject:To:ARC-Authentication-Results; bh=kN/qrbkSu0sVHHG3gq04dgBwRa+m/mw5EwlvjA8ixF0=; b=JOvcvLRE18F18w9QAwJ+BRMAPUilsZHnAFXWKMD369Wsk+xPxMCmF1C1wA6mv+HhGFKvPhzfLRXy5GetWMJLYA2VdvESAypOVCOXzIJDmgq71m0uy5Q0/OE8QkWCJaDNA1tbclh+BM7tJ32gnS1Cd0yn9/+KKup+XkDJlPAVS3k= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+42521+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1560822225123945.8724499381553; Mon, 17 Jun 2019 18:43:45 -0700 (PDT) Return-Path: X-Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by groups.io with SMTP; Mon, 17 Jun 2019 18:43:44 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 17 Jun 2019 18:43:43 -0700 X-ExtLoop1: 1 X-Received: from jshi19-mobl.ccr.corp.intel.com ([10.254.212.155]) by orsmga002.jf.intel.com with ESMTP; 17 Jun 2019 18:43:42 -0700 From: "Steven Shi" To: devel@edk2.groups.io Cc: liming.gao@intel.com, bob.c.feng@intel.com, christian.rodriguez@intel.com Subject: [edk2-devel] [PATCH] BaseTools:Change build cache option name from --binary- to --cache- Date: Tue, 18 Jun 2019 09:43:32 +0800 Message-Id: <20190618014332.10456-1-steven.shi@intel.com> Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,steven.shi@intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1560822224; bh=PbAlZjT9Ib+SvPlewhnFnlXHSZFcerS9GQf1ebHw2Lo=; h=Cc:Date:From:Reply-To:Subject:To; b=YPfxk+UgJx4GRwmuUGjRueGg0r/zC+gjkGHBVQNc0wlmgh3qK/cJShmWFH0/cMwPSqz kOs9iGiF7llZS1wgTsewL63LzM0wI/r8DnBv7GSclCHpM1J6R2pZ6+7x36Vy7uWWrZSFc FvOtQwyHhnDL8nfyuMM6klJLtEEiokZtOBE= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" https://bugzilla.tianocore.org/show_bug.cgi?id=3D1896 Current build cache is enabled by the below two options: --binary-destination, which is to produce the cache. --binary-source, which is to consume the cache. Since current build cache does not only support to store the binary file (e.g. .efi file) but also support the library and other type files (.lib, .pdb, .inf etc), so the prefix "--binary-" will confuse user with misunderstanding that the build cache will only support binary file. This patch is to change the build cache option prefix name from --binary- to --cache- like below: --cache-destination --cache-source --- BaseTools/Source/Python/build/build.py | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Pyth= on/build/build.py index 2dca3c7b34..bc90a34bf5 100644 --- a/BaseTools/Source/Python/build/build.py +++ b/BaseTools/Source/Python/build/build.py @@ -731,13 +731,13 @@ class Build(): GlobalData.gDisableIncludePathCheck =3D BuildOptions.DisableInclud= ePathCheck =20 if GlobalData.gBinCacheDest and not GlobalData.gUseHashCache: - EdkLogger.error("build", OPTION_NOT_SUPPORTED, ExtraData=3D"--= binary-destination must be used together with --hash.") + EdkLogger.error("build", OPTION_NOT_SUPPORTED, ExtraData=3D"--= cache-destination must be used together with --hash.") =20 if GlobalData.gBinCacheSource and not GlobalData.gUseHashCache: - EdkLogger.error("build", OPTION_NOT_SUPPORTED, ExtraData=3D"--= binary-source must be used together with --hash.") + EdkLogger.error("build", OPTION_NOT_SUPPORTED, ExtraData=3D"--= cache-source must be used together with --hash.") =20 if GlobalData.gBinCacheDest and GlobalData.gBinCacheSource: - EdkLogger.error("build", OPTION_NOT_SUPPORTED, ExtraData=3D"--= binary-destination can not be used together with --binary-source.") + EdkLogger.error("build", OPTION_NOT_SUPPORTED, ExtraData=3D"--= cache-destination can not be used together with --cache-source.") =20 if GlobalData.gBinCacheSource: BinCacheSource =3D os.path.normpath(GlobalData.gBinCacheSource) @@ -746,7 +746,7 @@ class Build(): GlobalData.gBinCacheSource =3D BinCacheSource else: if GlobalData.gBinCacheSource is not None: - EdkLogger.error("build", OPTION_VALUE_INVALID, ExtraData= =3D"Invalid value of option --binary-source.") + EdkLogger.error("build", OPTION_VALUE_INVALID, ExtraData= =3D"Invalid value of option --cache-source.") =20 if GlobalData.gBinCacheDest: BinCacheDest =3D os.path.normpath(GlobalData.gBinCacheDest) @@ -755,7 +755,7 @@ class Build(): GlobalData.gBinCacheDest =3D BinCacheDest else: if GlobalData.gBinCacheDest is not None: - EdkLogger.error("build", OPTION_VALUE_INVALID, ExtraData= =3D"Invalid value of option --binary-destination.") + EdkLogger.error("build", OPTION_VALUE_INVALID, ExtraData= =3D"Invalid value of option --cache-destination.") =20 if self.ConfDirectory: # Get alternate Conf location, if it is absolute, then just us= e the absolute directory name @@ -2334,15 +2334,14 @@ def MyOptionParser(): help=3D"Specify the specific option to parse EDK UNI file. Must be= one of: [-c, -s]. -c is for EDK framework UNI file, and -s is for EDK UEFI= UNI file. "\ "This option can also be specified by setting *_*_*_BUILD_FLA= GS in [BuildOptions] section of platform DSC. If they are both specified, t= his value "\ "will override the setting in [BuildOptions] section of platf= orm DSC.") - Parser.add_option("-N", "--no-cache", action=3D"store_true", dest=3D"D= isableCache", default=3DFalse, help=3D"Disable build cache mechanism") Parser.add_option("--conf", action=3D"store", type=3D"string", dest=3D= "ConfDirectory", help=3D"Specify the customized Conf directory.") Parser.add_option("--check-usage", action=3D"store_true", dest=3D"Chec= kUsage", default=3DFalse, help=3D"Check usage content of entries listed in = INF file.") Parser.add_option("--ignore-sources", action=3D"store_true", dest=3D"I= gnoreSources", default=3DFalse, help=3D"Focus to a binary build and ignore = all source files") Parser.add_option("--pcd", action=3D"append", dest=3D"OptionPcd", help= =3D"Set PCD value by command line. Format: \"PcdName=3DValue\" ") Parser.add_option("-l", "--cmd-len", action=3D"store", type=3D"int", d= est=3D"CommandLength", help=3D"Specify the maximum line length of build com= mand. Default is 4096.") Parser.add_option("--hash", action=3D"store_true", dest=3D"UseHashCach= e", default=3DFalse, help=3D"Enable hash-based caching during build process= .") - Parser.add_option("--binary-destination", action=3D"store", type=3D"st= ring", dest=3D"BinCacheDest", help=3D"Generate a cache of binary files in t= he specified directory.") - Parser.add_option("--binary-source", action=3D"store", type=3D"string"= , dest=3D"BinCacheSource", help=3D"Consume a cache of binary files from the= specified directory.") + Parser.add_option("--cache-destination", action=3D"store", type=3D"str= ing", dest=3D"BinCacheDest", help=3D"Generate build cache files in the spec= ified directory.") + Parser.add_option("--cache-source", action=3D"store", type=3D"string",= dest=3D"BinCacheSource", help=3D"Consume build cache files from the specif= ied directory.") Parser.add_option("--genfds-multi-thread", action=3D"store_true", dest= =3D"GenfdsMultiThread", default=3DFalse, help=3D"Enable GenFds multi thread= to generate ffs file.") Parser.add_option("--disable-include-path-check", action=3D"store_true= ", dest=3D"DisableIncludePathCheck", default=3DFalse, help=3D"Disable the i= nclude path check for outside of package.") (Opt, Args) =3D Parser.parse_args() --=20 2.17.1.windows.2 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#42521): https://edk2.groups.io/g/devel/message/42521 Mute This Topic: https://groups.io/mt/32102383/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-