Re: fndir introspection, site-packages documentation

[Peter Stephenson <p.stephenson@xxxxxxxxxxx>]

On Tue, 17 Mar 2015 08:52:33 -0700
Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx> wrote:
> On Mar 16,  5:13pm, Peter Stephenson wrote:
> } Subject: Re: fndir introspection, site-packages documentation
> }
> } On Mon, 16 Mar 2015 09:08:29 -0700
> } Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx> wrote:
> } > I will
> } > however note that I think --disable-site-fndir ought to do exactly that,
> } > and the documentation blurb is what needs fixing.  If the shell is being
> } > configured for some kind of secure/embedded environment, removing all
> } > access to locally added functions is perfectly reasonable.
> } 
> } Yes, that seems right.
> 
> OK, but that's not actually what happens if you change the --prefix.  You
> can't currently get both a different prefix and no extra directory in fpath.

I think you're suggesting the following.  Let we know explicitly how
this needs to change if not.

pws

diff --git a/configure.ac b/configure.ac
index 7e770cd..e4de193 100644
--- a/configure.ac
+++ b/configure.ac
@@ -318,7 +318,8 @@ fi], [sitefndir=${datadir}/${tzsh_name}/site-functions])
 dnl Add /usr/local/share/zsh/site-functions if not yet present
 dnl owing to $sitefndir, whether or not explicitly given.
 dnl If not explicitly given, it hasn't been expanded yet.
-if test X$sitefndir = X/usr/local/share/zsh/site-functions
+if test X$sitefndir = X/usr/local/share/zsh/site-functions || \
+   test X$sitefndir = Xno
 then fixed_sitefndir=''
 elif test X$prefix != X/usr/local; then
   if test X$prefix = XNONE && test X$ac_default_prefix = X/usr/local; then