report after installing zsh-4.3.2

report after installing zsh-4.3.2

[Pierre Albarede]

Hi,

on Mac OS X 10.3.9, installation worked well except for error messages 
in
make install.info

info files were installed in
/usr/local/info/
but there was no zsh entry in the info directory.

I had to run
install-info zsh.info
in order to make zsh appear in the info directory.

I hope this helps.

Thanks.

Re: report after installing zsh-4.3.2

[Wayne Davison]

[-- Attachment #1: Type: text/plain, Size: 652 bytes --]

On Tue, Apr 11, 2006 at 06:33:21PM +0200, Pierre Albarede wrote:
> Debian install-info 1.10.21.  Copyright (C) 1994,1995
[...]
> cp: /usr/local/info/dir/dir: No such file or directory

I checked this version of install-info, and it doesn't implement the
--dir-file=FOO option correctly.  It treats it as an alias for
--info-dir when the former should require trailing /dir that the
latter does not want.

I've fixed the problem by switching over to using the --info-dir option
with the "/dir" suffix removed.  It would appear that the GNU version
and the Debian version both treat this option properly.

Thanks for pointing out the problem!

..wayne..

[-- Attachment #2: info.patch --]
[-- Type: text/plain, Size: 441 bytes --]

--- Doc/Makefile.in	20 Mar 2006 11:06:24 -0000	1.31
+++ Doc/Makefile.in	11 Apr 2006 17:38:38 -0000
@@ -289,7 +289,7 @@ install.info: texi
 	    done \
 	); then \
 	    if $(SHELL) -c 'install-info --version' >/dev/null 2>&1; then \
-	      install-info --dir-file=$(DESTDIR)$(infodir)/dir \
+	      install-info --info-dir=$(DESTDIR)$(infodir) \
 	        $(DESTDIR)$(infodir)/$(tzsh).info; \
 	    else true; fi; \
 	    rm -rf infodir; \

Re: report after installing zsh-4.3.2

[Pierre Albarede]

I wrote
> on Mac OS X 10.3.9, installation worked well except for error messages 
> in
> make install.info

Wayne
> I assume that your version of install-info doesn't understand the
> --version option?

Apparently it does:

Ordinateur-de-alba% install-info
Debian install-info 1.10.21.  Copyright (C) 1994,1995
Ian Jackson.  This is free software; see the GNU General Public Licence
version 2 or later for copying conditions.  There is NO warranty.

usage: install-info [--version] [--help] [--debug] [--maxwidth=nnn]
              [--section regexp title] [--infodir=xxx] [--align=nnn]
              [--calign=nnn] [--quiet] [--menuentry=xxx] [--info-dir=xxx]
              [--keep-old] [--description=xxx] [--test]
              [--remove | --remove-exactly ] [--dir-file]
              [--]
              filename


Here is a complete output (of a second run):

Ordinateur-de-alba% sudo make install.info
Password:
/bin/sh ../mkinstalldirs /usr/local/info
rm -rf infodir
mkdir infodir
if ( \
     sed '/^@setfilename/s|zsh|zsh|' \
         < ./zsh.texi > infodir/tzsh.texi && \
     (cd infodir && makeinfo tzsh.texi) && \
     for file in infodir/zsh.info*; do \
         /usr/bin/install -c -m 644 $file /usr/local/info || exit 1; \
     done \
); then \
     if /bin/sh -c 'install-info --version' >/dev/null 2>&1; then \
       install-info --dir-file=/usr/local/info/dir \
         /usr/local/info/zsh.info; \
     else true; fi; \
     rm -rf infodir; \
     exit 0; \
else \
     rm -rf infodir; \
     exit 1; \
fi
* ZSH: (zsh). The Z Shell Manual.
install-info(/usr/local/info/zsh.info): no file 
/usr/local/info/dir/dir, retrieving backup file 
/var/backups/infodir.bak.
cp: /usr/local/info/dir/dir: No such file or directory
install-info(/usr/local/info/zsh.info): copying 
/var/backups/infodir.bak to /usr/local/info/dir/dir failed, giving up: 
No such file or directory

Re: report after installing zsh-4.3.2

[Wayne Davison]

On Tue, Apr 11, 2006 at 03:42:08PM +0200, Pierre Albarede wrote:
> I had to run
> install-info zsh.info
> in order to make zsh appear in the info directory.

I assume that your version of install-info doesn't understand the
--version option?  The Doc/Makefile currently has that as a prerequisite
for using install-info:

    if $(SHELL) -c 'install-info --version' >/dev/null 2>&1; then \
      install-info --dir-file=$(DESTDIR)$(infodir)/dir \
        $(DESTDIR)$(infodir)/$(tzsh).info; \
    else true; fi

It looks like we need some improved logic there.  Perhaps just replacing
"true" with "(cd $(DESTDIR)$(infodir) && install-info $(tzsh).info)" and
letting that succeed or fail would be a good change?

..wayne..