* [PATCH] Fix PR864, apply program_transform_name to ada tools
@ 2013-01-07 9:49 Richard Biener
2013-01-07 9:51 ` Arnaud Charlet
0 siblings, 1 reply; 2+ messages in thread
From: Richard Biener @ 2013-01-07 9:49 UTC (permalink / raw)
To: gcc-patches; +Cc: ebotcazou
This makes us unconditionally apply program_transform_name when installing
$(ADA_TOOLS). It's a patch distros have been applying for ages (now
I've updated it for 4.8 which meant re-writing it completely so I am
eligible to submit it).
Bootstrapped on x86_64-unknown-linux-gnu with --program-suffix=-4.8,
installed (and installation inspected) and tested (which uses the
unsuffixed names - names get changed at install time only).
Ok for trunk?
Thanks,
Richard.
2013-01-04 Richard Biener <rguenther@suse.de>
PR ada/864
* gcc-interface/Make-lang.in (ada.install-common): Always apply
program_transform_name.
Index: gcc/ada/gcc-interface/Make-lang.in
===================================================================
--- gcc/ada/gcc-interface/Make-lang.in (revision 194900)
+++ gcc/ada/gcc-interface/Make-lang.in (working copy)
@@ -736,14 +736,13 @@ ada.install-common:
-if [ -f gnat1$(exeext) ] ; \
then \
for tool in $(ADA_TOOLS) ; do \
+ install_name=`echo $$tool|sed '$(program_transform_name)'`$(exeext); \
+ $(RM) $(DESTDIR)$(bindir)/$$install_name; \
if [ -f $$tool-cross$(exeext) ] ; \
then \
- install_name=`echo $$tool|sed '$(program_transform_name)'`$(exeext); \
- $(RM) $(DESTDIR)$(bindir)/$$install_name; \
$(INSTALL_PROGRAM) $$tool-cross$(exeext) $(DESTDIR)$(bindir)/$$install_name; \
else \
- $(RM) $(DESTDIR)$(bindir)/$$tool$(exeext); \
- $(INSTALL_PROGRAM) $$tool$(exeext) $(DESTDIR)$(bindir)/$$tool$(exeext); \
+ $(INSTALL_PROGRAM) $$tool$(exeext) $(DESTDIR)$(bindir)/$$install_name; \
fi ; \
done; \
$(RM) $(DESTDIR)$(bindir)/gnatdll$(exeext); \
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix PR864, apply program_transform_name to ada tools
2013-01-07 9:49 [PATCH] Fix PR864, apply program_transform_name to ada tools Richard Biener
@ 2013-01-07 9:51 ` Arnaud Charlet
0 siblings, 0 replies; 2+ messages in thread
From: Arnaud Charlet @ 2013-01-07 9:51 UTC (permalink / raw)
To: Richard Biener; +Cc: gcc-patches, ebotcazou
> This makes us unconditionally apply program_transform_name when
> installing
> $(ADA_TOOLS). It's a patch distros have been applying for ages (now
> I've updated it for 4.8 which meant re-writing it completely so I am
> eligible to submit it).
>
> Bootstrapped on x86_64-unknown-linux-gnu with --program-suffix=-4.8,
> installed (and installation inspected) and tested (which uses the
> unsuffixed names - names get changed at install time only).
>
> Ok for trunk?
OK, thanks.
> 2013-01-04 Richard Biener <rguenther@suse.de>
>
> PR ada/864
> * gcc-interface/Make-lang.in (ada.install-common): Always apply
> program_transform_name.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-01-07 9:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-07 9:49 [PATCH] Fix PR864, apply program_transform_name to ada tools Richard Biener
2013-01-07 9:51 ` Arnaud Charlet
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 read-only IMAP folder(s) and NNTP newsgroup(s).