buildhistory: use bb.utils.mkdirhier instead of os.makedirs
* Multiple do_fetch[prefunc] can happen at the same time and if not os.path.exists(pkghistdir): os.makedirs(pkghistdir) isn't safe * Use bb.utils.mkdirhier which doesn't raise error when directory exists (From OE-Core rev: 9aac194d7db79129dcbed29cfb89c57dccf33729) Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
560dac6e7d
commit
5c7b8fc342
|
@ -169,7 +169,7 @@ python buildhistory_emit_pkghistory() {
|
|||
|
||||
packagelist = packages.split()
|
||||
if not os.path.exists(pkghistdir):
|
||||
os.makedirs(pkghistdir)
|
||||
bb.utils.mkdirhier(pkghistdir)
|
||||
else:
|
||||
# Remove files for packages that no longer exist
|
||||
for item in os.listdir(pkghistdir):
|
||||
|
@ -268,7 +268,7 @@ def write_pkghistory(pkginfo, d):
|
|||
|
||||
pkgpath = os.path.join(pkghistdir, pkginfo.name)
|
||||
if not os.path.exists(pkgpath):
|
||||
os.makedirs(pkgpath)
|
||||
bb.utils.mkdirhier(pkgpath)
|
||||
|
||||
infofile = os.path.join(pkgpath, "latest")
|
||||
with open(infofile, "w") as f:
|
||||
|
@ -579,7 +579,7 @@ python write_srcrev() {
|
|||
srcrevs, tag_srcrevs = _get_srcrev_values(d)
|
||||
if srcrevs:
|
||||
if not os.path.exists(pkghistdir):
|
||||
os.makedirs(pkghistdir)
|
||||
bb.utils.mkdirhier(pkghistdir)
|
||||
old_tag_srcrevs = {}
|
||||
if os.path.exists(srcrevfile):
|
||||
with open(srcrevfile) as f:
|
||||
|
|
Loading…
Reference in New Issue