On 9/18/2020 9:33 AM, Bruce Richardson wrote:
On Thu, Sep 17, 2020 at 10:05:48AM -0700, Stephen Hemminger wrote:
On Thu, 17 Sep 2020 15:34:43 +0200
Olivier Matz <olivier.m...@6wind.com> wrote:
Hi Dmitry,
On Fri, Jul 31, 2020 at 12:06:45AM +0300, Dmitry Kozlyuk wrote:
struct cmdline exposes platform-specific members it contains, most
notably struct termios that is only available on Unix. Make the
structure opaque.
Remove tests checking struct cmdline content as meaningless.
Add cmdline_get_rdline() to access history buffer.
The new function is currently used only in tests.
Signed-off-by: Dmitry Kozlyuk <dmitry.kozl...@gmail.com>
First, please forgive me for the very late feedback. It is all the more
problematic because I think this patch introduces an ABI breakage, that
should have been announced.
Since 20.11 is a API/ABI breaking release, I think breaking ABI
is okay without announcement. What matters more is if that API would
need to be impacted. API changes need some announcement.
This is something that we need to get a clear decision from technical board
on, I think, since there are some other proposed ABI changes in patches
that were not pre-announced, e.g. changing the lpm structure.
And we accepted another in ethdev one, but that library already has
bunch of deprecation notices on it and one more change won't has much
affect.
Overall looks OK to me to accept minor changes without deprecation
notices, big ones can be investigated case by case, and +1 to have some
guidance from the techboard.
