Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 14 Apr 2018 10:08:22 +0000 (UTC)
From:      Adriaan de Groot <adridg@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r467303 - head/audio/kwave/files
Message-ID:  <201804141008.w3EA8MJ8062393@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: adridg
Date: Sat Apr 14 10:08:22 2018
New Revision: 467303
URL: https://svnweb.freebsd.org/changeset/ports/467303

Log:
  Pre-emptively fix audio/kwave before CMake 3.11.
  
  The newer CMake respects CMAKE_REQUIRED_LIBRARIES in places that older
  CMake code doesn't expect. Temporarily reset to empty for that. No
  effect on CMake 3.10, which didn't use that variable in CHECK_INCLUDE_FILES
  anyway.
  
  Reported by:	antoine (via exp-run)
  Approved by:	tcberner (mentor, implicit)

Added:
  head/audio/kwave/files/patch-plugins_codec__ogg_CMakeLists.txt   (contents, props changed)

Added: head/audio/kwave/files/patch-plugins_codec__ogg_CMakeLists.txt
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/audio/kwave/files/patch-plugins_codec__ogg_CMakeLists.txt	Sat Apr 14 10:08:22 2018	(r467303)
@@ -0,0 +1,23 @@
+In CMake 3.11, CHECK_INCLUDE_FILES() respects CMAKE_REQUIRED_LIBRARIES,
+which this module doesn't expect. Temporarily reset the required libraries
+while finding the headers.
+
+--- plugins/codec_ogg/CMakeLists.txt.orig	2018-03-01 22:57:03 UTC
++++ plugins/codec_ogg/CMakeLists.txt
+@@ -95,6 +95,8 @@ IF (WITH_OGG_OPUS OR WITH_OGG_VORBIS)
+             ${VORBISENC_LIBRARIES}
+         )
+ 
++        SET(_vorbis_CMR ${CMAKE_REQUIRED_LIBRARIES})
++        SET(CMAKE_REQUIRED_LIBRARIES )
+         CHECK_INCLUDE_FILES(
+             "vorbis/codec.h;vorbis/vorbisenc.h"
+             HAVE_OGG_VORBIS_HEADERS
+@@ -102,6 +104,7 @@ IF (WITH_OGG_OPUS OR WITH_OGG_VORBIS)
+         IF (NOT HAVE_OGG_VORBIS_HEADERS)
+             MESSAGE(FATAL_ERROR "ogg vorbis header files are missing")
+         ENDIF (NOT HAVE_OGG_VORBIS_HEADERS)
++        SET(CMAKE_REQUIRED_LIBRARIES ${_vorbis_CMR})
+ 
+ 
+         SET(HAVE_OGG_VORBIS ON CACHE BOOL "enable Vorbis codec")



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