svn commit: r343440 - head/bin/sh

Rodney W. Grimes freebsd at pdx.rh.CN85.dnsmgr.net
Fri Jan 25 22:29:21 UTC 2019


> pt., 25 sty 2019 o 19:57 Rodney W. Grimes
> <freebsd at pdx.rh.cn85.dnsmgr.net> napisa?(a):
> >
> > > Author: trasz
> > > Date: Fri Jan 25 17:09:26 2019
> > > New Revision: 343440
> > > URL: https://svnweb.freebsd.org/changeset/base/343440
> > >
> > > Log:
> > >   Comment out the default sh(1) aliases for root, introduced in r343416.
> > >   The rest of this stuff is still to be discussed, but I think at this
> > >   point we have the agreement that the aliases should go.
> > >
> > >   MFC after:  2 weeks
> > >   Sponsored by:       DARPA, AFRL
> >
> > Please just revert this and the prior commit out, and when
> > the path forward is clear commit it.  I would not want any of this
> > merged to 12/ or 11/ until the time that it is all settled.
> 
> Oops, my bad - neither this nor the previous commit is supposed
> to be MFC-ed; the "2 weeks" above comes from my default Subversion
> config.
> 
> Regarding the backoff - just a few hours ago you said you don't have
> any problem with this, except for aliases and the default ENV.  The
> aliases problem has been addressed, and you hadn't yet responded
> to my explanations regarding the ENV.  Another committer asked for
> backoff, because "sh is not an interactive shell", while in fact sh(1)
> is FreeBSD's default interactive shell except for root.  Finally, there's
> one person who asked for revert, but without giving any reasons
> whatsoever.
> 
> So far nobody had proposed any scenario where this would break
> anything, or even affect existing users.  It seems like a typical bikeshed
> situation.

Within hours of your initial commit you had myself, ian and cy all in
opposition to this change , devin asked for a revert, first privately to me
which I encouraged a public posting of.
Thats four people who want this removed.
I have now asked publically for a revert.
Thats 2 public request for revert.
The hat was only about the MFC statement which you have retracted.

Just back it out and we can come to a better solution.
Especially dependent on the outcome of Bapts mksh proposal.

-- 
Rod Grimes                                                 rgrimes at freebsd.org


More information about the svn-src-head mailing list