This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fix for: "/usr/bin/install: cannot stat `libinproctrace.so': No such file or directory"


On Thursday 02 September 2010 23:12:46, Paralkar Anmol-B07584 wrote:
> Hello Michael,
> 
>  Sorry! Please see the (attached) re-spun patch.

Hi!  Thanks for spotting this.

> *************** install: all install-only
> *** 167,173 ****
>   install-only:
>   	n=`echo gdbserver | sed '$(program_transform_name)'`; \
>   	if [ x$$n = x ]; then n=gdbserver; else true; fi; \
> ! 	if [ x$IPA_DEPFILES != x ]; then \
>   		$(SHELL) $(srcdir)/../../mkinstalldirs $(DESTDIR)$(libdir); \
>   		$(INSTALL_PROGRAM) $(IPA_LIB) $(DESTDIR)$(libdir)/$(IPA_LIB); \
>   	fi; \
> --- 167,173 ----
>   install-only:
>   	n=`echo gdbserver | sed '$(program_transform_name)'`; \
>   	if [ x$$n = x ]; then n=gdbserver; else true; fi; \
> ! 	if [ x$$IPA_DEPFILES != x ]; then \
>   		$(SHELL) $(srcdir)/../../mkinstalldirs $(DESTDIR)$(libdir); \
>   		$(INSTALL_PROGRAM) $(IPA_LIB) $(DESTDIR)$(libdir)/$(IPA_LIB); \
>   	fi; \


This however isn't correct.  The intent is not to make the variable reference visible
to the shell (it'd always be empty), but to expand it before the shell sees it.

I've applied this patch below to fix this, after confirming "make install" does install
the library on a amd64-linux build.  I'll apply this to the 7.2 branch as well shortly.

2010-09-06  Pedro Alves  <pedro@codesourcery.com>

	* Makefile.in (install-only): Replace $IPA_DEPFILES with
	"$(IPA_DEPFILES)".

---
 gdb/gdbserver/Makefile.in |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: src/gdb/gdbserver/Makefile.in
===================================================================
--- src.orig/gdb/gdbserver/Makefile.in	2010-09-06 11:28:19.000000000 +0100
+++ src/gdb/gdbserver/Makefile.in	2010-09-06 11:42:54.000000000 +0100
@@ -167,7 +167,7 @@ install: all install-only
 install-only:
 	n=`echo gdbserver | sed '$(program_transform_name)'`; \
 	if [ x$$n = x ]; then n=gdbserver; else true; fi; \
-	if [ x$IPA_DEPFILES != x ]; then \
+	if [ x"$(IPA_DEPFILES)" != x ]; then \
 		$(SHELL) $(srcdir)/../../mkinstalldirs $(DESTDIR)$(libdir); \
 		$(INSTALL_PROGRAM) $(IPA_LIB) $(DESTDIR)$(libdir)/$(IPA_LIB); \
 	fi; \


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]