* PATCH: better configure-time handling of program-transform
@ 1999-12-02 22:13 Clint Adams
1999-12-03 9:16 ` Zefram
0 siblings, 1 reply; 3+ messages in thread
From: Clint Adams @ 1999-12-02 22:13 UTC (permalink / raw)
To: zsh-workers
This reduces the basename calculation to only once, which should make
everyone happier.
Index: Makefile.in
===================================================================
RCS file: /cvs/zsh/zsh/Makefile.in,v
retrieving revision 1.1.1.7
diff -u -r1.1.1.7 Makefile.in
--- Makefile.in 1999/11/30 21:10:36 1.1.1.7
+++ Makefile.in 1999/12/02 22:09:53
@@ -39,14 +39,14 @@
@DEFS_MK@
transform = @program_transform_name@
-tzsh = `echo zsh | sed "$(transform)"`
+tzsh = @tzsh@
# ========== DEPENDENCIES FOR BUILDING ==========
# default target
all: config.h
@for subdir in Src Doc; do \
- (cd $$subdir && $(MAKE) $(MAKEDEFS) tzsh=$(tzsh) $@) || exit 1; \
+ (cd $$subdir && $(MAKE) $(MAKEDEFS) $@) || exit 1; \
done
check test:
@@ -71,7 +71,7 @@
# install/uninstall just the binary
install.bin uninstall.bin:
- @cd Src && $(MAKE) $(MAKEDEFS) tzsh=$(tzsh) $@
+ @cd Src && $(MAKE) $(MAKEDEFS) $@
# install/uninstall just the modules
install.modules uninstall.modules:
@@ -88,7 +88,7 @@
# install/uninstall just the info pages
install.info uninstall.info:
- @cd Doc && $(MAKE) $(MAKEDEFS) tzsh=$(tzsh) $@
+ @cd Doc && $(MAKE) $(MAKEDEFS) $@
# install/uninstall just the html pages
install.html uninstall.html:
Index: configure.in
===================================================================
RCS file: /cvs/zsh/zsh/configure.in,v
retrieving revision 1.1.1.44
diff -u -r1.1.1.44 configure.in
--- configure.in 1999/11/30 18:16:49 1.1.1.44
+++ configure.in 1999/12/02 22:09:53
@@ -48,6 +48,15 @@
dnl Handle --program-prefix, --program-suffix, etc.
AC_ARG_PROGRAM
+u_ptn=`make -f - <<EOF
+all:
+ @echo ${program_transform_name}
+EOF
+`
+tzsh=`echo zsh | sed -e "${u_ptn}"`
+
+AC_SUBST(tzsh)dnl
+
dnl Do you want to debug zsh?
undefine([zsh-debug])dnl
AC_ARG_ENABLE(zsh-debug,
@@ -1601,5 +1610,6 @@
if test "$zshfndir" != no; then
echo "functions install path : ${zshfndir}
installed functions : ${FUNCTIONS_INSTALL}
+binary/info base name : ${tzsh}
"
fi
Index: Doc/Makefile.in
===================================================================
RCS file: /cvs/zsh/zsh/Doc/Makefile.in,v
retrieving revision 1.1.1.29
diff -u -r1.1.1.29 Makefile.in
--- Doc/Makefile.in 1999/12/02 19:19:54 1.1.1.29
+++ Doc/Makefile.in 1999/12/02 22:09:54
@@ -35,7 +35,7 @@
sdir_top = @top_srcdir@
INSTALL = @INSTALL@
-tzsh = zsh
+tzsh = @tzsh@
@DEFS_MK@
Index: Src/Makefile.in
===================================================================
RCS file: /cvs/zsh/zsh/Src/Makefile.in,v
retrieving revision 1.1.1.17
diff -u -r1.1.1.17 Makefile.in
--- Src/Makefile.in 1999/12/01 18:36:01 1.1.1.17
+++ Src/Makefile.in 1999/12/02 22:09:54
@@ -42,7 +42,7 @@
sdir_src = $(sdir)
dir_src = .
-tzsh = zsh
+tzsh = @tzsh@
# ========= DEPENDENCIES FOR BUILDING ==========
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: PATCH: better configure-time handling of program-transform
1999-12-02 22:13 PATCH: better configure-time handling of program-transform Clint Adams
@ 1999-12-03 9:16 ` Zefram
1999-12-03 15:12 ` Clint Adams
0 siblings, 1 reply; 3+ messages in thread
From: Zefram @ 1999-12-03 9:16 UTC (permalink / raw)
To: Clint Adams; +Cc: zsh-workers
Clint Adams wrote:
>+u_ptn=`make -f - <<EOF
>+all:
>+ @echo ${program_transform_name}
>+EOF
>+`
What's this about? What does it achieve that
eval "u_ptn=${program_transform_name}"
doesn't? Is the extra layer of expansion really necessary anyway?
-zefram
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: PATCH: better configure-time handling of program-transform
1999-12-03 9:16 ` Zefram
@ 1999-12-03 15:12 ` Clint Adams
0 siblings, 0 replies; 3+ messages in thread
From: Clint Adams @ 1999-12-03 15:12 UTC (permalink / raw)
To: Zefram; +Cc: Clint Adams, zsh-workers
> What's this about? What does it achieve that
>
> eval "u_ptn=${program_transform_name}"
>
> doesn't? Is the extra layer of expansion really necessary anyway?
Well, when $program_transform_name is s,$$,blah,
that would set u_ptn to something like "s,99999,blah,"
whereas it should be "s,$,blah,".
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~1999-12-03 15:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1999-12-02 22:13 PATCH: better configure-time handling of program-transform Clint Adams
1999-12-03 9:16 ` Zefram
1999-12-03 15:12 ` Clint Adams
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).