From owner-freebsd-multimedia@FreeBSD.ORG Fri Nov 27 03:11:48 2009 Return-Path: Delivered-To: multimedia@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 05F74106566C for ; Fri, 27 Nov 2009 03:11:48 +0000 (UTC) (envelope-from gslin@colo-p.gslin.org) Received: from colo-p.gslin.org (60-199-248-145.static.tfn.net.tw [60.199.248.145]) by mx1.freebsd.org (Postfix) with ESMTP id CA5608FC0A for ; Fri, 27 Nov 2009 03:11:47 +0000 (UTC) Received: by colo-p.gslin.org (Postfix, from userid 1000) id 944187E81B; Fri, 27 Nov 2009 11:11:44 +0800 (CST) Date: Fri, 27 Nov 2009 11:11:44 +0800 From: Gea-Suan Lin To: Wes Morgan Message-ID: <20091127031144.GA53564@gslin.org> References: <20091125161409.4F42E7E81B@colo-p.gslin.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Cc: gslin@gslin.org, multimedia@freebsd.org Subject: Re: [PATCH] multimedia/x264: update to 0.0.20091124 X-BeenThere: freebsd-multimedia@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Multimedia discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 27 Nov 2009 03:11:48 -0000 Thanks for the url. I've used another approach that define log2f(x) = (log(x) / M_LN2) when log2f not defined, this way will be much simple. Please check http://colo-p.gslin.org/~gslin/tmp/x264.shar this version. On Wed, Nov 25, 2009 at 01:56:06PM -0600, Wes Morgan wrote: > On Thu, 26 Nov 2009, Gea-Suan Lin wrote: > > > > >> Submitter-Id: current-users > >> Originator: Gea-Suan Lin > >> Organization: > >> Confidential: no > >> Synopsis: [PATCH] multimedia/x264: update to 0.0.20091124 > >> Severity: non-critical > >> Priority: low > >> Category: ports > >> Class: update > >> Release: FreeBSD 7.2-RELEASE-p3 i386 > >> Environment: > > System: FreeBSD colo-p.gslin.org 7.2-RELEASE-p3 FreeBSD 7.2-RELEASE-p3 #0: Sat Aug 22 10:48:39 CST 2009 > >> Description: > > - Update to 0.0.20091124 > > > > Port maintainer (multimedia@FreeBSD.org) is cc'd. > > > > Generated with FreeBSD Port Tools 0.99 > >> How-To-Repeat: > >> Fix: > > > > --- x264-0.0.20091124.patch begins here --- > > diff -ruN --exclude=CVS /usr/ports/multimedia/x264/Makefile /home/staff/gslin/work/ports/x264/Makefile > > --- /usr/ports/multimedia/x264/Makefile 2009-11-23 04:28:29.000000000 +0800 > > +++ /home/staff/gslin/work/ports/x264/Makefile 2009-11-26 00:13:16.000000000 +0800 > > @@ -6,7 +6,7 @@ > > # $Id: Makefile 30 2006-10-30 22:15:26Z buhnux $ > > > > PORTNAME= x264 > > -PORTVERSION= 0.0.20081218 > > +PORTVERSION= 0.0.20091124 > > CATEGORIES= multimedia > > MASTER_SITES= http://downloads.videolan.org/pub/videolan/x264/snapshots/ \ > > http://samples.mplayerhq.hu/yuv4mpeg2/:pgo > > @@ -16,7 +16,7 @@ > > EXTRACT_ONLY= ${PORTNAME}-snapshot-${PORTVERSION:S/0.0.//}-2245${EXTRACT_SUFX} > > > > MAINTAINER= multimedia@FreeBSD.org > > -COMMENT?= Multimedia library and tool for encoding H.264/AVC video streams > > +COMMENT= Multimedia library and tool for encoding H.264/AVC video streams > > > > BUILD_DEPENDS= yasm>=0.6.0:${PORTSDIR}/devel/yasm > > > > @@ -36,7 +36,7 @@ > > > > CONFIGURE_ARGS+= --extra-cflags="${CPPFLAGS} ${CFLAGS} -fPIC -I${LOCALBASE}/include -I${LOCALBASE}/include" \ > > --extra-ldflags="${LDFLAGS} -L${LOCALBASE}/lib -L${LOCALBASE}/lib" \ > > - --enable-shared > > + --enable-shared --disable-asm > > I'm pretty certain that this will not work on amd64. The gcc internal > "log2f" function won't be picked up. See this email for a fix: > > http://lists.freebsd.org/pipermail/freebsd-multimedia/2009-March/009819.html > > Would it also be possible to add knobs for enabling the internal assembler > and adding dependencies on both binutils and gcc 4.4 from ports? I've been > using a snapshot built this way for about 2 weeks now doing some pretty > heavy encoding and have no problems other than the log2f patch. -- * Gea-Suan Lin (public key: Using https://keyserver.pgp.com/ to search) * If you cannot convince them, confuse them. -- Harry S Truman