From d38968c4fca254dd478be8a54881b93ff29fadb5 Mon Sep 17 00:00:00 2001 From: Chris Larson Date: Tue, 8 Mar 2011 09:33:40 -0700 Subject: lockfile: ask for forgiveness, not permission Create the lockfile directory if it doesn't exist, rather than erroring out if it doesn't exist (was also racy). Also improve the wording of the error message shown when the lockfile's directory is not writable. Note for the future, this function should be improved, particularly with regard to its exception handling. It should be catching the *exact* exception(s) it will encounter when the file is locked, and continuing in that case only. If it did that, there'd be no need for the proactive directory writability check, as bb.utils.lockfile() would raise an appropriate IOError for that case. (Bitbake rev: 238151441c74db53d6e4d4753f4f96c32f6f13b6) Signed-off-by: Chris Larson Signed-off-by: Richard Purdie --- bitbake/lib/bb/utils.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'bitbake') diff --git a/bitbake/lib/bb/utils.py b/bitbake/lib/bb/utils.py index a6f1337..f49ee55 100644 --- a/bitbake/lib/bb/utils.py +++ b/bitbake/lib/bb/utils.py @@ -407,13 +407,12 @@ def lockfile(name, shared=False): Use the file fn as a lock file, return when the lock has been acquired. Returns a variable to pass to unlockfile(). """ - path = os.path.dirname(name) - if not os.path.isdir(path): - logger.error("Lockfile destination directory '%s' does not exist", path) - sys.exit(1) + dirname = os.path.dirname(name) + mkdirhier(dirname) - if not os.access(path, os.W_OK): - logger.error("Error, lockfile path is not writable!: %s" % path) + if not os.access(dirname, os.W_OK): + logger.error("Unable to acquire lock '%s', directory is not writable", + dirname) sys.exit(1) op = fcntl.LOCK_EX @@ -449,7 +448,7 @@ def unlockfile(lf): Unlock a file locked using lockfile() """ try: - # If we had a shared lock, we need to promote to exclusive before + # If we had a shared lock, we need to promote to exclusive before # removing the lockfile. Attempt this, ignore failures. fcntl.flock(lf.fileno(), fcntl.LOCK_EX|fcntl.LOCK_NB) os.unlink(lf.name) -- cgit v1.1