Re: Guile 1.5.2 beta ready for testing. (and-let* etc)

[Bill Schottstaedt]

I noticed that the install command by itself (using cp for instance)
worked fine, but failed in the context of install-sh, so I added
set -x to install-sh and got (sorry for the verbosity...):

-------

.././install-sh -c -m 644 and-let-star-compat.scm \
           /usr/local/share/guile/1.5.2/ice-9/'and-let*.scm'
doit=
mvprog=mv
cpprog=cp
chmodprog=chmod
chownprog=chown
chgrpprog=chgrp
stripprog=strip
rmprog=rm
mkdirprog=mkdir
transformbasename=
transform_arg=
instcmd=mv
chmodcmd=chmod 0755
chowncmd=
chgrpcmd=
stripcmd=
rmcmd=rm -f
mvcmd=mv
src=
dst=
dir_arg=
+ [ x-c != x ]
instcmd=cp
+ shift
+ continue
+ [ x-m != x ]
chmodcmd=chmod 644
+ shift
+ shift
+ continue
+ [ xand-let-star-compat.scm != x ]
+ [ x = x ]
src=and-let-star-compat.scm
+ shift
+ continue
+ [ x/usr/local/share/guile/1.5.2/ice-9/and-let*.scm != x ]
+ [ xand-let-star-compat.scm = x ]
+ :
dst=/usr/local/share/guile/1.5.2/ice-9/and-let*.scm
+ shift
+ continue
+ [ x != x ]
+ [ xand-let-star-compat.scm = x ]
+ true
+ [ x != x ]
+ [ -f and-let-star-compat.scm -o -d and-let-star-compat.scm ]
+ true
+ [ x/usr/local/share/guile/1.5.2/ice-9/and-let*.scm = x ]
+ true
+ [ -d /usr/local/share/guile/1.5.2/ice-9/and-let-star.scm ]
+ true
+ echo /usr/local/share/guile/1.5.2/ice-9/and-let-star.scm
+ sed -e s,[^/]*$,,;s,/$,,;s,^$,.,
dstdir=/usr/local/share/guile/1.5.2/ice-9
+ [ ! -d /usr/local/share/guile/1.5.2/ice-9 ]
+ [ x != x ]
+ [ x = x ]
+ basename /usr/local/share/guile/1.5.2/ice-9/and-let-star.scm
dstfile=and-let-star.scm
+ [ xand-let-star.scm = x ]
+ true
dsttmp=/usr/local/share/guile/1.5.2/ice-9/#inst.19207#
+ cp and-let-star-compat.scm /usr/local/share/guile/1.5.2/ice-9/#inst.19207#
+ trap rm -f /usr/local/share/guile/1.5.2/ice-9/#inst.19207# 0
+ [ x != x ]
+ true
+ [ x != x ]
+ true
+ [ x != x ]
+ true
+ [ xchmod 644 != x ]
+ chmod 644 /usr/local/share/guile/1.5.2/ice-9/#inst.19207#
+ rm -f -f /usr/local/share/guile/1.5.2/ice-9/and-let-star.scm
+ mv /usr/local/share/guile/1.5.2/ice-9/#inst.19207# 
/usr/local/share/guile/1.5.2/ice-9/and-let-star.scm
+ exit 0
+ rm -f /usr/local/share/guile/1.5.2/ice-9/#inst.19207#
make[2]: Leaving directory `/space/home/bil/test/guile-1.5.2/ice-9'
make[1]: Leaving directory `/space/home/bil/test/guile-1.5.2/ice-9'
Making install in guile-config

-------

I'm not a shell script expert, but I guess that the line

+ [ x/usr/local/share/guile/1.5.2/ice-9/and-let*.scm = x ]

expanded the *. Using the version 'and-let\*.scm' ended up
with

-rw-r--r--   1 root     other        425 Sep 24 10:52 and-let\*.scm

and left the and-let-star.scm file intact.

Using instead the form and-let\*.scm overwrote and-let-star.scm.

So on the Sun I'm using, Makefile.am lines 41-43 (or is it Makefile.in?)
should probably be:

install-data-local:
        -$(INSTALL_DATA) and-let-star-compat.scm \
           $(subpkgdatadir)/'and-let\*.scm'

The SGI (Irix 5.3) behaved in the same way.

--------------------------------------------------------------------------------

On the makeinfo --force problem, here's what that version says:

/usr/people/bil/test/guile-1.5.2/ makeinfo --version
This is GNU Makeinfo version 1.63.
/usr/people/bil/test/guile-1.5.2/ makeinfo --force
makeinfo: unrecognized option `--force'
[etc]

But it's not a big deal -- I should update a lot of programs
on that machine.


By the way, I get these two warnings from the SGI C compiler:

cc -DHAVE_CONFIG_H -I.. -I./.. -I../libltdl    -g -c c-tokenize.c
cfe: Warning 624: c-tokenize.lex, line 31: Trailing comma in enumerator list
     COOKIE,
 ----------^

cfe: Warning 740: goops.c, line 1551: conflicting declarations for 
'scm_sym_args'
 static SCM scm_sym_args    ;
 -----------^
cfe: Warning 653: goops.c, line 1551: (previous declaration of 'scm_sym_args' 
at line 181 in file '../libguile/eval.h')
 static SCM scm_sym_args    ;
 -----------^