grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 1/2] tests: Test aborts due to missing requirements should


From: Julian Andres Klode
Subject: Re: [PATCH v2 1/2] tests: Test aborts due to missing requirements should be marked as error instead of skipped
Date: Wed, 19 Jul 2023 17:37:55 +0200

On Thu, Oct 14, 2021 at 03:58:07PM -0500, Glenn Washburn wrote:
> Many tests abort due to not being root or missing tools, for instance mkfs
> commands for file system tests. The tests are exited with code 77, which
> means they were skipped. A skipped test is a test that should not be run,
> eg. a test specific to ARM64 should not be run on an x86 build. These aborts
> are actually a hard error, code 99. That means that the test could not be
> completed, but not because what was supposed to be tested failed, eg. in
> these cases where a missing tool prevents the running of a test.

This change is inappropriate. We can't just fail the build in distro
packaging just because our builders don't run as root.

Similarly dependencies - it's our choice which dependencies we install,
if we don't care about stuff, it should just skip it.

Every distro will have to revert this change presumably so they can
build grub.

-- 
debian developer - deb.li/jak | jak-linux.org - free software dev
ubuntu core developer                              i speak de, en



reply via email to

[Prev in Thread] Current Thread [Next in Thread]