On Fri, Mar 26, 2021 at 04:41:30PM +0100, Pavel Reichl wrote: > Add helper function that ensures that test is only executed on file > systems that implement chown and chmod. > > Signed-off-by: Pavel Reichl <preichl@xxxxxxxxxx> I think patch 1 and patch 2 could be folded into one patch, introduce new helpers and use them in the same patch. Thanks, Eryu > --- > common/rc | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/common/rc b/common/rc > index 0ce3cb0d..b18d37fd 100644 > --- a/common/rc > +++ b/common/rc > @@ -2129,6 +2129,24 @@ _require_user() > [ "$?" == "0" ] || _notrun "$qa_user cannot execute commands." > } > > +# check for a chown support > +# > +_require_chown() > +{ > + if [ "$FSTYP" = "exfat" ]; then > + _notrun "chown is not supported on $FSTYP" > + fi > +} > + > +# check for a chmod support > +# > +_require_chmod() > +{ > + if [ "$FSTYP" = "exfat" ]; then > + _notrun "chmod is not supported on $FSTYP" > + fi > +} > + > # check for a group on the machine, fsgqa as default > # > _require_group() > -- > 2.30.2