[82546] trunk/dports

[ryandesign at macports.org]

Revision: 82546
          http://trac.macports.org/changeset/82546
Author:   ryandesign at macports.org
Date:     2011-08-15 09:00:04 -0700 (Mon, 15 Aug 2011)
Log Message:
-----------
app-1.0.tcl: use variable tracing magic to automatically add makeicns build dependency if needed

Modified Paths:
--------------
    trunk/dports/_resources/port1.0/group/app-1.0.tcl
    trunk/dports/databases/sqliteman/Portfile
    trunk/dports/games/chromium-bsu/Portfile
    trunk/dports/games/gdash/Portfile
    trunk/dports/graphics/gvedit-devel/Portfile

Modified: trunk/dports/_resources/port1.0/group/app-1.0.tcl
===================================================================
--- trunk/dports/_resources/port1.0/group/app-1.0.tcl	2011-08-15 14:12:26 UTC (rev 82545)
+++ trunk/dports/_resources/port1.0/group/app-1.0.tcl	2011-08-15 16:00:04 UTC (rev 82546)
@@ -252,5 +252,21 @@
     }
 }
 
-# TODO: automatically add depends_build-append port:makeicns
+
+# Trace writes to app.icon and add or remove makeicns dependency as necessary.
+trace variable app.icon w app._icon_trace
+
+proc app._icon_trace {optionName unusedIndex unusedOperation} {
+    global depends_build
+    upvar ${optionName} option
+    set has_dep [expr {[lsearch ${depends_build} port:makeicns] != -1}]
+    set needs_dep [expr {[file extension ${option}] != ".icns"}]
+    if {${has_dep} && !${needs_dep}} {
+        depends_build-delete port:makeicns
+    } elseif {${needs_dep} && !${has_dep}} {
+        depends_build-append port:makeicns
+    }
+}
+
+
 # TODO: for Tiger we probably need our own lreverse implementation e.g. http://wiki.tcl.tk/17188

Modified: trunk/dports/databases/sqliteman/Portfile
===================================================================
--- trunk/dports/databases/sqliteman/Portfile	2011-08-15 14:12:26 UTC (rev 82545)
+++ trunk/dports/databases/sqliteman/Portfile	2011-08-15 16:00:04 UTC (rev 82546)
@@ -41,8 +41,4 @@
     -DQSCINTILLA_INCLUDE_DIR:PATH=${qt_includes_dir}/Qsci
 configure.args      ${qt_cmake_defines}
 
-platform macosx {
-    depends_build-append port:makeicns
-}
-
 app.icon            sqliteman/icons/sqliteman.png

Modified: trunk/dports/games/chromium-bsu/Portfile
===================================================================
--- trunk/dports/games/chromium-bsu/Portfile	2011-08-15 14:12:26 UTC (rev 82545)
+++ trunk/dports/games/chromium-bsu/Portfile	2011-08-15 16:00:04 UTC (rev 82546)
@@ -47,10 +47,6 @@
 app.name                Chromium B.S.U.
 app.icon                misc/chromium-bsu.png
 
-platform macosx {
-    depends_build-append port:makeicns
-}
-
 livecheck.distname      Chromium%20B.S.U.%20source%20code
 livecheck.url           http://sourceforge.net/projects/chromium-bsu/files/${livecheck.distname}
 livecheck.regex         files/${livecheck.distname}/(\[0-9.\]+)/

Modified: trunk/dports/games/gdash/Portfile
===================================================================
--- trunk/dports/games/gdash/Portfile	2011-08-15 14:12:26 UTC (rev 82545)
+++ trunk/dports/games/gdash/Portfile	2011-08-15 16:00:04 UTC (rev 82546)
@@ -32,10 +32,6 @@
                     --disable-gtktest \
                     --disable-sdltest
 
-platform macosx {
-    depends_build-append port:makeicns
-}
-
 app.name            SDash
 app.executable      sdash
 app.icon            docs/gdash.png

Modified: trunk/dports/graphics/gvedit-devel/Portfile
===================================================================
--- trunk/dports/graphics/gvedit-devel/Portfile	2011-08-15 14:12:26 UTC (rev 82545)
+++ trunk/dports/graphics/gvedit-devel/Portfile	2011-08-15 16:00:04 UTC (rev 82546)
@@ -50,10 +50,6 @@
 app.executable                  gvedit
 app.icon                        ${build.dir}/images/icon.png
 
-platform macosx {
-    depends_build-append port:makeicns
-}
-
 pre-destroot {
     app.short_version_string [get_short_version_string]
 }
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20110815/47c1d77a/attachment.html>