[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#928100: ebook2cwgui FTCBFS: strips with the wrong strip



Source: ebook2cwgui
Version: 0.1.2-4
Tags: patch upstream
User: helmutg@debian.org
Usertags: rebootstrap

ebook2cwgui fails to cross build from source, because it uses the wrong
strip via install -s. This also happens to break
DEB_BUILD_OPTIONS=nostrip as well as generation of a -dbgsym package. If
the Makefile made install substitutable, debhelper could take stripping
in its own hands. The attached patch implements that and enables cross
building a -dbgsym package or a nostrip package. Please consider
applying it.

Helmut
--- ebook2cwgui-0.1.2.orig/Makefile
+++ ebook2cwgui-0.1.2/Makefile
@@ -5,6 +5,8 @@
 VERSION=0.1.2
 override DESTDIR:=$(DESTDIR)/usr
 
+INSTALL ?= install
+
 all: ebook2cwgui
 
 ebook2cwgui:
@@ -16,14 +18,14 @@
 	       	-Wall `/wx/wx-config --libs --cxxflags` -o ebook2cwgui
 
 install:
-	install -d -v                      $(DESTDIR)/bin/
-	install -d -v                      $(DESTDIR)/share/pixmaps/
-	install -d -v                      $(DESTDIR)/share/man/man1/
-	install -d -v                      $(DESTDIR)/share/applications/
-	install -s -m 0755 ebook2cwgui     $(DESTDIR)/bin/
-	install    -m 0644 ebook2cw.xpm    $(DESTDIR)/share/pixmaps/
-	install    -m 0644 ebook2cwgui.1   $(DESTDIR)/share/man/man1/
-	install    -m 0644 ebook2cwgui.desktop  $(DESTDIR)/share/applications/
+	$(INSTALL) -d -v                      $(DESTDIR)/bin/
+	$(INSTALL) -d -v                      $(DESTDIR)/share/pixmaps/
+	$(INSTALL) -d -v                      $(DESTDIR)/share/man/man1/
+	$(INSTALL) -d -v                      $(DESTDIR)/share/applications/
+	$(INSTALL) -s -m 0755 ebook2cwgui     $(DESTDIR)/bin/
+	$(INSTALL)    -m 0644 ebook2cw.xpm    $(DESTDIR)/share/pixmaps/
+	$(INSTALL)    -m 0644 ebook2cwgui.1   $(DESTDIR)/share/man/man1/
+	$(INSTALL)    -m 0644 ebook2cwgui.desktop  $(DESTDIR)/share/applications/
 	
 uninstall:
 	rm -f $(DESTDIR)/bin/ebook2cwgui

Reply to: