From d8a0272bec319db198daccdcca7a3af8bdeadfa1 Mon Sep 17 00:00:00 2001
From: Ivan Kravets <me@ikravets.com>
Date: Sat, 24 Jun 2017 15:20:33 +0300
Subject: [PATCH] Escape non-valid file name characters when installing a new
 package (library) // Resolve #985

---
 HISTORY.rst                    | 2 ++
 platformio/managers/package.py | 9 +++++++--
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/HISTORY.rst b/HISTORY.rst
index 4592daf6..c4d0bdb1 100644
--- a/HISTORY.rst
+++ b/HISTORY.rst
@@ -40,6 +40,8 @@ PlatformIO 3.0
   `issue #952 <https://github.com/platformio/platformio-core/issues/952>`_)
 * Don't warn about known ``boards_dir`` option
   (`pull #949 <https://github.com/platformio/platformio-core/pull/949>`_)
+* Escape non-valid file name characters when installing a new package (library)
+  (`issue #985 <https://github.com/platformio/platformio-core/issues/985>`_)
 * Fixed infinite dependency installing when repository consists of multiple
   libraries
   (`issue #935 <https://github.com/platformio/platformio-core/issues/935>`_)
diff --git a/platformio/managers/package.py b/platformio/managers/package.py
index 9810a3d5..4e416392 100644
--- a/platformio/managers/package.py
+++ b/platformio/managers/package.py
@@ -126,6 +126,9 @@ class PkgRepoMixin(object):
 
 class PkgInstallerMixin(object):
 
+    PATH_ESCAPE_CHARS = ("/", "\\", "?", "%", "*", ":", "|", '"', "<", ">",
+                         ".", "(", ")", "&", "#", ",", "'")
+
     SRC_MANIFEST_NAME = ".piopkgmanager.json"
 
     FILE_CACHE_VALID = "1m"  # 1 month
@@ -189,9 +192,11 @@ class PkgInstallerMixin(object):
         fu = FileUnpacker(source_path, dest_dir)
         return fu.start()
 
-    @staticmethod
-    def get_install_dirname(manifest):
+    def get_install_dirname(self, manifest):
         name = manifest['name']
+        for c in self.PATH_ESCAPE_CHARS:
+            if c in name:
+                name = name.replace(c, "_")
         if "id" in manifest:
             name += "_ID%d" % manifest['id']
         return name