X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=recipes-connectivity%2Fkuksa-val%2Fkuksa-client%2F0001-kuksa-client-Update-cmd2-completer-usage.patch;h=3f55273842d611e8f0eae13841f5ed59c1cf78b8;hb=d9f9ddd177a57110d9c67ea438436c3160f8e429;hp=55e9cc2cbc9d2767d080596e9d1e8d669c8895f6;hpb=f6a2980917592000382d2a9af9ea4803e90f9f88;p=AGL%2Fmeta-agl-demo.git diff --git a/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch b/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch index 55e9cc2cb..3f5527384 100644 --- a/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch +++ b/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch @@ -1,7 +1,7 @@ -From ea0b48b9a900e9ce6d898fc5d17ae3238aadb289 Mon Sep 17 00:00:00 2001 +From acb2bbff3de6e9f0bafb8b96a339820a9ddfadca Mon Sep 17 00:00:00 2001 From: Scott Murray -Date: Tue, 2 May 2023 16:20:40 -0400 -Subject: [PATCH 2/2] kuksa_viss_client: Update cmd2 completer usage +Date: Mon, 27 Nov 2023 15:31:02 -0500 +Subject: [PATCH 1/2] kuksa-client: Update cmd2 completer usage Update cmd2 usage to avoid using internal methods and fix breakage with newer versions of the cmd2 module. @@ -10,31 +10,22 @@ Upstream-Status: pending Signed-off-by: Scott Murray --- - kuksa-client/kuksa_client/__main__.py | 31 +++++++++++++-------------- - 1 file changed, 15 insertions(+), 16 deletions(-) + kuksa-client/kuksa_client/__main__.py | 33 +++++++++++++-------------- + 1 file changed, 16 insertions(+), 17 deletions(-) diff --git a/kuksa-client/kuksa_client/__main__.py b/kuksa-client/kuksa_client/__main__.py -index c34f335..d386708 100755 +index 2b29623..7061ec1 100755 --- a/kuksa-client/kuksa_client/__main__.py +++ b/kuksa-client/kuksa_client/__main__.py -@@ -34,7 +34,6 @@ from cmd2 import Cmd - from cmd2 import CompletionItem +@@ -36,7 +36,6 @@ from cmd2 import CompletionItem from cmd2 import with_argparser from cmd2 import with_category + from cmd2 import constants -from cmd2.utils import basic_complete + from urllib.parse import urlparse - import kuksa_certificates from kuksa_client import KuksaClientThread -@@ -122,7 +121,7 @@ class TestClient(Cmd): - "Children of branch "+prefix+key), - ) - -- return basic_complete(text, line, begidx, endidx, self.pathCompletionItems) -+ return Cmd.basic_complete(self, text, line, begidx, endidx, self.pathCompletionItems) - - def subscribeCallback(self, logPath, resp): - with logPath.open('a', encoding='utf-8') as logFile: -@@ -132,7 +131,7 @@ class TestClient(Cmd): +@@ -151,7 +150,7 @@ class TestClient(Cmd): self.pathCompletionItems = [] for sub_id in self.subscribeIds: self.pathCompletionItems.append(CompletionItem(sub_id)) @@ -42,100 +33,129 @@ index c34f335..d386708 100755 + return Cmd.basic_complete(self, text, line, begidx, endidx, self.pathCompletionItems) COMM_SETUP_COMMANDS = "Communication Set-up Commands" - VSS_COMMANDS = "Kuksa Interaction Commands" -@@ -149,7 +148,7 @@ class TestClient(Cmd): + VSS_COMMANDS = "Kuksa Interaction Commands (Supported by both KUKSA Databroker and KUKSA Server)" +@@ -167,19 +166,19 @@ class TestClient(Cmd): + + ap_disconnect = argparse.ArgumentParser() + ap_authorize = argparse.ArgumentParser() +- tokenfile_completer_method = functools.partial( ++ tokenfile_completer = functools.partial( + Cmd.path_complete, + path_filter=lambda path: (os.path.isdir(path) or path.endswith(".token")), + ) ap_authorize.add_argument( - 'token_or_tokenfile', - help='JWT(or the file storing the token) for authorizing the client.', -- completer_method=tokenfile_completer_method,) -+ completer=tokenfile_completer_method,) - ap_setServerAddr = argparse.ArgumentParser() - ap_setServerAddr.add_argument( - 'IP', help='VISS/gRPC Server IP Address', default=DEFAULT_SERVER_ADDR) -@@ -164,7 +163,7 @@ class TestClient(Cmd): + "token_or_tokenfile", + help="JWT(or the file storing the token) for authorizing the client.", +- completer_method=tokenfile_completer_method, ++ completer=tokenfile_completer, + ) ap_setValue = argparse.ArgumentParser() ap_setValue.add_argument( -- "Path", help="Path to be set", completer_method=path_completer) -+ "Path", help="Path to be set", completer=path_completer) - ap_setValue.add_argument("Value", help="Value to be set") +- "Path", help="Path to be set", completer_method=path_completer ++ "Path", help="Path to be set", completer=path_completer + ) + ap_setValue.add_argument("Value", nargs="+", help="Value to be set") ap_setValue.add_argument( - "-a", "--attribute", help="Attribute to be set", default="value") -@@ -181,19 +180,19 @@ class TestClient(Cmd): +@@ -199,7 +198,7 @@ class TestClient(Cmd): ap_getValue = argparse.ArgumentParser() ap_getValue.add_argument( -- "Path", help="Path to be read", completer_method=path_completer) -+ "Path", help="Path to be read", completer=path_completer) +- "Path", help="Path to be read", completer_method=path_completer ++ "Path", help="Path to be read", completer=path_completer + ) ap_getValue.add_argument( - "-a", "--attribute", help="Attribute to be get", default="value") - - ap_getValues = argparse.ArgumentParser() - ap_getValues.add_argument( -- "Path", help="Path whose value is to be read", nargs='+', completer_method=path_completer) -+ "Path", help="Path whose value is to be read", nargs='+', completer=path_completer) + "-a", "--attribute", help="Attribute to be get", default="value" +@@ -210,7 +209,7 @@ class TestClient(Cmd): + "Path", + help="Path whose value is to be read", + nargs="+", +- completer_method=path_completer, ++ completer=path_completer, + ) ap_getValues.add_argument( - "-a", "--attribute", help="Attribute to be get", default="value") - - ap_setTargetValue = argparse.ArgumentParser() + "-a", "--attribute", help="Attribute to be get", default="value" +@@ -220,7 +219,7 @@ class TestClient(Cmd): ap_setTargetValue.add_argument( -- "Path", help="Path whose target value to be set", completer_method=path_completer) -+ "Path", help="Path whose target value to be set", completer=path_completer) + "Path", + help="Path whose target value to be set", +- completer_method=path_completer, ++ completer=path_completer, + ) ap_setTargetValue.add_argument("Value", help="Value to be set") - ap_setTargetValues = argparse.ArgumentParser() -@@ -206,35 +205,35 @@ class TestClient(Cmd): - - ap_getTargetValue = argparse.ArgumentParser() +@@ -236,7 +235,7 @@ class TestClient(Cmd): ap_getTargetValue.add_argument( -- "Path", help="Path whose target value is to be read", completer_method=path_completer) -+ "Path", help="Path whose target value is to be read", completer=path_completer) + "Path", + help="Path whose target value is to be read", +- completer_method=path_completer, ++ completer=path_completer, + ) ap_getTargetValues = argparse.ArgumentParser() - ap_getTargetValues.add_argument( -- "Path", help="Path whose target value is to be read", nargs='+', completer_method=path_completer) -+ "Path", help="Path whose target value is to be read", nargs='+', completer=path_completer) +@@ -244,12 +243,12 @@ class TestClient(Cmd): + "Path", + help="Path whose target value is to be read", + nargs="+", +- completer_method=path_completer, ++ completer=path_completer, + ) ap_subscribe = argparse.ArgumentParser() ap_subscribe.add_argument( -- "Path", help="Path to subscribe to", completer_method=path_completer) -+ "Path", help="Path to subscribe to", completer=path_completer) +- "Path", help="Path to subscribe to", completer_method=path_completer ++ "Path", help="Path to subscribe to", completer=path_completer + ) ap_subscribe.add_argument( - "-a", "--attribute", help="Attribute to subscribe to", default="value") + "-a", "--attribute", help="Attribute to subscribe to", default="value" +@@ -264,7 +263,7 @@ class TestClient(Cmd): ap_subscribeMultiple = argparse.ArgumentParser() ap_subscribeMultiple.add_argument( -- "Path", help="Path to subscribe to", nargs='+', completer_method=path_completer) -+ "Path", help="Path to subscribe to", nargs='+', completer=path_completer) +- "Path", help="Path to subscribe to", nargs="+", completer_method=path_completer ++ "Path", help="Path to subscribe to", nargs="+", completer=path_completer + ) ap_subscribeMultiple.add_argument( - "-a", "--attribute", help="Attribute to subscribe to", default="value") - - ap_unsubscribe = argparse.ArgumentParser() + "-a", "--attribute", help="Attribute to subscribe to", default="value" +@@ -280,18 +279,18 @@ class TestClient(Cmd): ap_unsubscribe.add_argument( -- "SubscribeId", help="Corresponding subscription Id", completer_method=subscriptionIdCompleter, -+ "SubscribeId", help="Corresponding subscription Id", completer=subscriptionIdCompleter, + "SubscribeId", + help="Corresponding subscription Id", +- completer_method=subscriptionIdCompleter, ++ completer=subscriptionIdCompleter, ) ap_getMetaData = argparse.ArgumentParser() ap_getMetaData.add_argument( -- "Path", help="Path whose metadata is to be read", completer_method=path_completer) -+ "Path", help="Path whose metadata is to be read", completer=path_completer) + "Path", + help="Path whose metadata is to be read", +- completer_method=path_completer, ++ completer=path_completer, + ) ap_updateMetaData = argparse.ArgumentParser() ap_updateMetaData.add_argument( -- "Path", help="Path whose MetaData is to update", completer_method=path_completer) -+ "Path", help="Path whose MetaData is to update", completer=path_completer) +- "Path", help="Path whose MetaData is to update", completer_method=path_completer ++ "Path", help="Path whose MetaData is to update", completer=path_completer + ) ap_updateMetaData.add_argument( "Json", - help="MetaData to update. Note, only attributes can be update, if update children or the whole vss tree, use" -@@ -245,7 +244,7 @@ class TestClient(Cmd): - jsonfile_completer_method = functools.partial(Cmd.path_complete, - path_filter=lambda path: (os.path.isdir(path) or path.endswith(".json"))) +@@ -300,14 +299,14 @@ class TestClient(Cmd): + ) + + ap_updateVSSTree = argparse.ArgumentParser() +- jsonfile_completer_method = functools.partial( ++ jsonfile_completer = functools.partial( + Cmd.path_complete, + path_filter=lambda path: (os.path.isdir(path) or path.endswith(".json")), + ) ap_updateVSSTree.add_argument( -- "Json", help="Json tree to update VSS", completer_method=jsonfile_completer_method) -+ "Json", help="Json tree to update VSS", completer=jsonfile_completer_method) + "Json", + help="Json tree to update VSS", +- completer_method=jsonfile_completer_method, ++ completer=jsonfile_completer, + ) - # Constructor - def __init__(self, server_ip=None, server_port=None, server_protocol=None, insecure=False, token_or_tokenfile=None): + # Constructor, request names after protocol to avoid errors -- -2.39.2 +2.44.0