zsh-users
 help / color / mirror / code / Atom feed
From: Bart Schaefer <schaefer@brasslantern.com>
To: zsh-users@zsh.org
Subject: Re: fndir introspection, site-packages documentation
Date: Sun, 15 Mar 2015 12:14:47 -0700	[thread overview]
Message-ID: <150315121447.ZM27996@torch.brasslantern.com> (raw)
In-Reply-To: <20150315021436.GQ4524@isis.sigpipe.cz>

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.

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 say there is no app for, and the other
half pass to some miscelleneous (wrong) app because of the ".patch" file
extension, which someone seems to think has something to do with setting
up network tunnels.

Grumble.  Text bleeping plain, or just put them in the message body.

I suppose I should also rant about email client authors who don't have
a "text/*" fallback.  Rant rant.  We now return you to your regularly
scheduled curmudgeonry.

On Mar 15,  3:14am, Roman Neuhauser wrote:
}
} uh, sorry, i meant "sitefndir".  anyway, how about the attached patch?

Indeed, I should have gotten that from your example instead of answering
the literal question.

}   ./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?

If there *is* such a situation, then the parameter name should be a lot
less configure-script-ified, "SITEFNDIR" is rather meaningless in any
other context.

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).

Also it's bad form in the doc to use references like var(prefix) when
you haven't mentioned where the value of prefix comes from, which in
turn requires delving into discussion of build configuration, which
is out of scope for end-user doc, which is another reason we haven't
included these details before.

(There are some parts of the doc where we try to actually pull in the
configuration values and embed them, so that we don't have to make
references to configure variables, but that gets ugly.)

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.


  reply	other threads:[~2015-03-15 19:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-13 22:41 Roman Neuhauser
2015-03-14  3:39 ` Bart Schaefer
2015-03-15  2:14   ` Roman Neuhauser
2015-03-15 19:14     ` Bart Schaefer [this message]
2015-03-16  9:36       ` patches format " Daniel Shahaf
2015-03-16 10:01         ` Peter Stephenson

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=150315121447.ZM27996@torch.brasslantern.com \
    --to=schaefer@brasslantern.com \
    --cc=zsh-users@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).