Fixed an issue when library keeps reinstalling for non-latin path // Resolve #1252

HISTORY.rst

@@ -26,6 +26,7 @@ PlatformIO 4.0
   - Install all project dependencies declared via `lib_deps <http://docs.platformio.org/page/projectconf/section_env_library.html#lib-deps>`__ option using a simple `platformio lib install <http://docs.platformio.org/page/userguide/lib/cmd_install.html>`__ command (`issue #2147 <https://github.com/platformio/platformio-core/issues/2147>`_)
   - Use isolated library dependency storage per project build environment (`issue #1696 <https://github.com/platformio/platformio-core/issues/1696>`_)
   - Override default source and include directories for a library via `library.json <http://docs.platformio.org/page/librarymanager/config.html>`__ manifest using ``includeDir`` and ``srcDir`` fields
+  - Fixed an issue when library keeps reinstalling for non-latin path (`issue #1252 <https://github.com/platformio/platformio-core/issues/1252>`_)
 
 * **Build System**
 

platformio/managers/package.py

@@ -26,7 +26,7 @@ import requests
 import semantic_version
 
 from platformio import __version__, app, exception, telemetry, util
-from platformio.compat import hashlib_encode_data, path_to_unicode
+from platformio.compat import hashlib_encode_data
 from platformio.downloader import FileDownloader
 from platformio.lockfile import LockFile
 from platformio.unpacker import FileUnpacker
@@ -394,7 +394,7 @@ class PkgInstallerMixin(object):
         if "version" not in manifest:
             manifest['version'] = "0.0.0"
 
-        manifest['__pkg_dir'] = path_to_unicode(pkg_dir)
+        manifest['__pkg_dir'] = pkg_dir
         self.cache_set(cache_key, manifest)
         return manifest
 
@@ -451,7 +451,7 @@ class PkgInstallerMixin(object):
 
     def get_package_by_dir(self, pkg_dir):
         for manifest in self.get_installed():
-            if manifest['__pkg_dir'] == path_to_unicode(abspath(pkg_dir)):
+            if manifest['__pkg_dir'] == abspath(pkg_dir):
                 return manifest
         return None