From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D0367A0A02; Wed, 24 Mar 2021 17:55:33 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 64916140F53; Wed, 24 Mar 2021 17:55:33 +0100 (CET) Received: from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com [66.111.4.29]) by mails.dpdk.org (Postfix) with ESMTP id 72D92140F52 for ; Wed, 24 Mar 2021 17:55:31 +0100 (CET) Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.nyi.internal (Postfix) with ESMTP id B0A5E5C0184; Wed, 24 Mar 2021 12:55:29 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Wed, 24 Mar 2021 12:55:29 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm3; bh= ElQr7lqffcffo+77qNPwsIjUnC5SStbTQ4Cqf3wxDFs=; b=KN9XMTH3F2etP1pn fOsfeq5REF1DZDV5bdQzlAS5daFeld+mdrTfsHBvslmVd5kAIUwxIsQoRzsvbjaL 59PzMPeS1HdrSlCBFJHMiR52veTrutcujmxiCVcPIBORTX2sKXL3sSXaiGpZGsZ2 JoXPSZyBWpDGe8QmJs82hWz7/vt18O9EOYpwFcDhBcRrrqtLMvVWYJs+NfnF3pDc KU8lAzeouFP4vRtM7qHSQkEFHweB2h9VctzxhjH2M4BhOgGC8Ft0IANTdW2QULC/ +hJEQ8e3tz51bh7X3aN3oYvP0c6wUO2qR5T0M/YGv46QRctdAJLJ60SdE4Yv1dYy uY4L2w== 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-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=ElQr7lqffcffo+77qNPwsIjUnC5SStbTQ4Cqf3wxD Fs=; b=Y8kHXtTx6HC1w57eOZHbOvaE1AgyU/4UY0O5CDsmv/XUs3pZyo+KTo+S6 UPO1h47seGr3RvBUE/lKQk3dHxiVE/NGR6mMRVbJo2jN6PlzS5QIi4CD689zVBQl CWMIoYzPOuPcHHAQT/6sCCaMVhI3jPoOu2981qlWsDGgo4Pm8OKRue/D2jgfxvt2 qnko/zI1BUL6e5bxKNYZg7TkfRRm7isyHy7olW2/dEmSBGYVMyRb4vLFytZV5ZHL r3UQQIkhwABTYyhChJqaF6rqFnTkAoyLbi2mWZ8quGlw8El4CBCVAR/kKAfn5Tjl HuhPoqwkfsVa4glvnj/A0KOJTcChg== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledrudegkedgledvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkfgjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepudeggfdvfeduffdtfeeglefghfeukefgfffhueejtdetuedtjeeu ieeivdffgeehnecukfhppeejjedrudefgedrvddtfedrudekgeenucevlhhushhtvghruf hiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghl ohhnrdhnvght X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 17D02108005C; Wed, 24 Mar 2021 12:55:29 -0400 (EDT) From: Thomas Monjalon To: David Marchand Cc: dev Date: Wed, 24 Mar 2021 17:55:27 +0100 Message-ID: <7341585.LLnYYtrRXG@thomas> In-Reply-To: References: <20210309233116.1934666-1-thomas@monjalon.net> <20210321223116.1340974-1-thomas@monjalon.net> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v4 00/11] improve options help X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 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" 24/03/2021 16:03, David Marchand: > The changes on usage() and the --log-level help are not related and > could have been separated in two series. Yes, I will split. It will help merging the non-controversial patches. > - The "Usage: " lists [EAL options] first, so I would expect them to > be listed first. > Hence I am not a fan of the reordering. If not reordering, we print the EAL options before giving the line "Usage: ..." with the generic syntax of the application. > - Testpmd options are not identified anymore while it was easier > before to find out about it with the "===== Application Usage =====" > banner. > Applications now must add a banner to differentiate their options from > the EAL ones. I think the banner "Application Usage" is strange when printing (sic) application usage. The EAL options are already clearly identified, so I though it was clear enough: EAL common options: [...] EAL options for DEBUG use only: [...] EAL Linux options: [...]