<pre style='margin:0'>
Mihai Moldovan (Ionic) pushed a commit to branch master
in repository macports-ports.

</pre>
<p><a href="https://github.com/macports/macports-ports/commit/593a59f76c85ef0b3eb37f41ceb2075d7139796a">https://github.com/macports/macports-ports/commit/593a59f76c85ef0b3eb37f41ceb2075d7139796a</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new 593a59f  gnome/gstreamer1-gst-plugins-good: force-enable features if selected via variants.
</span>593a59f is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 593a59f76c85ef0b3eb37f41ceb2075d7139796a
</span>Author: Mihai Moldovan <ionic@ionic.de>
AuthorDate: Sat Jun 16 06:53:30 2018 +0200

<span style='display:block; white-space:pre;color:#404040;'>    gnome/gstreamer1-gst-plugins-good: force-enable features if selected via variants.
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Just removing the --disable-foo flag will make configure auto-detect
</span><span style='display:block; white-space:pre;color:#404040;'>    feature foo, which is not what we want, if the variant is enabled.
</span><span style='display:block; white-space:pre;color:#404040;'>    Instead, it should be force-enabled and configure fail, if the feature
</span><span style='display:block; white-space:pre;color:#404040;'>    is not usable (for instance because we forgot to add proper dependencies
</span><span style='display:block; white-space:pre;color:#404040;'>    and the port is built in trace mode or the build system just lacks the
</span><span style='display:block; white-space:pre;color:#404040;'>    required dependencies because it's mostly clean).
</span>---
 gnome/gstreamer1-gst-plugins-good/Portfile | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/gnome/gstreamer1-gst-plugins-good/Portfile b/gnome/gstreamer1-gst-plugins-good/Portfile
</span><span style='display:block; white-space:pre;color:#808080;'>index fe4a352..ad3426a 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/gnome/gstreamer1-gst-plugins-good/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/gnome/gstreamer1-gst-plugins-good/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -11,6 +11,7 @@ name                gstreamer1-gst-plugins-good
</span> set my_name         gst-plugins-good
 # please only commit stable updates (even numbered releases)
 version             1.14.1
<span style='display:block; white-space:pre;background:#e0ffe0;'>+revision            1
</span> description         A set of good-quality plug-ins for GStreamer under GStreamer's preferred \
                     license, LGPL.
 long_description    GStreamer Good Plug-ins is a set of plug-ins that GStreamer's developers \
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -94,7 +95,8 @@ platform darwin {
</span> 
 variant gtk3 requires x11 description {Enable GTK3 X11/GL windowing elements} {
     depends_lib-append      port:gtk3
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    configure.args-delete   --disable-gtk3
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    configure.args-replace  --disable-gtk3 \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                            --enable-gtk3
</span> }
 
 # gtk3 elements require gtk3 +x11
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -105,12 +107,14 @@ if {[variant_isset gtk3]} {
</span> 
 variant jack description {Enable Jack plugin} {
     depends_lib-append      port:jack
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    configure.args-delete   --disable-jack
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    configure.args-replace  --disable-jack \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                            --enable-jack
</span> }
 
 variant pulseaudio description {Enable PulseAudio plugin} {
     depends_lib-append      port:pulseaudio
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    configure.args-delete   --disable-pulse
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    configure.args-replace  --disable-pulse \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                            --enable-pulse
</span> }
 
 variant x11 {
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -119,7 +123,8 @@ variant x11 {
</span>                             port:xorg-libXdamage \
                             port:xorg-libXext
     configure.args-append   --with-default-videosink=ximagesink
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    configure.args-delete   --disable-x
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    configure.args-replace  --disable-x \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                            --enable-x
</span> }
 
 default_variants    +pulseaudio +x11
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -131,7 +136,8 @@ variant qt description {Enable QT5 MAC windowing elements } {
</span>     qt5.depends_component   qtdeclarative \
                             qtmacextras
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    configure.args-delete   --disable-qt
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    configure.args-replace  --disable-qt \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                            --enable-qt
</span> 
     # avoid:
     #    ${prefix}/libexec/qt5/include/QtCore/qcompilerdetection.h:562:6: error: Qt requires a C++11 compiler and yours does not seem to be that.
</pre><pre style='margin:0'>

</pre>