zsh-workers
 help / color / mirror / code / Atom feed
From: Peter Stephenson <p.w.stephenson@ntlworld.com>
To: zsh-workers@zsh.org
Subject: Re: modify functions hierarchy (was: Install run-help and *.zwc files system wide in build system)
Date: Sun, 17 Nov 2013 20:13:59 +0000	[thread overview]
Message-ID: <20131117201359.1613ec4c@pws-pc.ntlworld.com> (raw)
In-Reply-To: <131117103047.ZM30518@torch.brasslantern.com>

On Sun, 17 Nov 2013 10:30:47 -0800
Bart Schaefer <schaefer@brasslantern.com> wrote:
> On Nov 16, 12:14pm, Martin Vaeth wrote:
> }
> } This patch implements a mechanism which supports "source/" subdirs
> } in the function hierarchy:
> 
> I'd rather these were not inside the tree like that.  There should be an
> entirely separate location for them.
> 
> In fact, in thinking about it, furter, I believe we should maintain the
> Completion/ and Functions/ trees as conceptually read-only at "make all"
> time.   Any files like _run_help and run-help that need to be generated
> from a .in file should be processed by "make install" (install.fns rule)
> directly into the $(sitefndir) target, and should never appear in the
> Zsh build tree.

I can see the potential confusion in having files that need processing
in the same tree as files that get installed verbatim, although I didn't
see it as necessarily a problem in practice, but the files should be
generated at make time, not install time, otherwise it's inconsistent
with the rest of the build system.  Up to now, installation has been a
straight copy and I don't see any point in changing that: it adds an
extra point of failure (other than those applicable to copying,
e.g. permissions), for one thing.  I'm not even sure the config.status
processing applicable to substitutions for .in files, which is what we
want here (and I don't see any point in inventing a new form of
processing), is designed to go straight to install directories.
 
> Taking that as step further, I'm beginning to think the helpfiles should
> also be generated by "make install" rather than being written to a tree
> underneath Doc/.

Again, this is inconsistent with other files that get installed,
and also (which is really part of the same thing) means we can't supply
them in the doc bundle with the other pre-generated documentation.

In both cases, where they get written is another matter.

We could have a parallel hierarchy Functions_pre for generated functions.

The doc bundle isn't supposed to be standalone, it's supposed to give
you additional files to install using the files from the normal build,
so I don't think that's a limitation.  So I'm not that fussed where the
generated helpfiles go in the first instance.

-- 
Peter Stephenson <p.w.stephenson@ntlworld.com>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/


  parent reply	other threads:[~2013-11-17 20:20 UTC|newest]

