Skip to content

Commit 85c2476

Browse files
committed
check_logs.py: Fix fetch_dtb() after refactoring
I failed to notice that the "dtbs" folder is needed in the URL too, which breaks downloading the dtb. Pull the common path into its own variable so that it is more obvious how this is expected to work. Signed-off-by: Nathan Chancellor <[email protected]>
1 parent 66e4a28 commit 85c2476

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

check_logs.py

+3-2
Original file line numberDiff line numberDiff line change
@@ -112,8 +112,9 @@ def fetch_dtb(build):
112112
"multi_v5_defconfig": "aspeed-bmc-opp-palmetto.dtb",
113113
"aspeed_g5_defconfig": "aspeed-bmc-opp-romulus.dtb",
114114
}[config]
115-
(dtb_path := Path(CI_ROOT, 'dtbs', dtb)).parent.mkdir(exist_ok=True)
116-
url = build["download_url"] + dtb_path.name
115+
remote_dtb_path = f"dtbs/{dtb}"
116+
(dtb_path := Path(CI_ROOT, remote_dtb_path)).parent.mkdir(exist_ok=True)
117+
url = build["download_url"] + remote_dtb_path
117118
_fetch("DTB", url, dtb_path)
118119

119120

0 commit comments

Comments
 (0)