Date: Mon, 14 Jan 2013 23:57:00 +0100 From: Jilles Tjoelker <jilles@stack.nl> To: Eitan Adler <eadler@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r245426 - head/bin/sh Message-ID: <20130114225700.GA93645@stack.nl> In-Reply-To: <CAF6rxgnPmjdmm8HMVweEFEOukRv=9QQ-D72-WCsirfBAAUn1yw@mail.gmail.com> References: <201301141640.r0EGepLc050623@svn.freebsd.org> <CAF6rxgnPmjdmm8HMVweEFEOukRv=9QQ-D72-WCsirfBAAUn1yw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Jan 14, 2013 at 01:38:51PM -0500, Eitan Adler wrote: > On 14 January 2013 11:40, Jilles Tjoelker <jilles@freebsd.org> wrote: > > Author: jilles > > Date: Mon Jan 14 16:40:50 2013 > > New Revision: 245426 > > URL: http://svnweb.freebsd.org/changeset/base/245426 > > Log: > > sh: Simplify cd-hash interaction. > > Instead of rechecking relative paths for all hashed utilities after a cd, > > track if any utility in cmdtable depends on a relative path in PATH. > > If there is such a utility, cd clears the entire table. > Why not clear just these entries instead of the entire table? Clearing too much might affect performance but not correctness. The utilities will be looked up again if necessary. Because it is uncommon nowadays to include a relative pathname in PATH, I handle this case in a very simple manner. -- Jilles Tjoelker
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130114225700.GA93645>