[dpdk-web] [PATCH] release 16.07.1

Van Haaren, Harry harry.van.haaren at intel.com
Wed Oct 26 10:58:11 CEST 2016


Hi,

Looks ok, I have a few comments:

> -----Original Message-----
> Another note: there is a <p> decorator after the "latest stalbe
> release" icon, because in my local testing, it can't show 4
> icons in one row. Thus I splitted it them in two rows, each with
> 2 icons.

I'm ok with the split into 2x2 grid, centering the buttons makes the page look better IMO.
I can patch after this is applied.

> -			<td><a href="http://fast.dpdk.org/rel/dpdk-16.07.tar.xz">DPDK 16.07</a></td>
> -			<td><a href="/doc/guides/rel_notes/release_16_07.html">2016 July 28</a></td>

Looks like release notes date is being updated, but it still points to the same link?

> -			<td><pre>690a2bb570103e58d12f9806e8bf21be</pre></td>
> +			<td><a href="http://fast.dpdk.org/rel/dpdk-16.07.1.tar.xz">DPDK 16.07.1</a></td>

It is not clear that this is the stable branch. Would it make more sense to rename to "DPDK Stable 16.07.1" or similar to make it more obvious this release has backports/fixes?

If there's no concerns about naming or release-notes link, please apply patch as is and I'll push a new patch to center the buttons on the download page.

Regards, -Harry


More information about the web mailing list