Thread overview: 79+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-10  8:57 Install run-help and *.zwc files system wide in build system Martin Vaeth
2013-11-10 18:13 ` Peter Stephenson
2013-11-10 22:28   ` Martin Vaeth
2013-11-11  2:30     ` Bart Schaefer
2013-11-11  9:09       ` Martin Vaeth
2013-11-11 16:42         ` Bart Schaefer
2013-11-11 22:59           ` Martin Vaeth
2013-11-12 10:11             ` Peter Stephenson
2013-11-12 17:41               ` Martin Vaeth
2013-11-12 17:48                 ` Peter Stephenson
2013-11-12 21:04                   ` Peter Stephenson
2013-11-12 21:42                     ` Martin Vaeth
2013-11-13  9:28                       ` Peter Stephenson
2013-11-13 10:10                         ` Martin Vaeth
2013-11-13 11:21                           ` Peter Stephenson
2013-11-13 16:06                             ` Bart Schaefer
2013-11-16 12:14                               ` modify functions hierarchy (was: Install run-help and *.zwc files system wide in build system) Martin Vaeth
2013-11-17 18:30                                 ` Bart Schaefer
2013-11-17 19:42                                   ` Martin Vaeth
2013-11-17 21:01                                     ` Bart Schaefer
2013-11-18  7:00                                       ` Martin Vaeth
2013-11-18 16:11                                         ` Doc/Doc/help (was: modify functions hierarchy (was: Install run-help and *.zwc files system wide in build system)) Bart Schaefer
2013-11-18 16:23                                           ` Peter Stephenson
2013-11-18 16:31                                             ` Bart Schaefer
2013-11-18 16:36                                               ` Peter Stephenson
2013-11-18 16:45                                                 ` Bart Schaefer
2013-11-18 16:58                                                   ` Peter Stephenson
2013-11-20 19:26                                       ` modify functions hierarchy (was: Install run-help and *.zwc files system wide in build system) Peter Stephenson
2013-11-21  6:01                                         ` Bart Schaefer
2013-11-23 17:48                                           ` Peter Stephenson
2013-11-23 19:47                                             ` Helpfiles again (was Re: modify functions hierarchy (was: etc.)) Bart Schaefer
2013-11-23 21:25                                               ` Martin Vaeth
2013-11-24  5:06                                                 ` Bart Schaefer
     [not found]                                                   ` <20131124175649.27c2559a@pws!>
2013-11-24 12:26                                                   ` Martin Vaeth
2013-11-24 12:31                                                     ` Martin Vaeth
2013-11-24 17:56                                                   ` Peter Stephenson
2013-11-25  8:18                                                     ` Bart Schaefer
2013-11-25 14:24                                                       ` Jun T.
2013-11-25 15:49                                                         ` Peter Stephenson
2013-11-25 16:56                                                           ` Bart Schaefer
2013-11-25 17:37                                                             ` Peter Stephenson
2013-11-25 20:14                                                             ` Phil Pennock
2013-11-26  9:24                                                               ` Peter Stephenson
2013-12-12 22:17                                                             ` PATCH: Util/helpfiles failing on old-fashioned unix pda
2013-12-13  9:08                                                               ` Martin Vaeth
2013-12-13 23:46                                                                 ` Martin Vaeth
2013-12-14  0:20                                                                 ` Wayne Davison
2013-12-14  9:23                                                                   ` Martin Vaeth
2013-12-14 18:49                                                                     ` Bart Schaefer
2013-12-15 23:37                                                                 ` pda
2013-12-18  8:39                                                                   ` Martin Vaeth
2013-11-17 20:13                                   ` Peter Stephenson [this message]
2013-11-17 20:50                                     ` modify functions hierarchy (was: Install run-help and *.zwc files system wide in build system) Bart Schaefer
2013-11-18  7:57                                       ` Martin Vaeth
2013-11-18 16:28                                         ` Bart Schaefer
2013-11-18 10:32                                       ` Peter Stephenson
2013-11-18 16:38                                         ` Bart Schaefer
2013-11-18 16:50                                           ` Peter Stephenson
2013-11-12 21:52                     ` Install run-help and *.zwc files system wide in build system Martin Vaeth
2013-11-13  1:13                       ` Bart Schaefer
2013-11-13  2:40                         ` Bart Schaefer
2013-11-13 13:30                 ` [PATCH] helpfiles: Also accept 'UTF-8' as an encoding name Jun T.
2013-11-13 15:37                   ` Jun T.
2013-11-13 16:56                     ` Martin Vaeth
2013-11-13 17:27                       ` Bart Schaefer
2013-11-13 18:28                         ` Martin Vaeth
2013-11-14  0:06                           ` Phil Pennock
2013-11-14  7:50                             ` Martin Vaeth
2013-11-14  8:18                               ` Phil Pennock
2013-11-14 11:11                                 ` Martin Vaeth
2013-11-15 14:58                                   ` Jun T.
2013-11-15 17:16                                     ` Bart Schaefer
2013-11-15 19:58                                     ` Martin Vaeth
2013-11-16  5:51                                       ` Bart Schaefer
2013-11-16 15:17                                         ` Jun T.
2013-11-16 18:05                                           ` Bart Schaefer
2013-11-18 18:43                                       ` Phil Pennock
2013-11-10 22:44   ` Install run-help and *.zwc files system wide in build system Martin Vaeth
2013-11-10 18:56 ` 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=20131117201359.1613ec4c@pws-pc.ntlworld.com \
    --to=p.w.stephenson@ntlworld.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).