zsh-workers
 help / color / mirror / code / Atom feed
From: Roman Neuhauser <neuhauser@sigpipe.cz>
To: Bart Schaefer <schaefer@brasslantern.com>
Cc: zsh-workers@zsh.org
Subject: Re: fndir introspection, site-packages documentation
Date: Mon, 16 Mar 2015 09:08:19 +0100	[thread overview]
Message-ID: <20150316080819.GR4524@isis.sigpipe.cz> (raw)
In-Reply-To: <150315121447.ZM27996@torch.brasslantern.com>

# schaefer@brasslantern.com / 2015-03-15 12:14:47 -0700:
> This really doesn't belong on zsh-users any more, in fact it probaby should
> have started out on zsh-workers to begin with.  I haven't changed the list
> target yet for this reply, but I suggest follow-ups go there.

but then i'm bound to miss replies.  i don't want to subscribe to
zsh-workers (yet)...

> Aside:  I don't suppose there's any point in again begging people to stop
> attaching patches as "text/x-patch" or "text/x-diff" or other silly MIME
> types that half my email clients [...]

i recon it'd be easier to reconfigure both halves of your email clients. :)
but, sorry to make your life difficult, it wasn't deliberate.

> On Mar 15,  3:14am, Roman Neuhauser wrote:
> }   ./Src/zsh -fc 'print -l "$ZSH_SITEFNDIR"'     
> }   /omg/share/zsh/site-functions
> 
> I don't have any strong objection to this although it seems like putting
> this value into every shell is overkill for the specialized purpose to
> which you propose to use it.  Can anyone give me a situation in which an
> oridinary user (i.e., not a software packager) would need this?

how about: a site admin wants to install a function for all users.

> Your doc change mentions
> 
> +tt(/usr/local/share/zsh/site-functions) is always included,
> +even if it does not exist, and cannot be configured away.
> 
> That's not quite correct: --disable-site-fndir turns it off, leaving
> only $prefix/share/zsh/$ZSH_VERSION/functions (or subdirs).

i described what i saw in testing.  what i haven't noticed is that
--prefix=/usr/local is a special case.  in fact, removing the hardcoded
failsafe neuters the purpose of the whole thing; you may treat this as a
bug report.

try it yourself:

  % ./configure --prefix=/omg --disable-site-fndir &&
    make &&
    ./Src/zsh -fc 'print -l $fpath'
  /usr/local/share/zsh/site-functions
  /omg/share/zsh/5.0.7-dev-1/functions

  % ./configure --disable-site-fndir &&
    make &&
    ./Src/zsh -fc 'print -l $fpath'
  /usr/local/share/zsh/5.0.7-dev-1/functions

the behavior with non-default --prefix is described in NEWS:

- The default $fpath/$FPATH is now designed always to include
  /usr/local/share/zsh/site-functions.  This directory does not need to
  exist.  Sites that set an explicit site directory can put that in
  /etc/zshenv as before.  The intention of the new path element is to
  increase the likelihood that locally added functions can be found with
  the shell running "out of the box", no matter how the shell was
  configured.

the behavior with *default* --prefix is, well, completely contrary to
the intent described: "no matter how the shell was configured" is blatantly
false thanks to this special case.

> Maybe the right thing is to throw all of this into a module (and doc
> for that module) that can be loaded by package maintainers to provide
> access to zsh's configure settings, but which other users can ignore.
> The more I think about it the more I like that idea.

zmodload or pkg-config?

-- 
roman


       reply	other threads:[~2015-03-16  8:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20150313224121.GO4524@isis.sigpipe.cz>
     [not found] ` <150313203904.ZM25016@torch.brasslantern.com>
     [not found]   ` <20150315021436.GQ4524@isis.sigpipe.cz>
     [not found]     ` <150315121447.ZM27996@torch.brasslantern.com>
2015-03-16  8:08       ` Roman Neuhauser [this message]
2015-03-16 16:08         ` Bart Schaefer
2015-03-16 16:46           ` Ray Andrews
2015-03-17 15:41             ` Bart Schaefer
2015-03-17 15:55               ` Ray Andrews
2015-03-16 17:05           ` Roman Neuhauser
2015-03-17 15:45             ` Bart Schaefer
2015-03-17 16:03               ` Roman Neuhauser
2015-03-16 17:13           ` Peter Stephenson
2015-03-16 21:22             ` Roman Neuhauser
2015-03-17 15:52             ` Bart Schaefer
2015-03-17 18:46               ` Peter Stephenson
2015-03-18  3:38                 ` Bart Schaefer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150316080819.GR4524@isis.sigpipe.cz \
    --to=neuhauser@sigpipe.cz \
    --cc=schaefer@brasslantern.com \
    --cc=zsh-workers@zsh.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).