From owner-svn-ports-all@freebsd.org Thu Mar 10 00:45:14 2016 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 8F5E3AC8A38 for ; Thu, 10 Mar 2016 00:45:14 +0000 (UTC) (envelope-from jbeich@freebsd.org) Received: from vfemail.net (onethreetwo.vfemail.net [199.16.11.132]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 369978B6 for ; Thu, 10 Mar 2016 00:45:13 +0000 (UTC) (envelope-from jbeich@freebsd.org) Received: (qmail 55772 invoked by uid 89); 10 Mar 2016 00:45:12 -0000 Received: from localhost (HELO freequeue.vfemail.net) (127.0.0.1) by localhost with (DHE-RSA-AES256-SHA encrypted) SMTP; 10 Mar 2016 00:45:12 -0000 Received: (qmail 55668 invoked by uid 89); 10 Mar 2016 00:44:55 -0000 Received: by simscan 1.3.1 ppid: 55660, pid: 55664, t: 0.0044s scanners:none Received: from unknown (HELO smtp102-2.vfemail.net) (172.16.100.62) by FreeQueue with SMTP; 10 Mar 2016 00:44:55 -0000 Received: (qmail 23684 invoked by uid 89); 10 Mar 2016 00:44:55 -0000 Received: by simscan 1.4.0 ppid: 23670, pid: 23681, t: 0.8038s scanners:none Received: from unknown (HELO nil) (amJlaWNoQHZmZW1haWwubmV0@172.16.100.27) by mail.vfemail.net with ESMTPA; 10 Mar 2016 00:44:54 -0000 From: Jan Beich To: Antoine Brodin Cc: "svn-ports-head\@freebsd.org" , "svn-ports-all\@freebsd.org" , "ports-committers\@freebsd.org" Subject: Re: svn commit: r410689 - in head/audio: sdl2_mixer sdl2_mixer/files sdl_mixer In-Reply-To: (Antoine Brodin's message of "Wed, 9 Mar 2016 23:47:02 +0100") References: <201603090518.u295IoEt073907@repo.freebsd.org> Date: Thu, 10 Mar 2016 01:44:44 +0100 Message-ID: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.21 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: Thu, 10 Mar 2016 00:45:14 -0000 --=-=-= Content-Type: text/plain Antoine Brodin writes: > On Wed, Mar 9, 2016 at 6:18 AM, Jan Beich wrote: > >> Author: jbeich >> Date: Wed Mar 9 05:18:50 2016 >> New Revision: 410689 >> URL: https://svnweb.freebsd.org/changeset/ports/410689 >> >> Log: >> audio/sdl{,2}_mixer: add TREMOR option >> >> TREMOR is enabled by default on platforms without hardware floating point. >> >> PR: 202689 >> Approved by: maintainer timeout (6 months) > ... >> @@ -46,6 +53,9 @@ SMPEG_CONFIGURE_OFF= --disable-music-mp >> VORBIS_LIB_DEPENDS= libvorbis.so:${PORTSDIR}/audio/libvorbis >> VORBIS_CONFIGURE_ENABLE= music-ogg >> >> +TREMOR_LIB_DEPENDS= libvorbisidec.so:${PORTSDIR}/audio/libtremor >> +TREMOR_CONFIGURE_ENABLE= music-ogg music-ogg-tremor > > This both --enable-music-ogg and --disable-music-ogg which looks wrong Thanks for noticing early. Fixed in r410736. I've decided to stick to option helpers rather than expanding into: VORBIS_CONFIGURE_ON= --enable-music-ogg TREMOR_CONFIGURE_ON= --enable-music-ogg --enable-music-ogg-tremor .include .if ! ${PORT_OPTIONS:MVORBIS} && ! ${PORT_OPTIONS:MTREMOR} CONFIGURE_ARGS+= --disable-music-ogg .endif For reference, here's what configure.in contains: AC_ARG_ENABLE([music-ogg],,[], [enable_music_ogg=yes]) AC_ARG_ENABLE(music-ogg-tremor,, [], enable_music_ogg_tremor=no) if test x$enable_music_ogg = xyes; then if test x$enable_music_ogg_tremor = xyes; then AC_CHECK_HEADER([tremor/ivorbisfile.h], [have_tremor_hdr=yes]) AC_CHECK_LIB([vorbisidec], [ov_open_callbacks], [have_tremor_lib=yes]) [...] else AC_CHECK_HEADER([vorbis/vorbisfile.h], [have_ogg_hdr=yes]) AC_CHECK_LIB([vorbisfile], [ov_open_callbacks], [have_ogg_lib=yes]) [...] compared to MOD options that don't depend on each other AC_ARG_ENABLE([music-mod],, [], [enable_music_mod=yes]) if test x$enable_music_mod = xyes; then AC_PATH_PROG(LIBMIKMOD_CONFIG, libmikmod-config, no, [$PATH]) [...] AC_ARG_ENABLE([music-mod-modplug],, [], [enable_music_mod_modplug=no]) if test x$enable_music_mod_modplug = xyes; then PKG_CHECK_MODULES([MODPLUG], [libmodplug >= 0.8.7]) [...] -- AC_ARG_ENABLE (FEATURE, HELP-STRING, [ACTION-IF-GIVEN], [ACTION-IF-NOT-GIVEN]) --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQF8BAEBCgBmBQJW4MN8XxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXREQjQ0MzY3NEM3RDIzNTc4NkUxNDkyQ0VF NEM3Nzg4MzQ3OURCRERCAAoJEOTHeINHnb3bXyIIAMtJj9rYC2Lk7WinjH2wEv5s CnlDIuhHlUMvmrlldW6homUbAbgH4W/Q63Yi1CmBSWYPuPLENjWXrXTAbWJgn5ll a8SozjyXdnnZyrWJjUDJ6bKM0Spsof7u+WnIlqrcru4aSK/4wNGarEhsaT2Y6FYT 3fVrHnzKzgYn8FF2HEDi00lchDU3QR8NOmqFWFlKBwMQfMOKIGzHwDpts/JG4BlX 5sH8XIZ6JMSa0Fi6ZdMRQRhF7xUgpujxVlS4v5fjxL2aZf3JW3PIKgWzfPLonwJQ VN4WyOtHVFT4ypfIy7uQ9O72W7FxJ86ecTxCA4COemkd0Gmo/ISPAJMgvqhOUT4= =rHbN -----END PGP SIGNATURE----- --=-=-=--