From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 5635A8D96 for ; Thu, 10 Dec 2015 07:39:28 +0100 (CET) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 09 Dec 2015 22:39:27 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,407,1444719600"; d="scan'208";a="870636589" Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.66.49]) by fmsmga002.fm.intel.com with ESMTP; 09 Dec 2015 22:39:27 -0800 Date: Thu, 10 Dec 2015 14:40:46 +0800 From: Yuanhan Liu To: "Iremonger, Bernard" Message-ID: <20151210064046.GS29571@yliu-dev.sh.intel.com> References: <1449664541-9546-1-git-send-email-bernard.iremonger@intel.com> <3756278.RbTW4O93Bj@xps13> <8CEF83825BEC744B83065625E567D7C219F8E971@IRSMSX108.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8CEF83825BEC744B83065625E567D7C219F8E971@IRSMSX108.ger.corp.intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH 1/1] doc: correct Vhost Sample Application guide X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 10 Dec 2015 06:39:28 -0000 On Wed, Dec 09, 2015 at 04:14:01PM +0000, Iremonger, Bernard wrote: > Hi John, > > > > 2015-12-09 14:50, Mcnamara, John: > > > Also, as a general note, not just to this document/patch. > > > > > > I don't think the "user@target:~$" prefixes used in some DPDK Doc > > > commandlines is useful. > > > > > > It prevents a straight copy and paste for testing, it makes the > > > literal block lines longer than they should be, it isn't used > > > consistently everywhere, it is visually distracting (in some cases it > > > is longer than the command being shown), and it isn't always correct > > > (I presume in the above case of "user@target" you would need run the > > > application as sudo if you are a non root user). > > > > > > Any objections to removing these as documents are updated? > > > > Generally speaking, +1 to remove extra bytes :) > > I will remove the "user@target:~$" prefixes in this document. I'd personally suggest to keep the "$ " prefix, or "# " if it needs root privilege. Which is a convention to paste commands. --yliu