Reviewed-by: Chasel Chiu <[email protected]>
> -----Original Message----- > From: Fan, ZhijuX > Sent: Monday, June 17, 2019 1:11 PM > To: [email protected] > Cc: Gao, Liming <[email protected]>; Feng, Bob C <[email protected]>; > Shi, Steven <[email protected]>; Lu, Shifei A <[email protected]>; > Zhou, > Bowen <[email protected]>; Oram, Isaac W <[email protected]>; > Chiu, Chasel <[email protected]>; Kubacki, Michael A > <[email protected]>; Desimone, Nathaniel L > <[email protected]> > Subject: [PATCH 2/3 V2] Platform/Intel:Add build parameter to support Binary > Cache > > Need extend the options in the Intel/build_bios.py file to support Binary > Cache. > > --hash: > Enable hash-based caching during build process. > --binary-destination: > Generate a cache of binary files in the specified directory. > --binary-source: > Consume a cache of binary files from the specified directory. > > Cc: Liming Gao <[email protected]> > Cc: Bob Feng <[email protected]> > Cc: Steven Shi <[email protected]> > Cc: Shifei A Lu <[email protected]> > Cc: Xiaohu Zhou <[email protected]> > Cc: Isaac W Oram <[email protected]> > Cc: Chasel Chiu <[email protected]> > Cc: Michael Kubacki <[email protected]> > Cc: Nate DeSimone <[email protected]> > Signed-off-by: Zhiju.Fan <[email protected]> > --- > Platform/Intel/build_bios.py | 27 +++++++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > > diff --git a/Platform/Intel/build_bios.py b/Platform/Intel/build_bios.py index > 09eceddeff..c01b953d16 100644 > --- a/Platform/Intel/build_bios.py > +++ b/Platform/Intel/build_bios.py > @@ -343,6 +343,7 @@ def build(config): > print(" SILENT_MODE = ", config.get("SILENT_MODE")) > print(" REBUILD_MODE = ", config.get("REBUILD_MODE")) > print(" BUILD_ROM_ONLY = ", config.get("BUILD_ROM_ONLY")) > + print(" BINARY_CACHE_CMD_LINE = ", config.get("HASH"), > + config.get("BINARY_CACHE_CMD_LINE")) > print("==========================================") > > command = ["build", "-n", config["NUMBER_OF_PROCESSORS"]] @@ > -353,6 +354,10 @@ def build(config): > if config["EXT_BUILD_FLAGS"] and config["EXT_BUILD_FLAGS"] != "": > command.append(config["EXT_BUILD_FLAGS"]) > > + if config.get('BINARY_CACHE_CMD_LINE'): > + command.append(config['HASH']) > + command.append(config['BINARY_CACHE_CMD_LINE']) > + > if config.get("SILENT_MODE", "FALSE") == "TRUE": > command.append("--silent") > command.append("--quiet") > @@ -858,6 +863,17 @@ def get_cmd_config_arguments(arguments): > if arguments.fspapi is True: > result["API_MODE_FSP_WRAPPER_BUILD"] = "TRUE" > > + if not arguments.UseHashCache: > + result['BINARY_CACHE_CMD_LINE'] = '' > + elif arguments.BinCacheDest: > + result['HASH'] = '--hash' > + result['BINARY_CACHE_CMD_LINE'] = '--binary-destination=%s' % > arguments.BinCacheDest > + elif arguments.BinCacheSource: > + result['HASH'] = '--hash' > + result['BINARY_CACHE_CMD_LINE'] = '--binary-source=%s' % > arguments.BinCacheSource > + else: > + result['BINARY_CACHE_CMD_LINE'] = '' > + > return result > > > @@ -934,6 +950,17 @@ def get_cmd_arguments(build_config): > parser.add_argument("--fspapi", help="API mode fsp wrapper build > enabled", > action='store_true', dest="fspapi") > > + parser.add_argument("--hash", action="store_true", > dest="UseHashCache", default=False, > + help="Enable hash-based caching during build > + process.") > + > + parser.add_argument("--binary-destination", help="Generate a cache of > binary \ > + files in the specified directory.", > + action='store', dest="BinCacheDest") > + > + parser.add_argument("--binary-source", help="Consume a cache of binary > files \ > + from the specified directory.", > + action='store', dest="BinCacheSource") > + > return parser.parse_args() > > > -- > 2.14.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#42486): https://edk2.groups.io/g/devel/message/42486 Mute This Topic: https://groups.io/mt/32091878/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
