From 03788d14fb0dd79b38817d5a40f1c5c2017f4551 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Mon, 4 Sep 2023 01:17:23 +0100 Subject: move ifd/gbe configs into config/ifd/ it doesn't really make sense for them to be under blobs/ - nominally, they are blobs, but they are well-understood data files containing config data, that is easily parsed by tools like ich9show or ifdtool (and tools like bincfg or nvmutil) blobs/ has been re-purposed: this directory no longer exists in lbmk, but it is created (and on .gitignore) when needed, by blobutil thus, the blobs/ directory shall only contain vendor files, and only those files that libreboot scrubs from releases. therefore, build/release/src can (and has been) simplified; it currently copies just the ifd and gbe files from blobs/, selectively, and this logic is quite error prone, requiring maintenance. now, the build/release/src script simply copies config/ (which only ever contains distributable files) and entirely ignores the blobs/ directory the blob download script already creates the required directory, except for the sch5545 download; this is now fixed lbmk code size is slightly smaller, due to this patch Signed-off-by: Leah Rowe --- script/update/blobs/download | 3 +++ 1 file changed, 3 insertions(+) (limited to 'script/update/blobs') diff --git a/script/update/blobs/download b/script/update/blobs/download index 012cf0b1..05730c4b 100755 --- a/script/update/blobs/download +++ b/script/update/blobs/download @@ -423,6 +423,9 @@ extract_sch5545ec() printf "sch5545 firmware already downloaded\n" 1>&2 return 0 fi + [ -d "${_sch5545ec_destination%/*}" ] || \ + mkdir -p "${_sch5545ec_destination%/*}" || \ + err "extract_sch: can't mkdir ${_sch5545ec_destination%/*}" [ ! -d "${appdir}" ] || rm -Rf "${appdir}" || \ err "extract_sch5545ec: can't remove ${appdir}" -- cgit v1.2.1