From: "Theodore A. Roth" <troth@openavr.org>
To: Jim Blandy <jimb@redhat.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: RFC: Use program_transform_name correctly
Date: Wed, 08 Oct 2003 19:52:00 -0000 [thread overview]
Message-ID: <Pine.LNX.4.53.0310081249520.19819@knuth.amplepower.com> (raw)
In-Reply-To: <vt2d6d7edcn.fsf@zenia.home>
On Wed, 8 Oct 2003, Jim Blandy wrote:
>
> Could someone proofread this boring patch very carefully?
As long as gdb is using a autoconf-2.13 generated configure script, I
think that this will still have a problem when the user tries to pass
both --program-prefix=PREFIX and --program-suffix=SUFFIX to configure
at the same time. That was why I had added the double quotes around
$$t in the install-only rule.
Ted Roth
>
> gdb/Makefile.in:
> 2003-10-08 Jim Blandy <jimb@redhat.com>
>
> * Makefile.in (CC_FOR_TARGET, CXX_FOR_TARGET, install-only): Use
> program_transform_name properly.
> * nlm/Makefile.in (CC_FOR_TARGET, NLMCONV_FOR_TARGET): Same.
>
> gdb/gdbserver/Makefile.in:
> 2003-10-08 Jim Blandy <jimb@redhat.com>
>
> * Makefile.in (install-only, uninstall): Use
> program_transform_name properly.
>
> gdb/testsuite/Makefile.in:
> 2003-10-08 Jim Blandy <jimb@redhat.com>
>
> * Makefile.in (RUNTEST_FOR_TARGET): Use program_transform_name
> properly.
>
> Index: gdb/Makefile.in
> ===================================================================
> RCS file: /cvs/src/src/gdb/Makefile.in,v
> retrieving revision 1.454
> diff -c -r1.454 Makefile.in
> *** gdb/Makefile.in 8 Oct 2003 02:41:49 -0000 1.454
> --- gdb/Makefile.in 8 Oct 2003 19:22:39 -0000
> ***************
> *** 465,471 ****
> if [ "$(host_canonical)" = "$(target_canonical)" ] ; then \
> echo $(CC); \
> else \
> ! t='$(program_transform_name)'; echo gcc | sed -e '' $$t; \
> fi; \
> fi`
>
> --- 465,471 ----
> if [ "$(host_canonical)" = "$(target_canonical)" ] ; then \
> echo $(CC); \
> else \
> ! t='$(program_transform_name)'; echo gcc | sed -e $$t; \
> fi; \
> fi`
>
> ***************
> *** 481,487 ****
> if [ "$(host_canonical)" = "$(target_canonical)" ] ; then \
> echo $(CXX); \
> else \
> ! t='$(program_transform_name)'; echo gcc | sed -e '' $$t; \
> fi; \
> fi`
>
> --- 481,487 ----
> if [ "$(host_canonical)" = "$(target_canonical)" ] ; then \
> echo $(CXX); \
> else \
> ! t='$(program_transform_name)'; echo gcc | sed -e $$t; \
> fi; \
> fi`
>
> ***************
> *** 963,969 ****
> install: all install-only
> install-only: $(CONFIG_INSTALL)
> transformed_name=`t='$(program_transform_name)'; \
> ! echo gdb | sed -e "$$t"` ; \
> if test "x$$transformed_name" = x; then \
> transformed_name=gdb ; \
> else \
> --- 963,969 ----
> install: all install-only
> install-only: $(CONFIG_INSTALL)
> transformed_name=`t='$(program_transform_name)'; \
> ! echo gdb | sed -e $$t` ; \
> if test "x$$transformed_name" = x; then \
> transformed_name=gdb ; \
> else \
> Index: gdb/gdbserver/Makefile.in
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbserver/Makefile.in,v
> retrieving revision 1.24
> diff -c -r1.24 Makefile.in
> *** gdb/gdbserver/Makefile.in 8 Aug 2003 17:30:36 -0000 1.24
> --- gdb/gdbserver/Makefile.in 8 Oct 2003 19:22:40 -0000
> ***************
> *** 149,155 ****
> # install-only is intended to address that need.
> install: all install-only
> install-only:
> ! n=`echo gdbserver | sed '$(program_transform_name)'`; \
> if [ x$$n = x ]; then n=gdbserver; else true; fi; \
> $(SHELL) $(srcdir)/../../mkinstalldirs $(DESTDIR)$(bindir); \
> $(INSTALL_PROGRAM) gdbserver $(DESTDIR)$(bindir)/$$n; \
> --- 149,155 ----
> # install-only is intended to address that need.
> install: all install-only
> install-only:
> ! n=`t='$(program_transform_name)'; echo gdbserver | sed $$t`; \
> if [ x$$n = x ]; then n=gdbserver; else true; fi; \
> $(SHELL) $(srcdir)/../../mkinstalldirs $(DESTDIR)$(bindir); \
> $(INSTALL_PROGRAM) gdbserver $(DESTDIR)$(bindir)/$$n; \
> ***************
> *** 157,163 ****
> $(INSTALL_DATA) $(srcdir)/gdbserver.1 $(DESTDIR)$(man1dir)/$$n.1
>
> uninstall: force
> ! n=`echo gdbserver | sed '$(program_transform_name)'`; \
> if [ x$$n = x ]; then n=gdbserver; else true; fi; \
> rm -f $(bindir)/$$n $(DESTDIR)$(man1dir)/$$n.1
>
> --- 157,163 ----
> $(INSTALL_DATA) $(srcdir)/gdbserver.1 $(DESTDIR)$(man1dir)/$$n.1
>
> uninstall: force
> ! n=`t='$(program_transform_name)'; echo gdbserver | sed $$t`; \
> if [ x$$n = x ]; then n=gdbserver; else true; fi; \
> rm -f $(bindir)/$$n $(DESTDIR)$(man1dir)/$$n.1
>
> Index: gdb/nlm/Makefile.in
> ===================================================================
> RCS file: /cvs/src/src/gdb/nlm/Makefile.in,v
> retrieving revision 1.2
> diff -c -r1.2 Makefile.in
> *** gdb/nlm/Makefile.in 6 Mar 2001 08:21:46 -0000 1.2
> --- gdb/nlm/Makefile.in 8 Oct 2003 19:22:40 -0000
> ***************
> *** 55,68 ****
> if [ -f ../../gcc/xgcc ] ; then \
> echo ../../gcc/xgcc -B../../gcc/; \
> else \
> ! t='$(program_transform_name)'; echo gcc | sed -e '' $$t; \
> fi`
>
> NLMCONV_FOR_TARGET = ` \
> if [ -f ../../binutils/nlmconv ] ; then \
> echo ../../binutils/nlmconv; \
> else \
> ! t='$(program_transform_name)'; echo nlmconv | sed -e '' $$t; \
> fi`
>
> # All the includes used for CFLAGS and for lint.
> --- 55,68 ----
> if [ -f ../../gcc/xgcc ] ; then \
> echo ../../gcc/xgcc -B../../gcc/; \
> else \
> ! t='$(program_transform_name)'; echo gcc | sed -e $$t; \
> fi`
>
> NLMCONV_FOR_TARGET = ` \
> if [ -f ../../binutils/nlmconv ] ; then \
> echo ../../binutils/nlmconv; \
> else \
> ! t='$(program_transform_name)'; echo nlmconv | sed -e $$t; \
> fi`
>
> # All the includes used for CFLAGS and for lint.
> Index: gdb/testsuite/Makefile.in
> ===================================================================
> RCS file: /cvs/src/src/gdb/testsuite/Makefile.in,v
> retrieving revision 1.8
> diff -c -r1.8 Makefile.in
> *** gdb/testsuite/Makefile.in 23 Aug 2003 03:55:58 -0000 1.8
> --- gdb/testsuite/Makefile.in 8 Oct 2003 19:22:48 -0000
> ***************
> *** 53,59 ****
> if [ "$(host_canonical)" = "$(target_canonical)" ]; then \
> echo runtest; \
> else \
> ! t='$(program_transform_name)'; echo runtest | sed -e '' $$t; \
> fi; \
> fi`
>
> --- 53,59 ----
> if [ "$(host_canonical)" = "$(target_canonical)" ]; then \
> echo runtest; \
> else \
> ! t='$(program_transform_name)'; echo runtest | sed -e $$t; \
> fi; \
> fi`
>
>
next prev parent reply other threads:[~2003-10-08 19:52 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-10-07 21:42 Jim Blandy
2003-10-07 21:50 ` Daniel Jacobowitz
2003-10-07 22:03 ` Theodore A. Roth
2003-10-07 22:53 ` Daniel Jacobowitz
2003-10-07 23:59 ` Felix Lee
2003-10-08 1:05 ` Daniel Jacobowitz
2003-10-08 19:15 ` Jim Blandy
2003-10-08 19:18 ` Jim Blandy
2003-10-08 19:31 ` Daniel Jacobowitz
2003-10-08 19:51 ` Andreas Schwab
2003-10-08 21:22 ` Felix Lee
2003-10-08 19:32 ` Jim Blandy
2003-10-08 19:52 ` Theodore A. Roth [this message]
2003-10-08 21:32 ` Felix Lee
2003-10-09 2:09 ` Daniel Jacobowitz
2003-10-08 19:14 ` Jim Blandy
2003-10-07 22:48 ` Felix Lee
2003-10-08 17:44 ` Jim Blandy
-- strict thread matches above, loose matches on Subject: below --
2003-01-29 22:48 [RFC] Move ``length'' from struct main_type to struct type Kevin Buettner
2003-01-29 23:14 ` Daniel Jacobowitz
[not found] ` <drow@mvista.com>
2003-01-30 0:57 ` Kevin Buettner
2003-02-07 21:45 ` Kevin Buettner
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=Pine.LNX.4.53.0310081249520.19819@knuth.amplepower.com \
--to=troth@openavr.org \
--cc=gdb-patches@sources.redhat.com \
--cc=jimb@redhat.com \
/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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox