Date: Fri, 25 Jan 2019 14:47:02 -0800 From: Devin Teske <dteske@FreeBSD.org> To: Edward Napierala <trasz@freebsd.org> Cc: Devin Teske <dteske@FreeBSD.org>, rgrimes@freebsd.org, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r343440 - head/bin/sh Message-ID: <B6C77240-B06E-4F47-AB30-C93D96FF1FEF@FreeBSD.org> In-Reply-To: <D7EEA6BA-EAA6-4A84-A7C4-904999B0E581@FreeBSD.org> References: <201901251709.x0PH9Rc4094379@repo.freebsd.org> <201901251957.x0PJvdTL089917@pdx.rh.CN85.dnsmgr.net> <CAFLM3-oSafSxcWSJPEXQ8szLEq5N593fr=sEsup6A2%2BP_VXrgQ@mail.gmail.com> <D7EEA6BA-EAA6-4A84-A7C4-904999B0E581@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Jan 25, 2019, at 2:41 PM, Devin Teske <dteske@FreeBSD.org> wrote: >=20 >=20 >=20 >> On Jan 25, 2019, at 1:37 PM, Edward Napierala <trasz@freebsd.org = <mailto:trasz@freebsd.org>> wrote: >>=20 >> pt., 25 sty 2019 o 19:57 Rodney W. Grimes >> <freebsd@pdx.rh.cn85.dnsmgr.net = <mailto:freebsd@pdx.rh.cn85.dnsmgr.net>> napisa=C5=82(a): >>>=20 >>>> Author: trasz >>>> Date: Fri Jan 25 17:09:26 2019 >>>> New Revision: 343440 >>>> URL: https://svnweb.freebsd.org/changeset/base/343440 = <https://svnweb.freebsd.org/changeset/base/343440>; >>>>=20 >>>> 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. >>>>=20 >>>> MFC after: 2 weeks >>>> Sponsored by: DARPA, AFRL >>>=20 >>> 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. >>=20 >> 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. >>=20 >> 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. >>=20 >> So far nobody had proposed any scenario where this would break >> anything, or even affect existing users. It seems like a typical = bikeshed >> situation. >=20 > It is not clear to me after reading r343416 and D18872 what this = change is trying to solve. >=20 > PS1 should have a reasonable default. If that default is not = reasonable, then we should change the C code. >=20 > Maybe I see things differently, but I'd rather see PS1 default change = so no profile/shrc change is necessary. >=20 > I prefer that sh, in its default configuration, not attempt to read = $HOME/.shrc, for security reasons. >=20 > Further, it is documented that the contents of ENV may be ignored in = privileged mode, negating these changes. >=20 > If you wanted your new shiny default PS1 to actually have an effect in = all modes (including privileged mode, where you probably want it), you = would have put it in /etc/profile and not in a file that is wholly = ignored by some modes (e.g., privileged mode). >=20 > So the solution is not even the right one for the desired result. I would also like to add, that the current default for PS1 is static for = a reason. Long ago, people used to write things in TCL/Expect. If PS1 is not = static, you either have to override it or account for the variance (# = for root, $ for others). This is an important distinction specifically because TCL/Expect is used = in the control of interactive shells. --=20 Devin=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?B6C77240-B06E-4F47-AB30-C93D96FF1FEF>