www/chromium/Makefile cleanups

Alexey Dokuchaev danfe at nsu.ru
Thu Apr 7 10:24:13 UTC 2011


On Mon, Apr 04, 2011 at 10:00:37PM +0200, Rene Ladan wrote:
> Op 02-04-2011 04:29, Alexey Dokuchaev schreef:
> > Please consider attached diff I'd like to commit (or perhaps you want
> > to do it yourself).  Proposed commit log:
> > 
> > - Consistently use spaces and tabs (get rid of utter mess thereof)
> > - Use PORTNAME in DISTNAME
> > - Wrap some overly long lines
> > - Pad pkg-message with ECHO_CMD's to bring port behavior in line with
> >   that of the package
> > 
> I committed the patch.

Meanwhile, I have some more:

- Convert two explicit dependencies to corresponding USE_GNOME and
  USE_XORG components
- Remove SSE2 option and use proper detection when building instead
- Use USE_PYTHON_BUILD as advised by comment
- Sort knobs a bit while I'm here

./danfe
-------------- next part --------------
Index: Makefile
===================================================================
RCS file: /mnt/fbsd/development/FreeBSD-CVS/ports/www/chromium/Makefile,v
retrieving revision 1.18
diff -u -r1.18 Makefile
--- Makefile	4 Apr 2011 19:45:48 -0000	1.18
+++ Makefile	7 Apr 2011 05:11:17 -0000
@@ -20,7 +20,6 @@
 BUILD_DEPENDS=	${LOCALBASE}/bin/flex:${PORTSDIR}/textproc/flex \
 		${LOCALBASE}/bin/gperf:${PORTSDIR}/devel/gperf \
 		bash:${PORTSDIR}/shells/bash \
-		pkg-config:${PORTSDIR}/devel/pkg-config \
 		yasm:${PORTSDIR}/devel/yasm \
 		nss>=3.12:${PORTSDIR}/security/nss
 # minimal version of nss, LIB_DEPENDS does not enforce this
@@ -29,7 +28,6 @@
 		cairo.2:${PORTSDIR}/graphics/cairo \
 		dbus-1.3:${PORTSDIR}/devel/dbus \
 		dbus-glib-1.2:${PORTSDIR}/devel/dbus-glib \
-		Xss.1:${PORTSDIR}/x11/libXScrnSaver \
 		asound.2:${PORTSDIR}/audio/alsa-lib \
 		freetype.9:${PORTSDIR}/print/freetype2 \
 		nss3.1:${PORTSDIR}/security/nss \
@@ -41,12 +39,12 @@
 ONLY_FOR_ARCHS=	i386 amd64
 USE_XZ=		yes
 USE_BISON=	build
-USE_PYTHON=	2.6+			# only needed at build time
-USE_PERL5_BUILD=yes
 USE_GMAKE=	yes
+USE_PERL5_BUILD=	yes
+USE_PYTHON_BUILD=	2.6+
+USE_XORG=	scrnsaverproto x11 xproto xscrnsaver xtst
+USE_GNOME=	glib20 gtk20 dconf libxslt pkgconfig
 MAN1=		chrome.1
-USE_XORG=	scrnsaverproto x11 xproto xtst
-USE_GNOME=	glib20 gtk20 dconf libxslt
 LICENSE_COMB=	multi
 LICENSE=	BSD LGPL21 MPL
 
@@ -70,7 +68,6 @@
 
 OPTIONS=	CODECS	"Compile and enable patented codecs like H.264"	off \
 		GCONF	"Use GConf2 for preferences"			on \
-		SSE2	"Use SSE2, disable this for PIII or older"	on \
 		VPX	"Use system libvpx for VP8 codec"		on
 
 .include <bsd.port.options.mk>
@@ -95,7 +92,7 @@
 GYP_DEFINES+=	use_gconf=0
 .endif
 
-.if defined(WITHOUT_SSE2)
+.if ! ${MACHINE_CPU:Msse2}
 GYP_DEFINES+=	disable_sse2=1
 .endif
 


More information about the freebsd-chromium mailing list