From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id D078AA046B for ; Fri, 26 Jul 2019 15:41:26 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id A79791BE07; Fri, 26 Jul 2019 15:41:26 +0200 (CEST) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id E30A24CAF for ; Fri, 26 Jul 2019 15:41:24 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 26 Jul 2019 06:41:23 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,311,1559545200"; d="scan'208";a="181865211" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga002.jf.intel.com with ESMTP; 26 Jul 2019 06:41:23 -0700 Received: from fmsmsx113.amr.corp.intel.com (10.18.116.7) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 26 Jul 2019 06:41:23 -0700 Received: from bgsmsx151.gar.corp.intel.com (10.224.48.42) by FMSMSX113.amr.corp.intel.com (10.18.116.7) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 26 Jul 2019 06:41:22 -0700 Received: from bgsmsx105.gar.corp.intel.com ([169.254.3.139]) by BGSMSX151.gar.corp.intel.com ([169.254.3.157]) with mapi id 14.03.0439.000; Fri, 26 Jul 2019 19:11:20 +0530 From: "Babu Radhakrishnan, AgalyaX" To: "Dumitrescu, Cristian" , "dev@dpdk.org" CC: "Pattan, Reshma" , "Byrne, Stephen1" , "Singh, Jasvinder" , "Parthasarathy, JananeeX M" , "Srinivasan, GuruX" Thread-Topic: [PATCH] examples: modify error message for ip pipeline Thread-Index: AQHVIrpZeHZnZKa6y06eZd2rMNtp8abB0GyAgBtXQlA= Date: Fri, 26 Jul 2019 13:41:18 +0000 Message-ID: <27BEB63E27FEB2409936481F1E06A394D45E42@BGSMSX105.gar.corp.intel.com> References: <1560521132-12232-1-git-send-email-agalyax.babu.radhakrishnan@intel.com> <3EB4FA525960D640B5BDFFD6A3D891268E8EBD48@IRSMSX108.ger.corp.intel.com> In-Reply-To: <3EB4FA525960D640B5BDFFD6A3D891268E8EBD48@IRSMSX108.ger.corp.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [10.223.10.10] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] examples: modify error message for ip pipeline X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, > -----Original Message----- > From: Dumitrescu, Cristian > Sent: Tuesday, July 9, 2019 3:01 PM > To: Babu Radhakrishnan, AgalyaX ; > dev@dpdk.org > Cc: Pattan, Reshma > Subject: RE: [PATCH] examples: modify error message for ip pipeline >=20 >=20 >=20 > > -----Original Message----- > > From: Babu Radhakrishnan, AgalyaX > > Sent: Friday, June 14, 2019 3:06 PM > > To: dev@dpdk.org > > Cc: Pattan, Reshma ; Dumitrescu, Cristian > > ; Babu Radhakrishnan, AgalyaX > > > > Subject: [PATCH] examples: modify error message for ip pipeline > > > > From: Agalya Babu RadhaKrishnan > > > > > Added help command in error message for ip pipeline commands. > > > > Signed-off-by: Agalya Babu RadhaKrishnan > > > > --- > > examples/ip_pipeline/cli.c | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/examples/ip_pipeline/cli.c b/examples/ip_pipeline/cli.c > > index 309b2936e..8a651bbbc 100644 > > --- a/examples/ip_pipeline/cli.c > > +++ b/examples/ip_pipeline/cli.c > > @@ -30,12 +30,12 @@ > > > > #define MSG_OUT_OF_MEMORY "Not enough memory.\n" > > #define MSG_CMD_UNKNOWN "Unknown command \"%s\".\n" > > -#define MSG_CMD_UNIMPLEM "Command \"%s\" not > implemented.\n" > > -#define MSG_ARG_NOT_ENOUGH "Not enough arguments for command > > \"%s\".\n" > > -#define MSG_ARG_TOO_MANY "Too many arguments for command > > \"%s\".\n" > > -#define MSG_ARG_MISMATCH "Wrong number of arguments for > > command \"%s\".\n" > > -#define MSG_ARG_NOT_FOUND "Argument \"%s\" not found.\n" > > -#define MSG_ARG_INVALID "Invalid value for argument \"%s\".\n" > > +#define MSG_CMD_UNIMPLEM "Command \"%s\" not implemented. > Try > > help \n" > > +#define MSG_ARG_NOT_ENOUGH "Not enough arguments for > command > > \"%s\". Try help \n" > > +#define MSG_ARG_TOO_MANY "Too many arguments for command > > \"%s\". Try help \n" > > +#define MSG_ARG_MISMATCH "Wrong number of arguments for > > command \"%s\". Try help \n" > > +#define MSG_ARG_NOT_FOUND "Argument \"%s\" not found. Try help > > \n" > > +#define MSG_ARG_INVALID "Invalid value for argument \"%s\". Try > help > > \n" > > #define MSG_FILE_ERR "Error in file \"%s\" at line %u.\n" > > #define MSG_FILE_NOT_ENOUGH "Not enough rules in file \"%s\".\n" > > #define MSG_CMD_FAIL "Command \"%s\" failed.\n" > > -- > > 2.14.1 >=20 > NAK >=20 > I don't see the value to add "try help command" on all the error messages= , > the user is already aware that there is a help command available, and som= e > people might actually find this repetition annoying. Okay, we would like to share our observations and views also on this. As the info on help command is not available in doc/nor in command line, us= er might have to explore its right usage for ip_pipeline. Following are the possible way of help command: --help, - help, help followed by command So ,Providing correct syntax for searching the command details in console w= ould be more clear from user perspective. Hence "try help command" is provided whenever a command fails. Kindly let us know if you still feel it is not appropriate. Regards, Agalya B