diff options
| author |   Paul Eggleton <paul.eggleton@linux.intel.com> | 2016-12-22 16:13:57 +1300 |
|---|---|---|
| committer |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-01-09 13:34:29 +0000 |
| commit | 263af91a0efd21e041ecdb0c40f9b2d4e735f67d (patch) | |
| tree | 1d4cbea1036471fc12b1ab33fa2f7e970bf68d13 | |
| parent | 0a8abc7681edec5f128ceb757559c5a50f139a9c (diff) | |
| download | openembedded-core-contrib-263af91a0efd21e041ecdb0c40f9b2d4e735f67d.tar.gz | |
oe-selftest: fix behaviour if oe-selftest.log is a dangling symlink
If you delete the log file that the oe-selftest.log symlink points to
but not the symlink itself, because we were using os.path.exists() here
the code assumed that the symlink didn't exist when in fact it still
did. Use os.path.lexists() instead.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
| -rwxr-xr-x | scripts/oe-selftest | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/scripts/oe-selftest b/scripts/oe-selftest index e166521238..adfa92f707 100755 --- a/scripts/oe-selftest +++ b/scripts/oe-selftest @@ -62,7 +62,8 @@ log_prefix = "oe-selftest-" + t.strftime("%Y%m%d-%H%M%S") def logger_create(): log_file = log_prefix + ".log" - if os.path.exists("oe-selftest.log"): os.remove("oe-selftest.log") + if os.path.lexists("oe-selftest.log"): + os.remove("oe-selftest.log") os.symlink(log_file, "oe-selftest.log") log = logging.getLogger("selftest") |