Hi! Here's v2 of this humble patchset. Here's what changed since v1: - 'user' variable in _require_user_exists() is now local, as suggested by Eric Biggers. (I've also added Darrick's Reviewed-by tag to the 2nd patch only.) Original cover-letter: Test generic/079 currently fails if for some reason the 'nobody' or 'daemon' users don't exist: *** starting up add_acl(/mnt/scratch/079/immutable.f) did not set errno == EPERM add_acl(/mnt/scratch/079/immutable.d) did not set errno == EPERM add_acl(/mnt/scratch/079/append-only.f) did not set errno == EPERM add_acl(/mnt/scratch/079/append-only.d) did not set errno == EPERM testing immutable...FAILED! (2 tests failed) testing append-only...FAILED! (2 tests failed) *** cleaning up The first patch that follows adds a new function _require_user_exists() that will verify a user exists in the system (and refactors existent _require_user() to use the new function). The second patch is the actual test fix. Luis Henriques (2): common/rc: add _require_user_exists() to check if a user exists generic/079: make sure users 'nobody' and 'daemon' exist common/rc | 27 ++++++++++++++++++--------- tests/generic/079 | 2 ++ 2 files changed, 20 insertions(+), 9 deletions(-)
