Fri Mar 18 12:42:14 2011 UTC ()
Fix patching on Mac OSX; fix installing the library; fix some pkglint issues


(adam)
diff -r1.25 -r1.26 pkgsrc/multimedia/libmatroska/Makefile
diff -r1.19 -r1.20 pkgsrc/multimedia/libmatroska/buildlink3.mk
diff -r1.18 -r1.19 pkgsrc/multimedia/libmatroska/distinfo
diff -r1.10 -r1.11 pkgsrc/multimedia/libmatroska/patches/patch-ac

cvs diff -r1.25 -r1.26 pkgsrc/multimedia/libmatroska/Makefile (expand / switch to context diff)
--- pkgsrc/multimedia/libmatroska/Makefile 2011/02/23 15:41:48 1.25
+++ pkgsrc/multimedia/libmatroska/Makefile 2011/03/18 12:42:13 1.26
@@ -1,5 +1,4 @@
-# $NetBSD: Makefile,v 1.25 2011/02/23 15:41:48 apb Exp $
-#
+# $NetBSD: Makefile,v 1.26 2011/03/18 12:42:13 adam Exp $
 
 DISTNAME=	libmatroska-1.1.0
 CATEGORIES=	multimedia devel
@@ -9,24 +8,28 @@
 MAINTAINER=	salo@NetBSD.org
 HOMEPAGE=	http://www.matroska.org/
 COMMENT=	Extensible open standard Audio/Video container format
+LICENSE=	gnu-lgpl-v2.1
 
 PKG_DESTDIR_SUPPORT=	user-destdir
 PKG_INSTALLATION_TYPES=	overwrite pkgviews
 
-USE_TOOLS+=	gmake
-USE_LANGUAGES=	c++
-USE_LIBTOOL=	yes
+USE_LANGUAGES=		c++
+USE_LIBTOOL=		yes
+USE_TOOLS+=		gmake
 
-NO_CONFIGURE=	yes
-MAKEFLAGS+=	prefix=${PREFIX:Q}
-MAKE_ENV+=	CXXFLAGS=${CXXFLAGS:M*:Q} RANLIB=${RANLIB:Q}
-BUILD_DIRS=	make/linux
-BUILD_TARGET=	sharedlib
-INSTALL_TARGET=	install_headers install_sharedlib
-TEST_TARGET=	test
+NO_CONFIGURE=		yes
+MAKEFLAGS+=		prefix=${PREFIX}
+MAKE_ENV+=		CXXFLAGS=${CXXFLAGS:Q} RANLIB=${RANLIB:Q}
+BUILD_DIRS=		make/linux
+BUILD_TARGET=		sharedlib
+INSTALL_TARGET=		install_headers install_sharedlib
+TEST_TARGET=		test
 
-# needed for libtoolization
-BUILDLINK_API_DEPENDS.libebml+=	libebml>=0.8
-.include "../../devel/libebml/buildlink3.mk"
+# remove CRs: fixes patching on Darwin and stops pkglint from complaining
+pre-patch:
+	cd ${WRKSRC}/make/linux && \
+		${TR} -d '\r' < Makefile > Makefile.new && \
+		${MV} Makefile.new Makefile
 
+.include "../../devel/libebml/buildlink3.mk"
 .include "../../mk/bsd.pkg.mk"

cvs diff -r1.19 -r1.20 pkgsrc/multimedia/libmatroska/buildlink3.mk (expand / switch to context diff)
--- pkgsrc/multimedia/libmatroska/buildlink3.mk 2011/02/14 19:00:45 1.19
+++ pkgsrc/multimedia/libmatroska/buildlink3.mk 2011/03/18 12:42:13 1.20
@@ -1,7 +1,4 @@
-# $NetBSD: buildlink3.mk,v 1.19 2011/02/14 19:00:45 drochner Exp $
-#
-# This Makefile fragment is included by packages that use libmatroska.
-#
+# $NetBSD: buildlink3.mk,v 1.20 2011/03/18 12:42:13 adam Exp $
 
 BUILDLINK_TREE+=	libmatroska
 
@@ -9,7 +6,7 @@
 LIBMATROSKA_BUILDLINK3_MK:=
 
 BUILDLINK_API_DEPENDS.libmatroska+=	libmatroska>=0.8.0
-BUILDLINK_ABI_DEPENDS.libmatroska?=	libmatroska>=1.1.0
+BUILDLINK_ABI_DEPENDS.libmatroska+=	libmatroska>=1.1.0
 BUILDLINK_PKGSRCDIR.libmatroska?=	../../multimedia/libmatroska
 
 .include "../../devel/libebml/buildlink3.mk"

