From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mx.groups.io with SMTP id smtpd.web10.4738.1589502602305913889 for ; Thu, 14 May 2020 17:30:02 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.115, mailfrom: ashley.e.desimone@intel.com) IronPort-SDR: 4DgQH35puIGRST6jSmDQz0hkyE3GIaYY9GnusOGteWNCzvdbG05LS/gem3SeljOyAkb1TKwe38 hdHudpayITaA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 14 May 2020 17:30:01 -0700 IronPort-SDR: st0WQDqkX6nqB/CxlFzTz07aowPujqOMdDjQK7jLSezVKcBEDHO/SGenscAm373X+E6qm31ml/ Wu0vkDNnbeag== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,392,1583222400"; d="scan'208";a="438115436" Received: from aedesimo-desk.amr.corp.intel.com ([10.212.39.189]) by orsmga005.jf.intel.com with ESMTP; 14 May 2020 17:30:01 -0700 From: "Ashley E Desimone" To: devel@edk2.groups.io Cc: Nate DeSimone , Puja Pandya , Erik Bjorge , Bret Barkelew , Prince Agyeman Subject: [edk2-staging/EdkRepo] [PATCH V2] EdkRepo: Only process the checkout command if a valid combo is used Date: Thu, 14 May 2020 17:29:58 -0700 Message-Id: <20200515002958.12432-1-ashley.e.desimone@intel.com> X-Mailer: git-send-email 2.16.2.windows.1 Signed-off-by: Ashley E Desimone Cc: Nate DeSimone Cc: Puja Pandya Cc: Erik Bjorge Cc: Bret Barkelew Cc: Prince Agyeman --- edkrepo/commands/checkout_command.py | 20 +++++++++++++------- edkrepo/commands/humble/checkout_humble.py | 10 ++++++++++ 2 files changed, 23 insertions(+), 7 deletions(-) create mode 100644 edkrepo/commands/humble/checkout_humble.py diff --git a/edkrepo/commands/checkout_command.py b/edkrepo/commands/checkout_command.py index 7d65eb8..abea6a5 100644 --- a/edkrepo/commands/checkout_command.py +++ b/edkrepo/commands/checkout_command.py @@ -3,7 +3,7 @@ ## @file # checkout_command.py # -# Copyright (c) 2017- 2019, Intel Corporation. All rights reserved.
+# Copyright (c) 2017- 2020, Intel Corporation. All rights reserved.
# SPDX-License-Identifier: BSD-2-Clause-Patent # # Standard modules @@ -14,8 +14,11 @@ import os # Our modules from edkrepo.commands.edkrepo_command import EdkrepoCommand, OverrideArgument -import edkrepo.commands.arguments.checkout_args as arguments -from edkrepo.common.common_repo_functions import checkout +import edkrepo.commands.arguments.checkout_args as arguments +import edkrepo.commands.humble.checkout_humble as humble +from edkrepo.common.common_repo_functions import checkout, combination_is_in_manifest +from edkrepo.common.edkrepo_exception import EdkrepoInvalidParametersException +from edkrepo.config.config_factory import get_workspace_manifest class CheckoutCommand(EdkrepoCommand): @@ -25,17 +28,20 @@ class CheckoutCommand(EdkrepoCommand): def get_metadata(self): metadata = {} metadata['name'] = 'checkout' - metadata['help-text'] = arguments.COMMAND_DESCRIPTION + metadata['help-text'] = arguments.COMMAND_DESCRIPTION args = [] metadata['arguments'] = args args.append({'name' : 'Combination', 'positional' : True, 'position' : 0, 'required': True, - 'description' : arguments.COMBINATION_DESCRIPTION, - 'help-text' : arguments.COMBINATION_HELP}) + 'description' : arguments.COMBINATION_DESCRIPTION, + 'help-text' : arguments.COMBINATION_HELP}) args.append(OverrideArgument) return metadata def run_command(self, args, config): - checkout(args.Combination, args.verbose, args.override) + if combination_is_in_manifest(args.Combination, get_workspace_manifest()): + checkout(args.Combination, args.verbose, args.override) + else: + raise EdkrepoInvalidParametersException(humble.NO_COMBO) diff --git a/edkrepo/commands/humble/checkout_humble.py b/edkrepo/commands/humble/checkout_humble.py new file mode 100644 index 0000000..a1ff1a3 --- /dev/null +++ b/edkrepo/commands/humble/checkout_humble.py @@ -0,0 +1,10 @@ +#!/usr/bin/env python3 +# +## @file +# checkout_humble.py +# +# Copyright (c) 2020, Intel Corporation. All rights reserved.
+# SPDX-License-Identifier: BSD-2-Clause-Patent +# + +NO_COMBO = 'A combination named: {} does not exist in the workspace manifest' -- 2.16.2.windows.1