changeset 7895:0288dd052033

vcs: drop unused lockfiles.py It popped up when investigating other things - make sure we don't waste more time on it.
author Mads Kiilerich <mads@kiilerich.com>
date Thu, 03 Oct 2019 23:02:07 +0200
parents f00117816704
children 8dbe46ca608f
files kallithea/lib/vcs/utils/lockfiles.py scripts/whitespacecleanup.sh
diffstat 2 files changed, 1 insertions(+), 73 deletions(-) [+]
line wrap: on
line diff
--- a/kallithea/lib/vcs/utils/lockfiles.py	Fri Sep 27 01:12:38 2019 +0200
+++ /dev/null	Thu Jan 01 00:00:00 1970 +0000
@@ -1,72 +0,0 @@
-import os
-
-
-class LockFile(object):
-    """Provides methods to obtain, check for, and release a file based lock which
-    should be used to handle concurrent access to the same file.
-
-    As we are a utility class to be derived from, we only use protected methods.
-
-    Locks will automatically be released on destruction"""
-    __slots__ = ("_file_path", "_owns_lock")
-
-    def __init__(self, file_path):
-        self._file_path = file_path
-        self._owns_lock = False
-
-    def __del__(self):
-        self._release_lock()
-
-    def _lock_file_path(self):
-        """:return: Path to lockfile"""
-        return "%s.lock" % (self._file_path)
-
-    def _has_lock(self):
-        """:return: True if we have a lock and if the lockfile still exists
-        :raise AssertionError: if our lock-file does not exist"""
-        if not self._owns_lock:
-            return False
-
-        return True
-
-    def _obtain_lock_or_raise(self):
-        """Create a lock file as flag for other instances, mark our instance as lock-holder
-
-        :raise IOError: if a lock was already present or a lock file could not be written"""
-        if self._has_lock():
-            return
-        lock_file = self._lock_file_path()
-        if os.path.isfile(lock_file):
-            raise IOError("Lock for file %r did already exist, delete %r in case the lock is illegal" % (self._file_path, lock_file))
-
-        try:
-            fd = os.open(lock_file, os.O_WRONLY | os.O_CREAT | os.O_EXCL, 0)
-            os.close(fd)
-        except OSError as e:
-            raise IOError(str(e))
-
-        self._owns_lock = True
-
-    def _obtain_lock(self):
-        """The default implementation will raise if a lock cannot be obtained.
-        Subclasses may override this method to provide a different implementation"""
-        return self._obtain_lock_or_raise()
-
-    def _release_lock(self):
-        """Release our lock if we have one"""
-        if not self._has_lock():
-            return
-
-        # if someone removed our file beforehand, lets just flag this issue
-        # instead of failing, to make it more usable.
-        lfp = self._lock_file_path()
-        try:
-            # on bloody windows, the file needs write permissions to be removable.
-            # Why ...
-            if os.name == 'nt':
-                os.chmod(lfp, 0777)
-            # END handle win32
-            os.remove(lfp)
-        except OSError:
-            pass
-        self._owns_lock = False
--- a/scripts/whitespacecleanup.sh	Fri Sep 27 01:12:38 2019 +0200
+++ b/scripts/whitespacecleanup.sh	Thu Oct 03 23:02:07 2019 +0200
@@ -2,7 +2,7 @@
 
 # Enforce some consistency in whitespace - just to avoid spurious whitespaces changes
 
-files=`hg mani | egrep -v '/fontello/|/email_templates/|(/lockfiles.py|^LICENSE-MERGELY.html|^docs/Makefile|^scripts/whitespacecleanup.sh|/(graph|mergely|native.history)\.js|/test_dump_html_mails.ref.html|\.png|\.gif|\.ico|\.pot|\.po|\.mo|\.tar\.gz|\.diff)$'`
+files=`hg mani | egrep -v '/fontello/|/email_templates/|(^LICENSE-MERGELY.html|^docs/Makefile|^scripts/whitespacecleanup.sh|/(graph|mergely|native.history)\.js|/test_dump_html_mails.ref.html|\.png|\.gif|\.ico|\.pot|\.po|\.mo|\.tar\.gz|\.diff)$'`
 
 sed -i "s/`printf '\r'`//g" $files
 sed -i -e "s,`printf '\t'`,    ,g" $files