Fix issue with Project Generator when optional build flags were passed using system environment variables

HISTORY.rst

@@ -4,6 +4,17 @@ Release Notes
 PlatformIO 2.0
 --------------
 
+2.8.5 (2016-02-??)
+~~~~~~~~~~~~~~~~~~
+
+* Fixed issue with incorrect handling of user's build flags where the base flags
+  were passed after user's flags to GCC compiler
+  (`issue #528 <https://github.com/platformio/platformio/issues/528>`_)
+* Fixed issue with Project Generator when optional build flags were passed using
+  system environment variables: `PLATFORMIO_BUILD_FLAGS <http://docs.platformio.org/en/latest/envvars.html#platformio-build-flags>`__
+  or `PLATFORMIO_BUILD_SRC_FLAGS <http://docs.platformio.org/en/latest/envvars.html#platformio-build-src-flags>`__
+
+
 2.8.4 (2016-02-17)
 ~~~~~~~~~~~~~~~~~~
 

platformio/__init__.py

@@ -14,7 +14,7 @@
 
 import sys
 
-VERSION = (2, 8, 4)
+VERSION = (2, 8, "5.dev0")
 __version__ = ".".join([str(s) for s in VERSION])
 
 __title__ = "platformio"

platformio/builder/tools/piomisc.py

@@ -228,6 +228,8 @@ def DumpIDEData(env):
     # https://github.com/platformio/platformio-atom-ide/issues/34
     _new_defines = []
     for item in env_.get("CPPDEFINES", []):
+        if isinstance(item, list):
+            item = "=".join(item)
         item = item.replace('\\"', '"')
         if " " in item:
             _new_defines.append(item.replace(" ", "\\\\ "))

platformio/telemetry.py

@@ -95,7 +95,7 @@ class MeasurementProtocol(TelemetryBase):
 
         # gather dependent packages
         dpdata = []
-        dpdata.append("Click/%s" % click.__version__)
+        dpdata.append("PlatformIO/%s" % __version__)
         if app.get_session_var("caller_id"):
             dpdata.append("Caller/%s" % app.get_session_var("caller_id"))
         if getenv("PLATFORMIO_IDE"):