On 11/07/2024 5:22 pm, Anthony PERARD wrote: > On Thu, Jul 11, 2024 at 12:15:13PM +0100, Andrew Cooper wrote: >> qemu-system-ppc64/8.1.0-ppc64 was added because bullseye's Qemu didn't > QEMU is normally spelled "QEMU" unless we are in camel case context, or > lower case context. > >> understand the powernv9 machine. >> >> However, bookworm's Qemu does, and this is preforable to maintaining a random > "preforable", that's an intended mispell right? :-)
Yes. both fixed. > >> build of Qemu ourselves. >> >> Add bookworm builds, and retain bullseye to keep our lower-bound toolchain >> testing. >> >> Rename the jobs to follow to sort coherently ($DISTRO-$VERSION-$ARCH-*) and > There's no renamed jobs in this patch, but the new jobs does follow the > new naming schema. Oh - good point. I'll tweak the wording. > Did you intend to rename both > "debian-bullseye-gcc-ppc64le*" jobs? No - that's done in the following patch. > >> reposition the jobs to optimise starting the smoke test. >> >> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Patch is mostly ok, so: > Reviewed-by: Anthony PERARD <anthony.per...@vates.tech> Thanks. ~Andrew