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

[Emmanuel Vadot]

On Sun, 17 Jan 2021 06:48:18 -0800 (PST)
"Rodney W. Grimes" <freebsd at gndrsh.dnsmgr.net> wrote:

> > 
> >  Hi Rod,
> > 
> >  Could you please fix your mailer, those issues have been buging me for
> > years :
> 
> Your mis-reading the quoted text, that is what my mailer is
> saying about the incoming mail, aka the below WAS pgp signed,
> and my mailer did not process that PGP signature.  
> My mailer is not
> > 
> > On Sun, 17 Jan 2021 06:03:02 -0800 (PST)
> > "Rodney W. Grimes" <freebsd at gndrsh.dnsmgr.net> wrote:
> > 
> > > -- Start of PGP signed section.
> > > [ Charset windows-1252 ignored, treated as US-ASCII ]
> > 
> >  I don't know why those lines are inserted, especially since your mail
> > isn't pgp signed.
> 
> Those are quoted lines, aka my mailer failed the PGP.  I usually
> do delete these.

 Those aren't quoted lines, those are lines inserted by your mailer
when you reply. (Or when you read no idea don't really care).

> 
> > 
> > > > Am 17.01.21 um 04:18 schrieb Rodney W. Grimes:
> > 
> >  Since your mailer doesn't add a quote introduction it always looks
> > like that you're replying to the previous mail (and here it looks like
> > you're replying to yourself).
> 
> The fact that the lines have added the leading "> " indicates otherwise.

 And the fact that everyone else on mailing list starts their reply
with something like "On XXX blah said :" and you're not making it
really hard to read your replies.

> > > > >> 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.
> > > 
> > > Critical statement, :"some commands": you fixed 1 of them,
> > > I also so a bit later:
> > >         if _tmpdir=$(mktemp -d -q /tmp/.diskless.XXXXXX); then
> > > 
> > > mktemp lives in /usr/bin also
> > > 
> > > > This dependency on /usr/bin/grep is easily removed, see:
> > > > 
> > > > 	https://reviews.freebsd.org/D28209
> > > > 
> > > 
> > > I like this fix, less dependance on /usr during boot
> > > processing is good.
> > > 
> > > > diff --git a/libexec/rc/rc.d/tmp b/libexec/rc/rc.d/tmp
> > > > index a61321e058ee..2a7205e92b59 100755
> > > > --- a/libexec/rc/rc.d/tmp
> > > > +++ b/libexec/rc/rc.d/tmp
> > > > @@ -40,10 +40,16 @@ load_rc_config $name
> > > > 
> > > >   mount_tmpmfs()
> > > >   {
> > > > -       if ! /bin/df /tmp | grep -q "^/dev/md[0-9].* /tmp"; then
> > > > -               mount_md ${tmpsize} /tmp "${tmpmfs_flags}"
> > > > -               chmod 01777 /tmp
> > > > -       fi
> > > > +       while read line; do
> > > > +               case $line in
> > > > +               /dev/md[0-9]*\ /tmp)
> > > > +                       return;;
> > > > +               esac
> > > > +       done <<*EOF
> > > > +$(df /tmp)
> > > > +*EOF
> > > > +       mount_md ${tmpsize} /tmp "${tmpmfs_flags}"
> > > > +       chmod 01777 /tmp
> > > >   }
> > > > 
> > > >   # If we do not have a writable /tmp, create a memory
> > > > 
> > > -- End of PGP section, PGP failed!
> > 
> >  See, not pgp signed.
> > 
> >  Cheers,
> > 
> > > -- 
> > > Rod Grimes                                                 rgrimes at freebsd.org
> > 
> > 
> > -- 
> > Emmanuel Vadot <manu at bidouilliste.com>
> > 
> 
> -- 
> Rod Grimes                                                 rgrimes at freebsd.org


-- 
Emmanuel Vadot <manu at bidouilliste.com>