cvs diff -r1.18 -r1.19 pkgsrc/multimedia/libmatroska/distinfo (expand / switch to context diff)
--- pkgsrc/multimedia/libmatroska/distinfo 2011/02/14 19:00:45 1.18
+++ pkgsrc/multimedia/libmatroska/distinfo 2011/03/18 12:42:13 1.19
@@ -1,6 +1,6 @@
-$NetBSD: distinfo,v 1.18 2011/02/14 19:00:45 drochner Exp $
+$NetBSD: distinfo,v 1.19 2011/03/18 12:42:13 adam Exp $
 
 SHA1 (libmatroska-1.1.0.tar.bz2) = ffc7d0b757854f9eb8c32e51f2fedadc796ef617
 RMD160 (libmatroska-1.1.0.tar.bz2) = 8da0bd5bd83e6b158b714b554f35f82c05181e9d
 Size (libmatroska-1.1.0.tar.bz2) = 80469 bytes
-SHA1 (patch-ac) = 69e511efd34597894a926d830549f09a3a449ad6
+SHA1 (patch-ac) = 8f7dd83d9ff04b29a918f97e65ef64a49310b6b6

cvs diff -r1.10 -r1.11 pkgsrc/multimedia/libmatroska/patches/Attic/patch-ac (expand / switch to context diff)
--- pkgsrc/multimedia/libmatroska/patches/Attic/patch-ac 2011/02/14 19:00:45 1.10
+++ pkgsrc/multimedia/libmatroska/patches/Attic/patch-ac 2011/03/18 12:42:14 1.11
@@ -1,87 +1,87 @@
-$NetBSD: patch-ac,v 1.10 2011/02/14 19:00:45 drochner Exp $
+$NetBSD: patch-ac,v 1.11 2011/03/18 12:42:14 adam Exp $
 
 --- make/linux/Makefile.orig	2011-01-29 11:48:16.000000000 +0000
 +++ make/linux/Makefile
-@@ -16,16 +16,16 @@ prefix=/boot/home/config
- else
- prefix=/usr/local
- endif
--libdir=$(prefix)/lib
--includedir=$(prefix)/include/matroska
-+libdir=$(PREFIX)/lib
-+includedir=$(PREFIX)/include/matroska
- 
- # Programs
- CROSS =
--CXX = $(CROSS)g++
-+#CXX = $(CROSS)g++
- LD = $(CXX)
--AR = $(CROSS)ar
--RANLIB = $(CROSS)ranlib
--INSTALL = install
-+#AR = $(CROSS)ar
-+#RANLIB = $(CROSS)ranlib
-+INSTALL = $(BSD_INSTALL)
- INSTALL_OPTS = -m 644
- INSTALL_OPTS_LIB = -m 644
- INSTALL_DIR_OPTS = -m 755
-@@ -53,7 +53,7 @@ MUX_LIBS=-lmatroska -lebml $(LIBICONV)
- 
- # Names
- LIBRARY=libmatroska.a
--LIBRARY_SO=libmatroska.so
-+LIBRARY_SO=libmatroska.la
- LIBRARY_SO_VER=libmatroska.so.3
- 
- # source-files
-@@ -92,16 +92,14 @@ lib:
- 	$(CXX) -c $(COMPILEFLAGS) -o $@ $<
- 
- %.lo: %$(EXTENSION)
--	$(CXX) -c $(COMPILEFLAGS) -fPIC -o $@ $<
-+	$(LIBTOOL) --mode=compile $(CXX) -c $(COMPILEFLAGS) -o $@ $<
- 
- $(LIBRARY): $(objects)
- 	$(AR) rcvu $@ $(objects)
- 	$(RANLIB) $@
- 
- $(LIBRARY_SO): $(objects_so)
--	$(CXX) -shared -Wl,-soname,$(LIBRARY_SO_VER) -o $(LIBRARY_SO_VER) $(objects_so) -lebml
--	rm -f $(LIBRARY_SO)
--	ln -s $(LIBRARY_SO_VER) $(LIBRARY_SO)
-+	$(LIBTOOL) --mode=link $(CXX) -o $(LIBRARY_SO) $(objects_so) -lebml -rpath $(PREFIX)/lib -version-info 0
- 
- clean:	cleantest
- 	rm -f $(objects) $(objects_so)
+@@ -16,16 +16,16 @@ prefix=/boot/home/config
+ else
+ prefix=/usr/local
+ endif
+-libdir=$(prefix)/lib
+-includedir=$(prefix)/include/matroska
++libdir=$(PREFIX)/lib
++includedir=$(PREFIX)/include/matroska
+ 
+ # Programs
+ CROSS =
+-CXX = $(CROSS)g++
++#CXX = $(CROSS)g++
+ LD = $(CXX)
+-AR = $(CROSS)ar
+-RANLIB = $(CROSS)ranlib
+-INSTALL = install
++#AR = $(CROSS)ar
++#RANLIB = $(CROSS)ranlib
++INSTALL = $(BSD_INSTALL)
+ INSTALL_OPTS = -m 644
+ INSTALL_OPTS_LIB = -m 644
+ INSTALL_DIR_OPTS = -m 755
+@@ -53,7 +53,7 @@ MUX_LIBS=-lmatroska -lebml $(LIBICONV)
+ 
+ # Names
+ LIBRARY=libmatroska.a
+-LIBRARY_SO=libmatroska.so
++LIBRARY_SO=libmatroska.la
+ LIBRARY_SO_VER=libmatroska.so.3
+ 
+ # source-files
+@@ -92,16 +92,14 @@ lib:
+ 	$(CXX) -c $(COMPILEFLAGS) -o $@ $<
+ 
+ %.lo: %$(EXTENSION)
+-	$(CXX) -c $(COMPILEFLAGS) -fPIC -o $@ $<
++	$(LIBTOOL) --mode=compile $(CXX) -c $(COMPILEFLAGS) -o $@ $<
+ 
+ $(LIBRARY): $(objects)
+ 	$(AR) rcvu $@ $(objects)
+ 	$(RANLIB) $@
+ 
+ $(LIBRARY_SO): $(objects_so)
+-	$(CXX) -shared -Wl,-soname,$(LIBRARY_SO_VER) -o $(LIBRARY_SO_VER) $(objects_so) -lebml
+-	rm -f $(LIBRARY_SO)
+-	ln -s $(LIBRARY_SO_VER) $(LIBRARY_SO)
++	$(LIBTOOL) --mode=link $(CXX) -o $(LIBRARY_SO) $(objects_so) -lebml -rpath $(PREFIX)/lib -version-info 0
+ 
+ clean:	cleantest
+ 	rm -f $(objects) $(objects_so)
 @@ -153,23 +151,22 @@ install: install_staticlib install_share
