> -----Original Message----- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bernard Iremonger > Sent: Wednesday, December 9, 2015 12:36 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 1/1] doc: correct Vhost Sample Application > guide > > correct sample console commands > > Fixes: d0dff9ba445e ("doc: sample application user guide" > Fixes: 9bc23cb8209c ("doc: add vhost-user to sample guide") > Fixes: 43866bf71d58 ("doc: fix vhost sample parameter") > Signed-off-by: Bernard Iremonger <bernard.iremonger at intel.com> > --- > doc/guides/sample_app_ug/vhost.rst | 40 +++++++++++++++++++-------------- > > ... > > Redistribution and use in source and binary forms, with or without @@ > -386,13 +386,13 @@ Running the Sample Code > > .. code-block:: console > > - user at target:~$ ./build/app/vhost-switch -c f -n 4 --huge-dir /
Also, as a general note, not just to this document/patch. I don't think the "user at 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 at 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? John