[DVDnav-discuss] [PATCH] destdir handling fix

[Dominik 'Rathann' Mierzejewski]

Hi!

The attached patch needs to be applied. Otherwise if you use
make DESTDIR=/some/path
the specified path ends up in pkgconfig file, which is wrong.
DESTDIR should only ever be used upon installing the files.

After it's applied, I'd like to propose making a new (4.1.2?) release.
I'll prepare the tarball and write a news entry for MPlayer page if you
like.

Regards,
R.

-- 
Fedora contributor http://fedoraproject.org/wiki/DominikMierzejewski
Livna contributor http://rpm.livna.org MPlayer developer http://mplayerhq.hu
"Faith manages."
        -- Delenn to Lennier in Babylon 5:"Confessions and Lamentations"
-------------- next part --------------
Index: configure2
===================================================================
--- configure2	(revision 997)
+++ configure2	(working copy)
@@ -127,11 +127,11 @@
 cat > config.mak << EOF 
 # Automatically generated by configure, do not edit
 DESTDIR =
-PREFIX=\$(DESTDIR)$PREFIX
-libdir=\$(DESTDIR)$libdir
-shlibdir=\$(DESTDIR)$shlibdir
-incdir=\$(DESTDIR)$incdir
-dvdread_incdir=\$(DESTDIR)$dvdread_incdir
+PREFIX=$PREFIX
+libdir=$libdir
+shlibdir=$shlibdir
+incdir=$incdir
+dvdread_incdir=$dvdread_incdir
 THREADLIB=$threadlib
 BUILD_SHARED=$SHARED
 BUILD_STATIC=$STATIC
Index: Makefile
===================================================================
--- Makefile	(revision 997)
+++ Makefile	(working copy)
@@ -120,45 +120,45 @@
 # Install targets
 
 install-headers:
-	install -d $(incdir)
-	install -m 644 $(HEADERS) $(incdir)
+	install -d $(DESTDIR)$(incdir)
+	install -m 644 $(HEADERS) $(DESTDIR)$(incdir)
 ifeq ($(DVDREAD),internal)
-	install -d $(dvdread_incdir)
-	install -m 644 $(DVDREAD_HEADERS) $(dvdread_incdir)
+	install -d $(DESTDIR)$(dvdread_incdir)
+	install -m 644 $(DVDREAD_HEADERS) $(DESTDIR)$(dvdread_incdir)
 endif
 
 install-shared: $(SHLIB)
-	install -d $(shlibdir)
+	install -d $(DESTDIR)$(shlibdir)
 
 	install $(INSTALLSTRIP) -m 755 $(.OBJDIR)/$(SHLIB) \
-		$(shlibdir)/$(SHLIB).$(SHLIB_VERSION)
+		$(DESTDIR)$(shlibdir)/$(SHLIB).$(SHLIB_VERSION)
 	install $(INSTALLSTRIP) -m 755 $(.OBJDIR)/$(MINI_SHLIB) \
-		$(shlibdir)/$(MINI_SHLIB).$(SHLIB_VERSION)
+		$(DESTDIR)$(shlibdir)/$(MINI_SHLIB).$(SHLIB_VERSION)
 
-	cd $(shlibdir) && \
+	cd $(DESTDIR)$(shlibdir) && \
 		ln -sf $(SHLIB).$(SHLIB_VERSION) $(SHLIB).$(SHLIB_MAJOR)
-	cd $(shlibdir) && \
+	cd $(DESTDIR)$(shlibdir) && \
 		ln -sf $(MINI_SHLIB).$(SHLIB_VERSION) $(MINI_SHLIB).$(SHLIB_MAJOR)
-	cd $(shlibdir) && \
+	cd $(DESTDIR)$(shlibdir) && \
 		ln -sf $(SHLIB).$(SHLIB_MAJOR) $(SHLIB)
-	cd $(shlibdir) && \
+	cd $(DESTDIR)$(shlibdir) && \
 		ln -sf $(MINI_SHLIB).$(SHLIB_MAJOR) $(MINI_SHLIB)
 
 ifeq ($(DVDREAD),internal)
 	install $(INSTALLSTRIP) -m 755 $(.OBJDIR)/$(DVDREAD_SHLIB) \
-		$(shlibdir)/$(DVDREAD_SHLIB).$(SHLIB_VERSION)
-	cd $(shlibdir) && \
+		$(DESTDIR)$(shlibdir)/$(DVDREAD_SHLIB).$(SHLIB_VERSION)
+	cd $(DESTDIR)$(shlibdir) && \
 		ln -sf $(DVDREAD_SHLIB).$(SHLIB_VERSION) $(DVDREAD_SHLIB).$(SHLIB_MAJOR)
-	cd $(shlibdir) && \
+	cd $(DESTDIR)$(shlibdir) && \
 		ln -sf $(DVDREAD_SHLIB).$(SHLIB_MAJOR) $(DVDREAD_SHLIB)
 endif
 
 install-static: $(LIB)
-	install -d $(libdir)
+	install -d $(DESTDIR)$(libdir)
 
-	install $(INSTALLSTRIP) -m 755 $(.OBJDIR)/$(LIB) $(libdir)/$(LIB)
+	install $(INSTALLSTRIP) -m 755 $(.OBJDIR)/$(LIB) $(DESTDIR)$(libdir)/$(LIB)
 ifeq ($(DVDREAD),internal)
-	install $(INSTALLSTRIP) -m 755 $(.OBJDIR)/$(DVDREAD_LIB) $(libdir)/$(DVDREAD_LIB)
+	install $(INSTALLSTRIP) -m 755 $(.OBJDIR)/$(DVDREAD_LIB) $(DESTDIR)$(libdir)/$(DVDREAD_LIB)
 endif
 
 
@@ -185,8 +185,8 @@
 	chmod 0755 $(.OBJDIR)/dvdnav-config
 
 install-dvdnav-config: dvdnav-config
-	install -d $(PREFIX)/bin
-	install -m 0755 $(.OBJDIR)/dvdnav-config $(PREFIX)/bin/dvdnav-config
+	install -d $(DESTDIR)$(PREFIX)/bin
+	install -m 0755 $(.OBJDIR)/dvdnav-config $(DESTDIR)$(PREFIX)/bin/dvdnav-config
 
 
 vpath %.so ${.OBJDIR}