Hi Tom, On Wed, 19 Mar 2025 at 15:52, Tom Rini <tr...@konsulko.com> wrote: > > On Wed, Mar 19, 2025 at 03:38:03PM +0100, Simon Glass wrote: > > > Add an extlinux image that contains a few Ubuntu entries. > > > > Increase the number of sandbox-USB-hub ports to permit this. > > > > Signed-off-by: Simon Glass <s...@chromium.org> > > I don't understand what this test adds. In neither the current Fedora > test nor in this new test are we actually booting something, we're just > taking a sample extlinux.conf and making sure it doesn't fail. Is it > that we're testing in a useful fashion now having two labels?
I didn't think so either, which is why I never did this before. But it turns out that there were some bugs, too. > > We should probably be clear about what we're doing in the tests and > instead of adding seemingly arbitrary distributions add an extlinux test > and testcases. This is not actually a test case. It is simply creating a new image. The test cases are in the other patches, so please take a look there. Regards, Simon