[88839] users/ryandesign/scripts/badpyports

[ryandesign at macports.org]

Revision: 88839
          http://trac.macports.org/changeset/88839
Author:   ryandesign at macports.org
Date:     2012-01-13 01:04:13 -0800 (Fri, 13 Jan 2012)
Log Message:
-----------
badpyports: rename PROBLEMPORTS => TOTALPROBLEMPORTS and compute it differently

Modified Paths:
--------------
    users/ryandesign/scripts/badpyports

Modified: users/ryandesign/scripts/badpyports
===================================================================
--- users/ryandesign/scripts/badpyports	2012-01-13 09:00:25 UTC (rev 88838)
+++ users/ryandesign/scripts/badpyports	2012-01-13 09:04:13 UTC (rev 88839)
@@ -8,7 +8,6 @@
 # Count them.
 TOTALPYPORTS=$(echo $NAMES | wc -w)
 
-PROBLEMPORTS=0
 DEPENDENCYPROBLEMS=0
 DEFAULTVERSIONPROBLEMS=0
 for NAME in $NAMES; do
@@ -21,21 +20,21 @@
     
     if [ -z "$DEFAULTVERSION" ]; then
         printf "%s declares dependencies even in the stub port\n" $STUBPORT
-        PROBLEMPORTS=$(($PROBLEMPORTS + 1))
         DEPENDENCYPROBLEMS=$(($DEPENDENCYPROBLEMS + 1))
     else
         # Get the file of the default subport. If the default is set incorrectly, this will be empty.
         FILE=$(port file py$DEFAULTVERSION-$NAME 2>/dev/null)
         if [ -z "$FILE" ]; then
             printf "%s declares default python version %s but does not offer python version %s\n" $STUBPORT $DEFAULTVERSION $DEFAULTVERSION
-            PROBLEMPORTS=$(($PROBLEMPORTS + 1))
             DEFAULTVERSIONPROBLEMS=$(($DEFAULTVERSIONPROBLEMS + 1))
         fi
     fi
 done
 
+TOTALPROBLEMPORTS=$(($DEPENDENCYPROBLEMS + $DEFAULTVERSIONPROBLEMS))
+
 echo
-printf "Out of %d ports using the unified python portgroup, %d have problems.\n" $TOTALPYPORTS $PROBLEMPORTS
+printf "Out of %d ports using the unified python portgroup, %d have problems.\n" $TOTALPYPORTS $TOTALPROBLEMPORTS
 
 if [ $DEPENDENCYPROBLEMS -gt 0 ]; then
     echo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20120113/5467a4f8/attachment-0001.html>