[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAD4GDZwF-ubL+enDVHOBm43CDPgovNHPBycjcNU6t-uADvkxAA@mail.gmail.com>
Date: Mon, 6 Jan 2025 17:33:20 +0000
From: Donald Hunter <donald.hunter@...il.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: davem@...emloft.net, netdev@...r.kernel.org, edumazet@...gle.com,
pabeni@...hat.com
Subject: Re: [PATCH net-next 1/3] tools: ynl: correctly handle overrides of
fields in subset
On Mon, 6 Jan 2025 at 15:36, Jakub Kicinski <kuba@...nel.org> wrote:
>
> On Mon, 06 Jan 2025 13:27:49 +0000 Donald Hunter wrote:
> > > We stated in documentation [1] and previous discussions [2]
> > > that the need for overriding fields in members of subsets
> > > is anticipated. Implement it.
> > >
> > > [1] https://docs.kernel.org/next/userspace-api/netlink/specs.html#subset-of
> > > [2] https://lore.kernel.org/netdev/20231004171350.1f59cd1d@kernel.org/
> > >
> > > Signed-off-by: Jakub Kicinski <kuba@...nel.org>
> >
> > I guess we're okay with requiring Python >= 3.9 for combining
> > dicts with |
>
> Ah, I didn't realize. Does YNL work on older versions today?
> I thought we already narrowed down to 3.9+. That may have
> been tests not YNL itself.
You're right, we may already be committed to 3.9+
> The "oldest" OS I have is CentOS 9(-derived) and has 3.9,
> so from my selfish perspective 3.9+ is perfectly fine :)
To be fair, on a previous commit you mentioned that it affected CentOS
8 which EOLed back in May 2024 so we shouldn't feel compelled to
support it any more.
https://lore.kernel.org/all/20230524170712.2036128-1-kuba@kernel.org/
Powered by blists - more mailing lists