mirror of
git://git.yoctoproject.org/poky.git
synced 2025-07-19 21:09:03 +02:00
sstate: Ensure SDE is accounted for in package task timestamps
When creating packages we build them with --clamp-mtime and use SOURCE_DATE_EPOCH as the maximum mtime. This makes the end packages reproducible. The data stored in sstate for do_package and the package task doesn't benefit from this though and have varying timestamps. This means their outhash varies and means hash equivalance isn't effective at all and doesn't work as intended/desired. We could create the sstate archives with the same clamping however that would lead to different results depending on whether a task was installed from sstate or not. Making that differ is a path to madness. It also wouldn't fix the outhash of the task to be determninistic without clamping of the date in the hash calculation code. Instead, iterate over the files in sstate output and clamp them at the code level. This isn't ideal but does make the file timestamps determnistic everywhere and means we don't have to change the hash calculation code. This issue can be clearly seen looking at the do_package outhash for a recipe which you then re-run the package task for after adding something like whitespace to the install task. The outhash shouldn't change but currently does. (From OE-Core rev: c3b3cc4745811b48b9193f83889946b2e1788932) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
95fa69f3d3
commit
6c5a29035b
|
@ -640,10 +640,21 @@ python sstate_hardcode_path () {
|
|||
|
||||
def sstate_package(ss, d):
|
||||
import oe.path
|
||||
import time
|
||||
|
||||
tmpdir = d.getVar('TMPDIR')
|
||||
|
||||
fixtime = False
|
||||
if ss['task'] == "package":
|
||||
fixtime = True
|
||||
|
||||
def fixtimestamp(root, path):
|
||||
f = os.path.join(root, path)
|
||||
if os.lstat(f).st_mtime > sde:
|
||||
os.utime(f, (sde, sde), follow_symlinks=False)
|
||||
|
||||
sstatebuild = d.expand("${WORKDIR}/sstate-build-%s/" % ss['task'])
|
||||
sde = int(d.getVar("SOURCE_DATE_EPOCH") or time.time())
|
||||
d.setVar("SSTATE_CURRTASK", ss['task'])
|
||||
bb.utils.remove(sstatebuild, recurse=True)
|
||||
bb.utils.mkdirhier(sstatebuild)
|
||||
|
@ -656,6 +667,8 @@ def sstate_package(ss, d):
|
|||
# to sstate tasks but there aren't many of these so better just avoid them entirely.
|
||||
for walkroot, dirs, files in os.walk(state[1]):
|
||||
for file in files + dirs:
|
||||
if fixtime:
|
||||
fixtimestamp(walkroot, file)
|
||||
srcpath = os.path.join(walkroot, file)
|
||||
if not os.path.islink(srcpath):
|
||||
continue
|
||||
|
@ -677,6 +690,11 @@ def sstate_package(ss, d):
|
|||
bb.utils.mkdirhier(plain)
|
||||
bb.utils.mkdirhier(pdir)
|
||||
bb.utils.rename(plain, pdir)
|
||||
if fixtime:
|
||||
fixtimestamp(pdir, "")
|
||||
for walkroot, dirs, files in os.walk(pdir):
|
||||
for file in files + dirs:
|
||||
fixtimestamp(walkroot, file)
|
||||
|
||||
d.setVar('SSTATE_BUILDDIR', sstatebuild)
|
||||
d.setVar('SSTATE_INSTDIR', sstatebuild)
|
||||
|
|
Loading…
Reference in New Issue
Block a user