On Fri, May 08, 2020 at 09:23:42AM -0500, Eric Sandeen wrote:
> Add a _require_mknod test so that filesystems with no ->mknod
> operation will _notrun instead of fail.
>
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
> ---
>
> V2 trim the long line, sorry.
>
> diff --git a/common/rc b/common/rc
> index 2734dbf6..b50a4294 100644
> --- a/common/rc
> +++ b/common/rc
> @@ -4214,6 +4215,13 @@ _require_sysctl()
> sysctl $name &>/dev/null || _notrun "$name sysctl unavailable"
> }
>
> +_require_mknod()
> +{
> + mknod $TEST_DIR/$tmp.null c 1 3 \
$tmp points to /tmp/$$, so this mknod is trying to make node at
$TEST_DIR//tmp/<pid>
which will always fail.
Use $TEST_DIR/$seq.null instead?
And I don't see any callers of this require rule, shouldn't existing
tests that use mknod be updated to require this rule?
Thanks,
Eryu
> + || _notrun "$FSTYP does not support mknod/mkfifo"
> + rm -f $TEST_DIR/$tmp.null
> +}
> +
> init_rc
>
> ################################################################################
>
