Move gdbserver to top level
Commit Message
Am Donnerstag, 23. Januar 2020, 22:55:20 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
> I think I will wait a little while longer before pushing.
make install-strip doesn't work in gdbserver (it worked before indirectly
when called from within gdb).
I made it work again with these changes:
From a3cc5583ca738daa3fdb5805d0c7ab0734d28f41 Mon Sep 17 00:00:00 2001
From: Hannes Domani <ssbssa@yahoo.de>
Date: Fri, 24 Jan 2020 15:03:14 +0100
Subject: [PATCH] gdbserver install-strip
---
gdbserver/Makefile.in | 10 ++++++++++
gdbserver/configure.ac | 1 +
2 files changed, 11 insertions(+)
--
2.15.1.windows.2
Comments
>>>>> "Hannes" == Hannes Domani via gdb-patches <gdb-patches@sourceware.org> writes:
Hannes> Am Donnerstag, 23. Januar 2020, 22:55:20 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
>> I think I will wait a little while longer before pushing.
Hannes> make install-strip doesn't work in gdbserver (it worked before indirectly
Hannes> when called from within gdb).
Thanks. I'll pull this in.
Maybe when the move is complete, and when gdbserver is not building its
own copies of the libraries, I'll switch gdbserver to use Automake.
This would provide a lot of these things for free.
Tom
Hannes> make install-strip doesn't work in gdbserver (it worked before indirectly
Hannes> when called from within gdb).
Tom> Thanks. I'll pull this in.
Actually, it seems like this could go in now.
Would you mind arranging that?
Tom
Am Freitag, 24. Januar 2020, 16:49:26 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
> Hannes> make install-strip doesn't work in gdbserver (it worked before indirectly
> Hannes> when called from within gdb).
>
> Tom> Thanks. I'll pull this in.
>
> Actually, it seems like this could go in now.
> Would you mind arranging that?
I'm not sure what you mean.
Pushing your patch?
>>>>> "Hannes" == Hannes Domani via gdb-patches <gdb-patches@sourceware.org> writes:
Hannes> Am Freitag, 24. Januar 2020, 16:49:26 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
Hannes> make install-strip doesn't work in gdbserver (it worked before indirectly
Hannes> when called from within gdb).
>>
Tom> Thanks. I'll pull this in.
>>
>> Actually, it seems like this could go in now.
>> Would you mind arranging that?
Hannes> I'm not sure what you mean.
Hannes> Pushing your patch?
Applying your "make install-strip" gdbserver to trunk.
Tom
Am Freitag, 24. Januar 2020, 17:24:34 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
> >>>>> "Hannes" == Hannes Domani via gdb-patches <gdb-patches@sourceware.org> writes:
>
> Hannes> Am Freitag, 24. Januar 2020, 16:49:26 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
> Hannes> make install-strip doesn't work in gdbserver (it worked before indirectly
> Hannes> when called from within gdb).
> >>
> Tom> Thanks. I'll pull this in.
> >>
> >> Actually, it seems like this could go in now.
> >> Would you mind arranging that?
>
> Hannes> I'm not sure what you mean.
>
> Hannes> Pushing your patch?
>
> Applying your "make install-strip" gdbserver to trunk.
Oooh, yes, that makes a lot more sense.
Ok, I will do that.
Regards
Hannes Domani
Am Freitag, 24. Januar 2020, 17:28:57 MEZ hat Hannes Domani via gdb-patches <gdb-patches@sourceware.org> Folgendes geschrieben:
> Am Freitag, 24. Januar 2020, 17:24:34 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
>
> > >>>>> "Hannes" == Hannes Domani via gdb-patches <gdb-patches@sourceware.org> writes:
> >
> > Hannes> Am Freitag, 24. Januar 2020, 16:49:26 MEZ hat Tom Tromey <tom@tromey.com> Folgendes geschrieben:
> > Hannes> make install-strip doesn't work in gdbserver (it worked before indirectly
> > Hannes> when called from within gdb).
> > >>
> > Tom> Thanks. I'll pull this in.
> > >>
> > >> Actually, it seems like this could go in now.
> > >> Would you mind arranging that?
> >
> > Hannes> I'm not sure what you mean.
> >
> > Hannes> Pushing your patch?
> >
> > Applying your "make install-strip" gdbserver to trunk.
>
> Oooh, yes, that makes a lot more sense.
>
> Ok, I will do that.
I've pushed this now.
Regards
Hannes Domani
@@ -45,11 +45,14 @@ htmldir = $(prefix)/html
 includedir = @includedir@
 CONFIG_SRC_SUBDIR = @CONFIG_SRC_SUBDIR@
Â
+install_sh = @install_sh@
+
 SHELL = @SHELL@
 EXEEXT = @EXEEXT@
Â
 INSTALL = @INSTALL@
 INSTALL_PROGRAM = @INSTALL_PROGRAM@
+INSTALL_STRIP_PROGRAM = @INSTALL_STRIP_PROGRAM@
 INSTALL_DATA = @INSTALL_DATA@
 RANLIB = @RANLIB@
Â
@@ -58,6 +61,7 @@ CXX = @CXX@
 CXX_DIALECT = @CXX_DIALECT@
 AR = @AR@
 AR_FLAGS = rc
+STRIP = @STRIP@
Â
 # Dependency tracking information.
 DEPMODE = @CCDEPMODE@
@@ -394,6 +398,12 @@ install-only:
    # in libiberty.
    @$(MAKE) $(FLAGS_TO_PASS) DO=install "DODIRS=$(SUBDIRS)" subdir_do
Â
+install-strip:
+Â Â Â $(MAKE) $(FLAGS_TO_PASS) INSTALL_PROGRAM="$(INSTALL_STRIP_PROGRAM)" \
+Â Â Â Â install_sh_PROGRAM="$(INSTALL_STRIP_PROGRAM)" INSTALL_STRIP_FLAG=-s \
+Â Â Â Â `test -z '$(STRIP)' || \
+Â Â Â Â Â Â echo "INSTALL_PROGRAM_ENV=STRIPPROG='$(STRIP)'"` install-only
+
 uninstall: force
    n=`echo gdbserver | sed '$(program_transform_name)'`; \
    if [ x$$n = x ]; then n=gdbserver; else true; fi; \
@@ -27,6 +27,7 @@ AC_PROG_CC
 AC_PROG_CXX
 AC_GNU_SOURCE
 AC_SYS_LARGEFILE
+AM_PROG_INSTALL_STRIP
Â
 AC_CANONICAL_SYSTEM
Â