From owner-svn-src-all@freebsd.org Fri Jan 25 00:23:56 2019 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 32DAB14C46E2; Fri, 25 Jan 2019 00:23:56 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 97E958EF61; Fri, 25 Jan 2019 00:23:55 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id x0P0Nr60085676; Thu, 24 Jan 2019 16:23:53 -0800 (PST) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id x0P0NrC0085675; Thu, 24 Jan 2019 16:23:53 -0800 (PST) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201901250023.x0P0NrC0085675@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r343416 - head/bin/sh In-Reply-To: <5bf59c74ab2bfaf831811df273ae1adde1599acc.camel@freebsd.org> To: Ian Lepore Date: Thu, 24 Jan 2019 16:23:53 -0800 (PST) CC: rgrimes@freebsd.org, Edward Tomasz Napierala , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 97E958EF61 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.99 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; REPLY(-4.00)[]; NEURAL_HAM_SHORT(-0.99)[-0.986,0] X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Jan 2019 00:23:56 -0000 [ Charset UTF-8 unsupported, converting... ] > On Thu, 2019-01-24 at 15:55 -0800, Rodney W. Grimes wrote: > > > Author: trasz > > > Date: Thu Jan 24 23:34:51 2019 > > > New Revision: 343416 > > > URL: https://svnweb.freebsd.org/changeset/base/343416 > > > > > > Log: > > > Install .shrc for root, and set PS1 for the toor account. > > > > And a dozen other aliases :-( > > Please do not contaiminate the prestine environment with > > personal preferences. In the start of the project we > > did a great deal of work to remove and eliminate these > > types of things, only the few csh aliases where retained. > > > > This is really the domain of a systems administrator to > > decide and making work for them to clean this out is > > not going to make them happy. > > > > The commands in an ENV file well be executed by > > every single shell that is started, not just an > > interactive shell, unless I am mis reading the > > man page: > > If the environment variable ENV is set on > > entry to a shell, or is set in the .profile of a login shell, the > > shell > > then subjects its value to parameter expansion and arithmetic > > expansion > > and reads commands from the named file. The man page needs fixed here too, it does infact later state that THIS version of sh does not always run ENV, only for interactive shells. Which is a good thing, but there is still this un needed contaimination with aliases. You also just caused me to have to revive my .profile diff cause the addition of the ENV falls in the context range of my local mods. I am sure this effects others. Please do consider that these files have been very very rarely touched over decades of time. > > > > I agree with all of tat. If the local sysadmin wants to make /bin/sh > friendly as an interactive shell, maybe we could provide some .sample > files to make that easy, but we SHOULD NOT put all this junk in by > default. > > -- Ian > > > > Reviewed by: jilles > > > MFC after: 2 weeks > > > Sponsored by: DARPA, AFRL > > > Differential Revision: https://reviews.freebsd.org/D18872 > > > > > > Added: > > > head/bin/sh/dot.shrc > > > - copied unchanged from r343399, head/share/skel/dot.shrc > > > Modified: > > > head/bin/sh/Makefile > > > head/bin/sh/dot.profile > > > > > > Modified: head/bin/sh/Makefile > > > =================================================================== > > > =========== > > > --- head/bin/sh/Makefile Thu Jan 24 22:34:30 2019 (r343415) > > > +++ head/bin/sh/Makefile Thu Jan 24 23:34:51 2019 (r343416) > > > @@ -3,9 +3,12 @@ > > > > > > .include > > > > > > -CONFS= dot.profile profile > > > -CONFSDIR_dot.profile= /root > > > -CONFSNAME_dot.profile= .profile > > > +CONFGROUPS= ETC ROOT > > > +ETC= profile > > > +ROOT= dot.shrc dot.profile > > > +ROOTDIR= /root > > > +ROOTNAME_dot.shrc= .shrc > > > +ROOTNAME_dot.profile= .profile > > > PACKAGE=runtime > > > PROG= sh > > > INSTALLFLAGS= -S > > > > > > Modified: head/bin/sh/dot.profile > > > =================================================================== > > > =========== > > > --- head/bin/sh/dot.profile Thu Jan 24 22:34:30 2019 (r343415) > > > +++ head/bin/sh/dot.profile Thu Jan 24 23:34:51 2019 (r343416) > > > @@ -9,6 +9,9 @@ export TERM > > > PAGER=less > > > export PAGER > > > > > > +# set ENV to a file invoked each time sh is started for > > > interactive use. > > > +ENV=$HOME/.shrc; export ENV > > > + > > > # Query terminal size; useful for serial lines. > > > if [ -x /usr/bin/resizewin ] ; then /usr/bin/resizewin -z ; fi > > > > > > > > > Copied: head/bin/sh/dot.shrc (from r343399, > > > head/share/skel/dot.shrc) > > > =================================================================== > > > =========== > > > --- /dev/null 00:00:00 1970 (empty, because file is > > > newly added) > > > +++ head/bin/sh/dot.shrc Thu Jan 24 23:34:51 2019 (r343416, > > > copy of r343399, head/share/skel/dot.shrc) > > > @@ -0,0 +1,39 @@ > > > +# $FreeBSD$ > > > +# > > > +# .shrc - bourne shell startup file > > > +# > > > +# This file will be used if the shell is invoked for interactive > > > use and > > > +# the environment variable ENV is set to this file. > > > +# > > > +# see also sh(1), environ(7). > > > +# > > > + > > > + > > > +# file permissions: rwxr-xr-x > > > +# > > > +# umask 022 > > > + > > > +# Uncomment this to enable the builtin vi(1) command line editor > > > in sh(1), > > > +# e.g. ESC to go into visual mode. > > > +# set -o vi > > > + > > > + > > > +# some useful aliases > > > +alias h='fc -l' > > > +alias j=jobs > > > +alias m="$PAGER" > > > +alias ll='ls -laFo' > > > +alias l='ls -l' > > > +alias g='egrep -i' > > > + > > > +# # be paranoid > > > +# alias cp='cp -ip' > > > +# alias mv='mv -i' > > > +# alias rm='rm -i' > > > + > > > + > > > +# set prompt: ``username@hostname:directory $ '' > > > +PS1="\u@\h:\w \\$ " > > > + > > > +# search path for cd(1) > > > +# CDPATH=:$HOME > > > > > > > > > > > > -- Rod Grimes rgrimes@freebsd.org