[Toybox] [PATCH] Remove mount.test awk dependency.

Rob Landley rob at landley.net
Fri Apr 7 09:47:09 PDT 2017


On 04/03/2017 10:35 AM, enh wrote:
> 
> Parsing file(1) output isn't a good way to determine file system type
> anyway.

The mount tests (which there need to be soooo many more of, there's
comments in mount.c about that) are one of the category of "tests really
needing mkroot" because:

A) Requires root to run.
B) Requires a host with known stuff on it to test
C) Hard to clean up after if it fails.
D) Can potentially hose the system badly enough to require a reboot if
it goes wrong.

The other cannonical example of this is ifconfig. :)

(Possibly it could be done with containers instead of qemu, but
insmod.test can't. And although I finally figured out how to test ps, it
requires a --bind mount over /proc after the first few simple ones.)

Rob



More information about the Toybox mailing list