Changeset 48581 in vbox

Timestamp:

Sep 20, 2013 12:35:56 PM (12 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

89141

Message:

Config.kmk: VBOX_WITH_PYLINT_V1_0 -> VBOX_WITH_PYLINT_PRE_V1_0.

File:

• trunk/Config.kmk (modified) (4 diffs)

trunk/Config.kmk

@@ -1695 +1695 @@
 # Note! Yes, --max-branchs is a typo, but not by me.
 #
-# Note! Users of pylint 1.0 and later needs to define VBOX_WITH_PYLINT_V1_0 in
-#       LocalConfig because they changed the output arguments in an
-#       backwards incomaptible way.
+# Note! Users of older pylint release (< 1.0) needs to define
+#       VBOX_WITH_PYLINT_PRE_V1_0 in LocalConfig because they changed the
+#        output arguments in an backwards incompatible way and added more
+#       attribute regex options.
 #
 ifndef VBOX_PYLINT
@@ -1709 +1710 @@
 endif
 VBOX_PYLINT_FLAGS   := --report=n \
-        $(if-expr defined(VBOX_WITH_PYLINT_V1_0)\
+        $(if-expr !defined(VBOX_WITH_PYLINT_PRE_V1_0)\
           ,--msg-template='{path}:{line}: {msg_id}({obj}:{symbol}): {msg}' \
           ,--output-format=msvs --include-ids=y) \
@@ -1716 +1717 @@
         --max-line-length=130 \
         --max-attributes=20 \
-        --max-branch$(if-expr defined(VBOX_WITH_PYLINT_V1_0),e,)s=50 \
+        --max-branch$(if-expr !defined(VBOX_WITH_PYLINT_PRE_V1_0),e,)s=50 \
         --max-statements=100 \
         --max-returns=100 \
@@ -1733 +1734 @@
         --const-rgx='(g_(k|)[abcdefhmnpilorstu]+[A-Z][a-zA-Z0-9]+|[A-Z_][A-Z0-9_]+|__copyright__|__version__|__file__|__all__)$(DOLLAR)' \
         --good-names='i,j,k,t,o,c,e,_' \
-        $(if-expr defined(VBOX_WITH_PYLINT_V1_0),--class-attribute-rgx='(k[abcdefhmnpilorstu]+[A-Z0-9][a-zA-Z0-9_]{0,32}|[A-Z_][A-Z0-9_]{2,32})',) \
+        $(if-expr !defined(VBOX_WITH_PYLINT_PRE_V1_0),--class-attribute-rgx='(k[abcdefhmnpilorstu]+[A-Z0-9][a-zA-Z0-9_]{0,32}|[A-Z_][A-Z0-9_]{2,32})',) \
         --disable=W0301,W0511,W0603,W0702,W0703,C0321,R0201,I0011 \
         --ignored-classes=testdriver.vboxcon \