git: 7e4eca7136ea - main - Revert 97ec6eba653a07. There shouldn't be a dependency of 'tmp' on remote filesystems. Discussed this with Brooks and he can't find evidence that provoked the change in 2005. If anything gets broken I will fix it in a dif

Rodney W. Grimes freebsd at gndrsh.dnsmgr.net
Sun Jan 17 03:18:37 UTC 2021


> The branch main has been updated by glebius:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=7e4eca7136eaa35e15f67682468f09aa7127b543
> 
> commit 7e4eca7136eaa35e15f67682468f09aa7127b543
> Author:     Gleb Smirnoff <glebius at FreeBSD.org>
> AuthorDate: 2021-01-11 20:13:41 +0000
> Commit:     Gleb Smirnoff <glebius at FreeBSD.org>
> CommitDate: 2021-01-15 20:44:06 +0000
> 
>     Revert 97ec6eba653a07.  There shouldn't be a dependency of 'tmp' on
>     remote filesystems.  Discussed this with Brooks and he can't find
>     evidence that provoked the change in 2005.  If anything gets broken
>     I will fix it in a different way, not via rc sequence change.

This is because /etc/rc.d/tmp has this line:
        if ! /bin/df /tmp | grep -q "^/dev/md[0-9].* /tmp"; then

and when your doing NFS boots with seperate /usr you do not have
/usr/bin/grep to run that line.

Almost always the reason things have a REQUIRE: mountcritremote
is that they use some commands from /usr.

>     Discussed with:         brooks
>     Reviewed by:            ngie
>     Differential Revision:  https://reviews.freebsd.org/D28097
> ---
>  libexec/rc/rc.d/tmp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libexec/rc/rc.d/tmp b/libexec/rc/rc.d/tmp
> index 3b75ec0338ef..a61321e058ee 100755
> --- a/libexec/rc/rc.d/tmp
> +++ b/libexec/rc/rc.d/tmp
> @@ -28,7 +28,7 @@
>  #
>  
>  # PROVIDE: tmp
> -# REQUIRE: mountcritremote
> +# REQUIRE: mountcritlocal
>  
>  . /etc/rc.subr
>  
> 

-- 
Rod Grimes                                                 rgrimes at freebsd.org


More information about the dev-commits-src-all mailing list