- endif
+ endif
- 
+ 
- install_headers:
+ install_headers:
--	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(includedir)
+-	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(includedir)
-+	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(includedir)
++	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(includedir)
- 	for i in $(INCLUDE_DIR)/*.h; do \
+ 	for i in $(INCLUDE_DIR)/*.h; do \
--		$(INSTALL) $(INSTALL_OPTS) $$i $(includedir) ; \
+-		$(INSTALL) $(INSTALL_OPTS) $$i $(includedir) ; \
-+		$(INSTALL) $(INSTALL_OPTS) $$i $(DESTDIR)$(includedir) ; \
++		$(INSTALL) $(INSTALL_OPTS) $$i $(DESTDIR)$(includedir) ; \
- 	done
+ 	done
--	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(includedir)/c
+-	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(includedir)/c
-+	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(includedir)/c
++	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(includedir)/c
- 	for i in $(INCLUDE_DIR)/c/*.h; do \
+ 	for i in $(INCLUDE_DIR)/c/*.h; do \
--		$(INSTALL) $(INSTALL_OPTS) $$i $(includedir)/c ; \
+-		$(INSTALL) $(INSTALL_OPTS) $$i $(includedir)/c ; \
-+		$(INSTALL) $(INSTALL_OPTS) $$i $(DESTDIR)$(includedir)/c ; \
++		$(INSTALL) $(INSTALL_OPTS) $$i $(DESTDIR)$(includedir)/c ; \
- 	done
+ 	done
- 
+ 
- install_staticlib: $(LIBRARY)
+ install_staticlib: $(LIBRARY)
--	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(libdir)
+-	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(libdir)
--	$(INSTALL) $(INSTALL_OPTS_LIB) $(LIBRARY) $(libdir)
+-	$(INSTALL) $(INSTALL_OPTS_LIB) $(LIBRARY) $(libdir)
-+	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(libdir)
++	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(libdir)
-+	$(INSTALL) $(INSTALL_OPTS_LIB) $(LIBRARY) $(DESTDIR)$(libdir)
++	$(INSTALL) $(INSTALL_OPTS_LIB) $(LIBRARY) $(DESTDIR)$(libdir)
- 
+ 
- install_sharedlib: $(LIBRARY_SO)
+ install_sharedlib: $(LIBRARY_SO)
--	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(libdir)
+-	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(libdir)
--	$(INSTALL) $(INSTALL_OPTS_LIB) $(LIBRARY_SO_VER) $(libdir)
+-	$(INSTALL) $(INSTALL_OPTS_LIB) $(LIBRARY_SO_VER) $(libdir)
--	ln -fs $(LIBRARY_SO_VER) $(libdir)/$(LIBRARY_SO)
+-	ln -fs $(LIBRARY_SO_VER) $(libdir)/$(LIBRARY_SO)
-+	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(libdir)
++	$(INSTALL) $(INSTALL_DIR_OPTS) -d $(DESTDIR)$(libdir)
-+	$(LIBTOOL) --mode=install $(INSTALL) $(INSTALL_OPTS_LIB) $(LIBRARY_SO) $(DESTDIR)$(libdir)
++	$(LIBTOOL) --mode=install $(BSD_INSTALL_LIB) $(LIBRARY_SO) $(DESTDIR)$(libdir)
- 
+ 
- ifneq ($(wildcard .depend),)
+ ifneq ($(wildcard .depend),)
- include .depend
+ include .depend