Subject: Re: [PATCH net] tools: ynl: avoid dict errors on older Python versions

From: Donald Hunter <[email-masked]>
Date: Thu, 25 May 2023 11:45:28 +0100

Jakub Kicinski <[email-masked]> writes:

> Python 3.9.0 or newer supports combining dicts() with |,
> but older versions of Python are still used in the wild
> (e.g. on CentOS 8, which goes EoL May 31, 2024).
> With Python 3.6.8 we get:
>
>   TypeError: unsupported operand type(s) for |: 'dict' and 'dict'
>
> Use older syntax. Tested with non-legacy families only.
>
> Fixes: f036d936ca57 ("tools: ynl: Add fixed-header support to ynl")
> Signed-off-by: Jakub Kicinski <[email-masked]>
> ---
> CC: [email-masked]
>
> This is arguably not a fix, but those trying YNL on 6.4 will likely
> appreciate not running into the problem.

I just hit the issue on Ubuntu on a BF2 so thank you for getting a fix
out before me! Tested with ovs_datapath.

Reviewed-by: Donald Hunter <[email-masked]>
Tested-by: Donald Hunter <[email-masked]>