Re: [121029] trunk/dports/multimedia

2014-06-16 Thread Frank Schima
Thanks. Applied in r121061. On Jun 15, 2014, at 4:07 PM, Mihai Moldovan io...@ionic.de wrote: multimedia/audacious-plugins: set sdl1 and ffmpeg as default variants only if sdl2 is not selected. audacious-plugins-sdl-variants.diff___ macports-dev

Re: [121029] trunk/dports/multimedia

2014-06-15 Thread Frank Schima
On Jun 14, 2014, at 10:17 PM, Mihai Moldovan io...@ionic.de wrote: This patch fixes the darwin typo, a TCL issue that has been shadowed by the aforementioned typo and doesn't explicitly check for the existence of ${configure.cxx_stdlib}. Also revbumped to have the correct min version

Re: [121029] trunk/dports/multimedia

2014-06-15 Thread Mihai Moldovan
* On 15.06.2014 03:53 pm, Frank Schima wrote: default_variants +full if {![variant_isset sdl2]} { default_variants-append +sdl1 } I can apply this change if you want? Thanks for the idea. Hold on, though, I'll send a patch later. Mihai smime.p7s Description: S/MIME Cryptographic

Re: [121029] trunk/dports/multimedia

2014-06-15 Thread Mihai Moldovan
multimedia/audacious-plugins: set sdl1 and ffmpeg as default variants only if sdl2 is not selected. --- multimedia/audacious-plugins/Portfile.old +++ multimedia/audacious-plugins/Portfile @@ -98,7 +98,7 @@ configure.args --enable-nls \ --disable-soxr \

Re: [121029] trunk/dports/multimedia

2014-06-14 Thread Ryan Schmidt
On Jun 14, 2014, at 6:42 PM, m...@macports.org wrote: Revision 121029 Author m...@macports.org Date 2014-06-14 16:42:28 -0700 (Sat, 14 Jun 2014) Log Message audacious audacious-core audacious-plugins: New ports. (#25260) --- trunk/dports/multimedia/audacious-core/Portfile

Re: [121029] trunk/dports/multimedia

2014-06-14 Thread Craig Treleaven
+platform darin { Both audacious-core and audacious-plugins contain this line. I think you darwin. Craig ___ macports-dev mailing list macports-dev@lists.macosforge.org https://lists.macosforge.org/mailman/listinfo/macports-dev

Re: [121029] trunk/dports/multimedia

2014-06-14 Thread Mihai Moldovan
* On 15.06.2014 02:51 am, Ryan Schmidt wrote: On Jun 14, 2014, at 6:42 PM, m...@macports.org wrote: libtool is also usually needed to autoreconf. You should verify whether this port builds ok when libtool is deactivated; if not, add the libtool build dependency as well. I built with trace

Re: [121029] trunk/dports/multimedia

2014-06-14 Thread Mihai Moldovan
* On 15.06.2014 03:40 am, Mihai Moldovan wrote: * On 15.06.2014 02:51 am, Ryan Schmidt wrote: +if {[info exists configure.cxx_stdlib] Now that MacPorts 2.3.0 has been released, configure.cxx_stdlib does exist, so there's no need to check for it. What's a better way to handle this? That

Re: [121029] trunk/dports/multimedia

2014-06-14 Thread Mihai Moldovan
This patch fixes the darwin typo, a TCL issue that has been shadowed by the aforementioned typo and doesn't explicitly check for the existence of ${configure.cxx_stdlib}. Also revbumped to have the correct min version target set. --- multimedia/audacious/Portfile.old 2014-06-15