On Fri, 30 Jan 2026 at 04:10, Chen Qi via lists.openembedded.org <[email protected]> wrote: > This patch is basically a revert of the following three patches: > - 0c10a78796 scripts/runqemu: raise an error when bitbake was not found > - b931f74442 scripts/runqemu: remove the code block that works around the > missing bitbake environment > - 8197be4dd3 runqemu: ensure that bitbake environment is either returned, or > an exception is raised
Again, you can't just revert these commits. Please read the commit messages for the reasons the commits were added. Particularly the first, and the third one are real improvements, one gives users a better error message, the other significantly improves the function call API. > Using runqemu from SDK is useful. Here's a typical case for it: > Team A build different images and SDKs for different machines and copy them > to a central place. > Team B test these different images with different methods. For qemu images, > they extract > SDK and use runqemu from SDK to test these qemu images. > > Below are example steps for the above qemu image copying and running process: > 1. mkdir destdir > 2. cp -r /path/to/build/tmp/deploy/image/qemux86-64 destdir > 3. Install SDK to destdir > 4. Source SDK > 5. runqemu qemux86-64 nographic slirp So you need to start with this: a test case for these steps. Then when we can see how the test case fails, we can work out a fix that doesn't simply undo the changes. I've already said this in the previous review, so please do not simply resend the patch. Alex
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#230377): https://lists.openembedded.org/g/openembedded-core/message/230377 Mute This Topic: https://lists.openembedded.org/mt/117539270/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
