| Message ID | 334218650.28135138.1579875742146@mail.yahoo.com |
|---|---|
| State | New |
| Headers | show |
>>>>> "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
diff --git a/gdbserver/Makefile.in b/gdbserver/Makefile.in index 902d54cfe5..4f9196227c 100644 --- a/gdbserver/Makefile.in +++ b/gdbserver/Makefile.in @@ -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; \ diff --git a/gdbserver/configure.ac b/gdbserver/configure.ac index 24b832b90e..b7ed8050fc 100644 --- a/gdbserver/configure.ac +++ b/gdbserver/configure.ac @@ -27,6 +27,7 @@ AC_PROG_CC  AC_PROG_CXX  AC_GNU_SOURCE  AC_SYS_LARGEFILE +AM_PROG_INSTALL_STRIP   AC_CANONICAL_SYSTEM Â