From owner-svn-ports-all@FreeBSD.ORG Wed Aug 14 11:58:54 2013 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 5EB2B938; Wed, 14 Aug 2013 11:58:54 +0000 (UTC) (envelope-from danfe@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 34435240D; Wed, 14 Aug 2013 11:58:54 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.7/8.14.7) with ESMTP id r7EBwsQD028048; Wed, 14 Aug 2013 11:58:54 GMT (envelope-from danfe@svn.freebsd.org) Received: (from danfe@localhost) by svn.freebsd.org (8.14.7/8.14.5/Submit) id r7EBwrbq028043; Wed, 14 Aug 2013 11:58:53 GMT (envelope-from danfe@svn.freebsd.org) Message-Id: <201308141158.r7EBwrbq028043@svn.freebsd.org> From: Alexey Dokuchaev Date: Wed, 14 Aug 2013 11:58:53 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r324705 - in head/multimedia/handbrake: . files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Aug 2013 11:58:54 -0000 Author: danfe Date: Wed Aug 14 11:58:53 2013 New Revision: 324705 URL: http://svnweb.freebsd.org/changeset/ports/324705 Log: Unbreak the build after r324037: 1) `devel/libnotify' no longer pulls gtk20; we must depend explicitly; 2) `x11-toolkits/pango' now depends on `print/harfbuzz', which installs header file ($localbase/include/harfbuzz/hb.h) that in turn confuses bundled libass' configure script and conflicts with HandBrake's own internal header. NB: supposedly it should have been caught by exp-run for r324037, but it was not for some reason. Reported by: Sergey V. Dyatko Modified: head/multimedia/handbrake/Makefile head/multimedia/handbrake/files/patch-contrib_libass_module.defs Modified: head/multimedia/handbrake/Makefile ============================================================================== --- head/multimedia/handbrake/Makefile Wed Aug 14 11:58:48 2013 (r324704) +++ head/multimedia/handbrake/Makefile Wed Aug 14 11:58:53 2013 (r324705) @@ -37,12 +37,12 @@ COMMENT= Versatile DVD ripper and video LICENSE= GPLv2 BUILD_DEPENDS= yasm:${PORTSDIR}/devel/yasm -LIB_DEPENDS= fribidi:${PORTSDIR}/converters/fribidi +LIB_DEPENDS= libfribidi.so:${PORTSDIR}/converters/fribidi RUN_DEPENDS= ${LOCALBASE}/lib/libdvdcss.so:${PORTSDIR}/multimedia/libdvdcss +USES= gmake USE_AUTOTOOLS= libtool aclocal LIBTOOLFILES= configure -USE_GMAKE= yes USE_PYTHON_BUILD= yes WANT_GNOME= yes @@ -76,9 +76,9 @@ MAKE_ENV= COMPILER_PATH=${LOCALBASE}/bin .if ${PORT_OPTIONS:MX11} CONFIGURE_ARGS+= --disable-gtk-update-checks --disable-gst -LIB_DEPENDS+= dbus-glib-1:${PORTSDIR}/devel/dbus-glib \ - notify:${PORTSDIR}/devel/libnotify -USE_GNOME= intltool +LIB_DEPENDS+= libdbus-glib-1.so:${PORTSDIR}/devel/dbus-glib \ + libnotify.so:${PORTSDIR}/devel/libnotify +USE_GNOME= gtk20 PLIST_FILES+= bin/HandBrake .else CONFIGURE_ARGS+= --disable-gtk @@ -116,6 +116,8 @@ post-extract: .SILENT ${MV} ${WRKDIR}/mp4v2-trunk-r355 ${BUILD_WRKSRC}/contrib/mp4v2 # Remove one patch file that does not apply cleanly ${RM} ${WRKSRC}/contrib/fontconfig/A00-config.patch +# Avoid conflict with `print/harfbuzz' (also installs "hb.h") + ${MV} ${WRKSRC}/libhb/hb.h ${WRKSRC}/libhb/hb-of-handbrake.h post-patch: @${REINPLACE_CMD} -e 's|machine/soundcard|sys/soundcard|' \ @@ -128,6 +130,8 @@ post-patch: ${WRKSRC}/libhb/module.defs ${WRKSRC}/test/module.defs @${REINPLACE_CMD} -e 's|libdvdcss\.so\.2|libdvdcss.so|' \ ${BUILD_WRKSRC}/contrib/libdvdread/libdvdread/src/dvd_input.c + @${FIND} ${WRKSRC} -type f -name '*.[ch]' -print0 | ${XARGS} -0 \ + ${REINPLACE_CMD} -e '/#include/s|hb\.h|hb-of-handbrake.h|' do-install: ${INSTALL_PROGRAM} ${BUILD_WRKSRC}/HandBrakeCLI ${PREFIX}/bin Modified: head/multimedia/handbrake/files/patch-contrib_libass_module.defs ============================================================================== --- head/multimedia/handbrake/files/patch-contrib_libass_module.defs Wed Aug 14 11:58:48 2013 (r324704) +++ head/multimedia/handbrake/files/patch-contrib_libass_module.defs Wed Aug 14 11:58:53 2013 (r324705) @@ -1,10 +1,14 @@ --- ./contrib/libass/module.defs.orig 2010-09-28 18:10:49.000000000 -0400 +++ ./contrib/libass/module.defs 2011-05-24 05:50:46.096807667 -0400 -@@ -10,6 +10,6 @@ +@@ -7,9 +7,9 @@ + # Disable as many external dependencies as I can get away with + # and tell configure where to find our version of freetype LIBASS.CONFIGURE.extra = \ - --disable-png --disable-enca \ +- --disable-png --disable-enca \ ++ --disable-png --disable-enca --disable-harfbuzz \ FREETYPE_LIBS="-L$(call fn.ABSOLUTE,$(CONTRIB.build/))lib -lfreetype" \ - FREETYPE_CFLAGS="-I$(call fn.ABSOLUTE,$(CONTRIB.build/))include/freetype2" \ + FREETYPE_CFLAGS="-I$(call fn.ABSOLUTE,$(CONTRIB.build/))include/freetype2 -I/usr/local/include" \ FONTCONFIG_LIBS="-L$(call fn.ABSOLUTE,$(CONTRIB.build/))lib -lfontconfig" \ FONTCONFIG_CFLAGS="-I$(call fn.ABSOLUTE,$(CONTRIB.build/))include" +