]> xenbits.xensource.com Git - qemu-xen-4.5-testing.git/commitdiff
qemu-traditional: do not strip binaries during make install
authorOlaf Hering <olaf@aepfle.de>
Tue, 15 Oct 2013 09:42:26 +0000 (11:42 +0200)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Tue, 15 Oct 2013 14:42:16 +0000 (15:42 +0100)
It is wrong to strip code during make install, unless explicit
requested. Introduce a new variable INSTALL_PROG and use it along with
an optional STRIP_OPT where currently install -s -m 755 is used.
This is what upstream qemu offers in version 1.6.

Signed-off-by: Olaf Hering <olaf@aepfle.de>
Makefile
Makefile.target
configure

index 37c706667fb9427326286df0b39dcd7c6e56ee61..ed9b28a694c49ec7a31de0ecf9379f5d9d370d05 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -243,7 +243,7 @@ endif
 install: all $(if $(BUILD_DOCS),install-doc)
        mkdir -p "$(DESTDIR)$(bindir)"
 ifneq ($(TOOLS),)
-       $(INSTALL) -m 755 -s $(TOOLS) "$(DESTDIR)$(bindir)"
+       $(INSTALL_PROG) $(STRIP_OPT) $(TOOLS) "$(DESTDIR)$(bindir)"
 endif
 ifneq ($(BLOBS),)
        mkdir -p "$(DESTDIR)$(datadir)"
index 19bb0fd3caa0cf360bba2a0b2da0bf98b0832635..3c3db2b433d7453924f5442df6d63921740f5b0b 100644 (file)
@@ -755,7 +755,7 @@ clean:
 
 install: all install-hook
 ifneq ($(PROGS),)
-       $(INSTALL) -m 755 -s $(PROGS) "$(DESTDIR)$(bindir)"
+       $(INSTALL_PROG) $(STRIP_OPT) $(PROGS) "$(DESTDIR)$(bindir)"
 endif
 
 # Include automatically generated dependency files
index ace3c3e1fe89ac4c4e76fe795de64dabbf7c2694..454735981a3a2e004dfb2cb426c5e54cd01fcaac 100755 (executable)
--- a/configure
+++ b/configure
@@ -1215,6 +1215,7 @@ echo "docdir=\${prefix}$docsuffix" >> $config_mak
 echo "#define CONFIG_QEMU_SHAREDIR \"$prefix$datasuffix\"" >> $config_h
 echo "MAKE=$make" >> $config_mak
 echo "INSTALL=$install" >> $config_mak
+echo "INSTALL_PROG=$install -m 0755" >> $config_mak
 echo "CC=$cc" >> $config_mak
 echo "HOST_CC=$host_cc" >> $config_mak
 echo "AR=$ar" >> $config_mak