Hi Tom, On Sun, 11 May 2025 at 16:39, Tom Rini <tr...@konsulko.com> wrote: > > On Sun, May 11, 2025 at 04:18:15PM +0200, Simon Glass wrote: > > > This series updates the two QEMU scripts available in U-Boot: > > > > - Converts build-qemu to Python > > - Converts build-efi to avoid sudo (unless SCT is used) > > - Introduces a proper settings file shared between both scripts > > > > It also includes a few documentation updates. > > > > The next step is to get these scripts running in CI. > > This extends the anti-pattern of "one big repository for everything" > while also expanding upon the scripting that I already said I don't want > in mainline and I thought you were going to maintain in the > u-boot-contrib repository instead.
Apart from the philosophical side, your original complaint was about use of sudo which I have mostly resolved. If you are worried about the size of the repo, these files are small and the repo is already big; in fact it grew nearly 30% in the last year. Another comment you had was about CVEs, but I'm not sure it is relevant in this case. I suspect we could find another way to make UEFI SCT work than mounting a loopback device. In fact I was hoping that Linaro might get SCT running in CI. Anyway, yes, I did add a script in your contrib repo [1] . But then I decided to see what others would do. It has been three months now and I haven't seen any contributions to it. So it may actually be that no one sees any value in it. I would like people to use and extend these scripts, e.g. for RISC-V and for booting other OSes. I very much doubt anything like that will happen in contrib/ but we can keep an eye on it and perhaps I will be wrong. I'd like to suggest that you consider another PoV on this. If contrib/ becomes a thriving marketplace, we can always delete the scripts from the tree. Regards, Simon [1] https://source.denx.de/u-boot/u-boot-extras/-/tree/master/contrib?ref_type=heads
