]> code.ossystems Code Review - openembedded-core.git/commit
bb.utils: check if lock file is writable, to fix bug 606
authorYu Ke <ke.yu@intel.com>
Wed, 29 Dec 2010 07:28:48 +0000 (15:28 +0800)
committerRichard Purdie <rpurdie@linux.intel.com>
Thu, 30 Dec 2010 09:28:45 +0000 (09:28 +0000)
commit6ee0c26e21f48dcd47af88ed5c174e76e76a3e42
tree11b741182e8a2e2cb5a105dd74c9de15f07cf3ff
parent93043c55dd3ea0005801bd692f9ec31d86f5aec5
bb.utils: check if lock file is writable, to fix bug 606

Bug 606 report that if $DL_DIR is read-only, do_fetch will
simply hang without any error message.

The root cause is that: bb.fetch.go()->bb.utils.lockfile()
will try to lock file ${DL_DIR}/xxxxx.lock. Since ${DL_DIR}
is read-only, it will cause IOError exception. Although
lockfile() can catch the exception, currently code simply
ignore all the exception and continue the loop. it make
sense if the exception is caused by locking contention,
but in the read-only $DL_DIR case, it cause endless waiting
unfortunately.

So this patch add read-only check for lockfile to avoid the
silent hang.

Fix [BUGID #606]

Signed-off-by: Yu Ke <ke.yu@intel.com>
bitbake/lib/bb/utils.py