[dpdk-dev] [PATCH] doc: fix required tools list layout
Baruch Siach
baruch at tkos.co.il
Sun Dec 18 20:11:00 CET 2016
Hi John,
On Thu, Dec 15, 2016 at 03:09:32PM +0000, Mcnamara, John wrote:
> > -----Original Message-----
> > From: Baruch Siach [mailto:baruch at tkos.co.il]
> > Sent: Tuesday, December 13, 2016 10:04 AM
> > To: dev at dpdk.org
> > Cc: Mcnamara, John <john.mcnamara at intel.com>; David Marchand
> > <david.marchand at 6wind.com>; Baruch Siach <baruch at tkos.co.il>
> > Subject: [PATCH] doc: fix required tools list layout
> >
> > The Python requirement should appear in the bullet list.
> >
> > Signed-off-by: Baruch Siach <baruch at tkos.co.il>
> > ---
> > doc/guides/linux_gsg/sys_reqs.rst | 4 +---
> > 1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/doc/guides/linux_gsg/sys_reqs.rst
> > b/doc/guides/linux_gsg/sys_reqs.rst
> > index 3d743421595a..621cc9ddaef6 100644
> > --- a/doc/guides/linux_gsg/sys_reqs.rst
> > +++ b/doc/guides/linux_gsg/sys_reqs.rst
> > @@ -84,9 +84,7 @@ Compilation of the DPDK
> > x86_x32 ABI is currently supported with distribution packages only on
> > Ubuntu
> > higher than 13.10 or recent Debian distribution. The only supported
> > compiler is gcc 4.9+.
> >
> > -.. note::
> > -
> > - Python, version 2.6 or 2.7, to use various helper scripts included in
> > the DPDK package.
> > +* Python, version 2.6 or 2.7, to use various helper scripts included in
> > the DPDK package.
>
> In addition to this change the note on the previous item should be indented
> to the level of the bullet item. It is probably worth making that change at
> the same time.
All items are equally aligned as far as I can see. The 32bit on 64bit
requirement bullets are sub-items of the previous item. Am I missing anything?
> Also, the Python version should probably say 2.7+ and 3.2+ if this patch is
> accepted:
>
> http://dpdk.org/dev/patchwork/patch/17775/
>
> However, since that change hasn't been acked/merged yet you can leave that
> part of your patch as it is and I'll fix the version numbers in the other
> patch.
Note that your updated patch[1] conflicts with this one.
[1] http://dpdk.org/dev/patchwork/patch/18152/
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
More information about the dev
mailing list