Re: [OE-core] [PATCH v2 2/3] devtool: upgrade: read changelog blobs via a temp file

[Daniel Turull <daniel.turull@ericsson.com>]

On Thu, 2026-08-06 at 14:05 +0000, Ross Burton wrote:
> On 4 Aug 2026, at 07:06, Daniel Turull via lists.openembedded.org
> <daniel.turull=ericsson.com@lists.openembedded.org> wrote:
> > The alternative to use a tmp file was to modify _run to allow other encodings than utf-8, but it
> > will be a bigger refactor. Or not use _run and call directly the raw subprocess calls.
> > 
> > I can change it to either option or keep the current patch.
> 
> bb.process.run() is meant for use by bitbake itself, it’s a glorified/bad wrapper around
> subprocess.run() that just does some logging magic whilst also getting in the way and doing things
> like forcing utf-8...  That magic isn’t useful in devtool, so I think we should be calling
> subprocess.run() directly.
> 
> Ross
ok. I'll change it to that. It was one one of the options that I had in mind but I was not sure if
we wanted to have all going to run()

Thanks Ross.

Daniel