On Mon, 11 May 2026 at 16:16, Daniel P. Berrangé <[email protected]> wrote: > > On Mon, May 11, 2026 at 03:47:33PM +0100, Peter Maydell wrote: > > On Fri, 1 May 2026 at 12:55, Peter Maydell <[email protected]> wrote: > > > > > > The Python os.setxattr() API is Linux-specific, so trying to use > > > it on other OSes triggers a failure: > > > > > > File "/Users/pm215/src/qemu/tests/functional/qemu_test/asset.py", > > > line 227, in fetch > > > os.setxattr(str(tmp_cache_file), "user.qemu-asset-url", > > > ^^^^^^^^^^^ > > > AttributeError: module 'os' has no attribute 'setxattr' > > > > > > Since we only set the attributes here for informational > > > purposes, skip them when os.setxattr() isn't available. > > > > > > Cc: [email protected] > > > Fixes: 9903217a4ed013 ("tests/functional: add a module for handling asset > > > download & caching") > > > Signed-off-by: Peter Maydell <[email protected]> > > > > Is anybody going to pick this up via their tree, or should > > I just put it into target-arm.next ? > > I have it for my misc queue, but if you want to send a PR now then > consider it to havve > > Acked-by: Daniel P. Berrangé <[email protected]>
I probably won't have a full pullreq ready to send until the end of the week, so feel free to keep it in your queue. thanks -- PMM
