Merge pull request #111 from ARMmbed/devel_lockfile_timeout

Replace lockfile context mgr usage with lock.aquire() and lock.release()

Author: PrzemekWirkus

mbed_lstools/lstools_base.py

@@ -22,7 +22,7 @@
 import lockfile
 from os import listdir
 from os.path import isfile, join
-from lockfile import LockFailed
+from lockfile import LockFailed, LockTimeout
 
 class MbedLsToolsBase:
     """ Base class for mbed-lstools, defines mbed-ls tools interface for mbed-enabled devices detection for various hosts
@@ -252,16 +252,21 @@ def read_mock_file(filename):
             return {}
 
         try:
-            with self.mbedls_get_global_lock():
+            lock = self.mbedls_get_global_lock()
+            if lock.acquire(timeout=0.5):
                 # This read is for backward compatibility
                 # When user already have on its system local mock-up it will work
                 # overwriting global one
                 if isfile(self.MOCK_FILE_NAME):
-                    return read_mock_file(self.MOCK_FILE_NAME)
+                    ret = read_mock_file(self.MOCK_FILE_NAME)
+                    lock.release()
+                    return ret
 
                 if isfile(self.MOCK_HOME_FILE_NAME):
-                    return read_mock_file(self.MOCK_HOME_FILE_NAME)
-        except LockFailed as e:
+                    ret = read_mock_file(self.MOCK_HOME_FILE_NAME)
+                    lock.release()
+                    return ret
+        except (LockFailed, LockTimeout) as e:
             self.err(str(e))
         return {}
 
@@ -284,9 +289,12 @@ def write_mock_file(filename, mock_ids):
             return False
 
         try:
-            with self.mbedls_get_global_lock():
-                return write_mock_file(self.MOCK_HOME_FILE_NAME, mock_ids)
-        except LockFailed as e:
+            lock = self.mbedls_get_global_lock()
+            if lock.acquire(timeout=0.5):
+                ret = write_mock_file(self.MOCK_HOME_FILE_NAME, mock_ids)
+                lock.release()
+                return ret
+        except (LockFailed, LockTimeout) as e:
             self.err(str(e))
         return False