forked from lix-project/lix
Merge changes I9cf007c8,I9b9ba058 into main
* changes: releng: fix broken manifest from 2.90-rc1 Change the sqlite missing valid path message to say it's the db
This commit is contained in:
commit
6c541e0bef
|
@ -106,17 +106,17 @@ def upload_drv_paths_and_outputs(env: RelengEnvironment, paths: list[str]):
|
|||
raise subprocess.CalledProcessError(rv, proc.args)
|
||||
|
||||
|
||||
def make_manifest(eval_result):
|
||||
manifest = {vs['system']: vs['outputs']['out'] for vs in eval_result}
|
||||
def make_manifest(builds_by_system):
|
||||
|
||||
def manifest_line(system, out):
|
||||
return f' {system} = "{out}";'
|
||||
|
||||
manifest_text = textwrap.dedent("""\
|
||||
# This file was generated by releng/create_release.xsh in Lix
|
||||
{{
|
||||
{lines}
|
||||
{lines}
|
||||
}}
|
||||
""").format(lines='\n'.join(manifest_line(s, p) for (s, p) in manifest.items()))
|
||||
""").format(lines='\n'.join(manifest_line(s, p) for (s, p) in builds_by_system.items()))
|
||||
|
||||
return manifest_text
|
||||
|
||||
|
@ -142,6 +142,18 @@ def sha256_file(f: Path):
|
|||
return hasher.hexdigest()
|
||||
|
||||
|
||||
def extract_builds_by_system(eval_result):
|
||||
# This could be a dictionary comprehension, but we want to be absolutely
|
||||
# sure we don't have duplicates.
|
||||
ret = {}
|
||||
for attr in eval_result:
|
||||
if attr['attrPath'][0] != 'build':
|
||||
continue
|
||||
assert attr['system'] not in ret
|
||||
ret[attr['system']] = attr['outputs']['out']
|
||||
return ret
|
||||
|
||||
|
||||
def make_artifacts_dir(eval_result, d: Path):
|
||||
d.mkdir(exist_ok=True, parents=True)
|
||||
version_dir = d / 'lix' / f'lix-{VERSION}'
|
||||
|
@ -150,12 +162,14 @@ def make_artifacts_dir(eval_result, d: Path):
|
|||
tarballs_drv = next(p for p in eval_result if p['attr'] == 'tarballs')
|
||||
cp --no-preserve=mode -r @(tarballs_drv['outputs']['out'])/* @(version_dir)
|
||||
|
||||
builds_by_system = extract_builds_by_system(eval_result)
|
||||
|
||||
# FIXME: upgrade-nix searches for manifest.nix at root, which is rather annoying
|
||||
with open(d / 'manifest.nix', 'w') as h:
|
||||
h.write(make_manifest(eval_result))
|
||||
h.write(make_manifest(builds_by_system))
|
||||
|
||||
with open(version_dir / 'manifest.nix', 'w') as h:
|
||||
h.write(make_manifest(eval_result))
|
||||
h.write(make_manifest(builds_by_system))
|
||||
|
||||
print('[+] Make sources tarball')
|
||||
|
||||
|
|
|
@ -958,7 +958,7 @@ uint64_t LocalStore::queryValidPathId(State & state, const StorePath & path)
|
|||
{
|
||||
auto use(state.stmts->QueryPathInfo.use()(printStorePath(path)));
|
||||
if (!use.next()) // TODO: I guess if SQLITE got corrupted..?
|
||||
throw InvalidPath("path '%s' does not exist", printStorePath(path));
|
||||
throw InvalidPath("path '%s' does not exist in the Lix database", printStorePath(path));
|
||||
return use.getInt(0);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue