[External] Re: [PATCH 2/3] tools: binman: fix deprecated Python ConfigParser methods

[Maier, Brandon L Collins]

Hi Simon,

> -----Original Message-----
> From: Simon Glass <sjg at chromium.org>
> Sent: Tuesday, June 4, 2024 9:14 PM
> To: Maier, Brandon L Collins <Brandon.Maier at collins.com>
> Cc: u-boot at lists.denx.de
> Subject: [External] Re: [PATCH 2/3] tools: binman: fix deprecated Python
> ConfigParser methods
>
> Hi Brandon,
>
> On Tue, 4 Jun 2024 at 10:16, Brandon Maier <brandon.maier at collins.com>
> wrote:
> >
> > The method `ConfigParser.readfp()` is marked deprecated[1].
> >
> > In Python 3.12 this method have been removed, so replace it with
> > `ConfigParser.read_file()`.
> >
> > [1]
> https://docs.python.org/3.11/library/configparser.html#configparser.ConfigParser.read_file
> > Signed-off-by: Brandon Maier <brandon.maier at collins.com>
> > CC: Simon Glass <sjg at chromium.org>
> > ---
> >  tools/buildman/bsettings.py | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> Does this still work in earlier Pythons?

The Python docs say ConfigParser.read_file() was added in Python 3.2.
The tools/buildman/pyproject.toml says `requires-python = ">=3.7"` so I assume this is fine.

>
> Regards,
> Simon