From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id 3DF331B2DF for ; Thu, 16 Nov 2017 19:05:05 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 8788B20AEB; Thu, 16 Nov 2017 13:05:04 -0500 (EST) Received: from frontend1 ([10.202.2.160]) by compute1.internal (MEProxy); Thu, 16 Nov 2017 13:05:04 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc; s=mesmtp; bh=bPDzu63zUk7UifSzs8X40FwmSW +xM8h7YBWsGXkKWw4=; b=JsRvApyTDmtcrLMJFJLnFv1Zq+lME88uOCUsReXPv4 lqBlbiwCRJcaL3NzvAE8WgBKY4vj/ZTJYO0/hZ4YAHq3BVjPMTGmgOhpn3r/3Er/ 0E4r4ePvTEMInbcerVRPRLDFKX6CoaL888QkIbaiUxNZQ92bo38G5aONL9uF78NR M= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=bPDzu6 3zUk7UifSzs8X40FwmSW+xM8h7YBWsGXkKWw4=; b=LQqLgc8dhiHlUw4ub1cqEi UGFYVCoSi11F6YLM+p2fBIB9onxgMLmmYagaV6UX5c9XS+KJfI5Ym6cQ085graJa /9/OqY8bxLkVwZpzBFJunv5R4iuX1KU1wgOPfgEw6ZMsCINdC8UDDa0qeaRkV22F sRKvNqb47S6xgjdPNqe86VrJt+OYrbehkOwpAms0MCqq7CptV4eWfRbxexI8PSPd gcP+ItM60CFxyUxTHytup4pBlyhn8hdaQiKhsK0Jyciv6A6rvql9QavjUKCfICAL rkSo9/0DRW98yjBC7gbOEnrYrpoYJAKN9wCfEW5sNMxxmo3lWg6L8YasglJPBYwQ == X-ME-Sender: Received: from xps.localnet (unknown [64.124.208.80]) by mail.messagingengine.com (Postfix) with ESMTPA id 31F007E3E0; Thu, 16 Nov 2017 13:05:04 -0500 (EST) From: Thomas Monjalon To: "Wiles, Keith" Cc: "Yigit, Ferruh" , Adrien Mazarguil , Olivier Matz , "Wu, Jingjing" , dev@dpdk.org Date: Thu, 16 Nov 2017 19:05:03 +0100 Message-ID: <2113924.NBhQAuUZo3@xps> In-Reply-To: <966D87A8-0B36-4B01-862C-EC9DB55667B3@intel.com> References: <1510234868-31053-1-git-send-email-adrien.mazarguil@6wind.com> <6df8c8c1-325c-e924-94bf-9fced812ecf9@intel.com> <966D87A8-0B36-4B01-862C-EC9DB55667B3@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [RFC] cmdline: rework as a wrapper to libedit 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: , X-List-Received-Date: Thu, 16 Nov 2017 18:05:05 -0000 16/11/2017 18:27, Wiles, Keith: > > > On Nov 16, 2017, at 9:06 AM, Yigit, Ferruh wrote: > > > > On 11/16/2017 1:23 AM, Adrien Mazarguil wrote: > > <...> > > > >>> I do not agree it has severed DPDK well, just look at test-pmd and the hoops people have to jump thru to get a new command or variation of an existing command integrated into test-pmd it is very difficult. Also if you look at the command sets in test-pmd they are very odd in that similar commands can some times be set up completely different as cmdline is too rigid and difficult to use. > >> > >> Testpmd is indeed messy, but this is not librte_cmdline's fundamental fault > >> in my opinion, more likely the result of using a copy/paste approach to new > >> commands due to lack of time or interest in making things nicer than the > >> bare minimum to validate features. There's no design direction for it hence > >> the lack of uniformity in the command hierarchy. > > > > Unrelated to cmdline discussion +1 for this paragraph. > > As a side note should CLI change the test-pmd commands to be more reasonable or maintain the current commands to not break testing. I would like to fix these commands with CLI, but as I have both built into test-pmd currently we can could keep the old cmdline commands and if you use the new CLI then it would be the new cleaner commands. I think we must keep the existing testpmd commands for a while to not disturb people using it integrated in their testing tools. If we introduce a new set of commands, we can define a deadline for removal of old commands.