summaryrefslogtreecommitdiffstats
path: root/bitbake/lib/bb/fetch2
diff options
context:
space:
mode:
authorMartin Jansa <martin.jansa@gmail.com>2019-09-03 07:49:57 +0000
committerRichard Purdie <richard.purdie@linuxfoundation.org>2019-09-03 10:18:58 +0100
commitbf2e0b039a036f3e3f21d05c9dff843d0d293994 (patch)
tree9601b9ae01ce4c7ae82927aa4035a9dbc65884db /bitbake/lib/bb/fetch2
parent3e21648723c0b2e0c806fc5a446a1ca253c9a62f (diff)
downloadpoky-bf2e0b039a036f3e3f21d05c9dff843d0d293994.tar.gz
bitbake: fetch2: show warning when renaming the archive with bad checksum failed
* noticed on read-only sshfs premirror * it was showing the warning about renaming the file: WARNING: laser-geometry-1.6.4-r0 do_fetch: Renaming /jenkins/mjansa/sshfs/webos-ose-thud/downloads/laser_geometry-1.6.4.tar.gz to /jenkins/mjansa/sshfs/webos-ose-thud/downloads/laser_geometry-1.6.4.tar.gz_bad-checksum_1ee7479b8c5914b4ffae996945121441 and then failed because of movefile() issue with python3 (fixed in previous commit): ERROR: laser-geometry-1.6.4-r0 do_fetch: Error executing a python function in exec_python_func() autogenerated: with movefile() fixed, it let do_fetch continue and re-fetch locally with the right checksum, but still the renamed file didn't exist, because of movefile failure - add another warning when the movefile fails - for whatever reason - unfortunately movefile prints error messages with just print() so the real error is hidden only in log.do_fetch in this case: movefile: Failed to move /jenkins/mjansa/sshfs/webos-ose-thud/downloads/laser_geometry-1.6.4.tar.gz to /jenkins/mjansa/sshfs/webos-ose-thud/downloads/laser_geometry-1.6.4.tar.gz_bad-checksum_1ee7479b8c5914b4ffae996945121441 [Errno 30] Read-only file system: '/jenkins/mjansa/sshfs/webos-ose-thud/downloads/laser_geometry-1.6.4.tar.gz' -> '/jenkins/mjansa/sshfs/webos-ose-thud/downloads/laser_geometry-1.6.4.tar.gz_bad-checksum_1ee7479b8c5914b4ffae996945121441' (Bitbake rev: 9a1bf4ba9ec00c2a222d820f8f83d1f056b021d6) Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib/bb/fetch2')
-rw-r--r--bitbake/lib/bb/fetch2/__init__.py3
1 files changed, 2 insertions, 1 deletions
diff --git a/bitbake/lib/bb/fetch2/__init__.py b/bitbake/lib/bb/fetch2/__init__.py
index f6b5529bb8..1f5f8f1f14 100644
--- a/bitbake/lib/bb/fetch2/__init__.py
+++ b/bitbake/lib/bb/fetch2/__init__.py
@@ -962,7 +962,8 @@ def rename_bad_checksum(ud, suffix):
962 962
963 new_localpath = "%s_bad-checksum_%s" % (ud.localpath, suffix) 963 new_localpath = "%s_bad-checksum_%s" % (ud.localpath, suffix)
964 bb.warn("Renaming %s to %s" % (ud.localpath, new_localpath)) 964 bb.warn("Renaming %s to %s" % (ud.localpath, new_localpath))
965 bb.utils.movefile(ud.localpath, new_localpath) 965 if not bb.utils.movefile(ud.localpath, new_localpath):
966 bb.warn("Renaming %s to %s failed, grep movefile in log.do_fetch to see why" % (ud.localpath, new_localpath))
966 967
967 968
968def try_mirror_url(fetch, origud, ud, ld, check = False): 969def try_mirror_url(fetch, origud, ud, ld, check = False):