Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Sep 2018 10:41:10 +0000 (UTC)
From:      Tobias Kortkamp <tobik@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r479389 - in head/audio/cava: . files
Message-ID:  <201809101041.w8AAfADO060924@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: tobik
Date: Mon Sep 10 10:41:09 2018
New Revision: 479389
URL: https://svnweb.freebsd.org/changeset/ports/479389

Log:
  audio/cava: Unbreak build on powerpc (or with gcc 4.2)
  
  - Remove command line options that gcc 4.2 does not recognize
  - Allow build on powerpc64 again as it most likely will work now too
  - Move post-patch to a patch file while here

Added:
  head/audio/cava/files/patch-Makefile.am   (contents, props changed)
Modified:
  head/audio/cava/Makefile

Modified: head/audio/cava/Makefile
==============================================================================
--- head/audio/cava/Makefile	Mon Sep 10 10:40:06 2018	(r479388)
+++ head/audio/cava/Makefile	Mon Sep 10 10:41:09 2018	(r479389)
@@ -10,8 +10,6 @@ COMMENT=	Console-based Audio Visualizer for MPD, Pulse
 LICENSE=	MIT
 LICENSE_FILE=	${WRKSRC}/LICENSE
 
-BROKEN_powerpc64=	fails to compile: cc1: unrecognized command line option -Wno-unused-result
-
 LIB_DEPENDS=	libfftw3.so:math/fftw3 \
 		libiniparser.so:devel/iniparser
 
@@ -37,9 +35,6 @@ PULSEAUDIO_LIB_DEPENDS=		libpulse-simple.so:audio/puls
 
 SNDIO_CONFIGURE_ENV_OFF=	ac_cv_lib_sndio_sio_open=no
 SNDIO_LIB_DEPENDS=		libsndio.so:audio/sndio
-
-post-patch:
-	@${REINPLACE_CMD} '/^cava_font/d' ${WRKSRC}/Makefile.am
 
 pre-configure:
 	@${ECHO_CMD} ${DISTVERSION} > ${WRKSRC}/version

Added: head/audio/cava/files/patch-Makefile.am
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/audio/cava/files/patch-Makefile.am	Mon Sep 10 10:41:09 2018	(r479389)
@@ -0,0 +1,18 @@
+- Remove some flags that gcc 4.2 (on tier 2 archs like powerpc)
+  does not recognize
+- Do not install the Linux-only font
+
+--- Makefile.am.orig	2018-09-10 08:31:56 UTC
++++ Makefile.am
+@@ -14,10 +14,8 @@ cava_LDADD = -liniparser
+ cava_LDFLAGS = -L/usr/local/lib -Wl,-rpath /usr/local/lib
+ cava_CPPFLAGS = -DPACKAGE=\"$(PACKAGE)\" -DVERSION=\"$(VERSION)\" \
+            -D_POSIX_SOURCE -D _POSIX_C_SOURCE=200809L
+-cava_CFLAGS = -std=c99 -Wall -Wextra -Wno-unused-result -Wno-maybe-uninitialized 
++cava_CFLAGS = -std=c99 -Wall -Wextra
+ 
+-cava_font_dir = @FONT_DIR@
+-cava_font__DATA = cava.psf
+ 
+ if !SYSTEM_LIBINIPARSER
+     cava_SOURCES += iniparser/libiniparser.la



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201809101041.w8AAfADO060924>