Skip to content

assemble_cvd: Support prebuilt android_esp.img in android_efi_loader - #3006

Open
zbw182 wants to merge 2 commits into
google:mainfrom
zbw182:feature/gbl-esp-boot
Open

assemble_cvd: Support prebuilt android_esp.img in android_efi_loader#3006
zbw182 wants to merge 2 commits into
google:mainfrom
zbw182:feature/gbl-esp-boot

Conversation

@zbw182

@zbw182 zbw182 commented Aug 12, 2026

Copy link
Copy Markdown

When --android_efi_loader is not explicitly specified on the command line, check for a prebuilt android_esp.img partition in system_image_dir (including system/etc/android_esp.img) before falling back to android_efi_loader.efi.

If android_esp.img is present, copy it directly to the instance ESP image path in BuildOSImage, bypassing host-side ESP generation via esp.cpp. If --android_efi_loader is explicitly passed (e.g. pointing to a standalone GBL .efi binary), preserve the existing dynamic generation behavior.

zbw182 and others added 2 commits August 11, 2026 17:05
…flow

When --android_efi_loader is not explicitly specified on the command line,
check for a prebuilt android_esp.img partition in system_image_dir
(including system/etc/android_esp.img) before falling back to
android_efi_loader.efi.

If android_esp.img is present, copy it directly to the instance ESP image
path in BuildOSImage, bypassing host-side ESP generation via esp.cpp.
If --android_efi_loader is explicitly passed (e.g. pointing to a standalone
GBL .efi binary), preserve the existing dynamic generation behavior.

Bug: 401619000
Test: assemble_cvd
Flag: EXEMPT host tool change
@Databean Databean added the kokoro:run Run e2e tests. label Aug 12, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label Aug 12, 2026
@Databean

Copy link
Copy Markdown
Member

FYI, it looks like there are some build/test errors.
From build-debian-package-amd64

 ERROR: /mnt/build/base/cvd/cuttlefish/host/libs/config/BUILD.bazel:147:14: Compiling cuttlefish/host/libs/config/data_image.cpp failed: (Exit 1): cc_wrapper.sh failed: error executing CppCompile command (from target //cuttlefish/host/libs/config:data_image) external/toolchains_llvm++llvm+llvm_toolchain/bin/cc_wrapper.sh -U_FORTIFY_SOURCE '--target=x86_64-unknown-linux-gnu' -U_FORTIFY_SOURCE -fstack-protector -fno-omit-frame-pointer -fcolor-diagnostics ... (remaining 166 arguments skipped)
  cuttlefish/host/libs/config/data_image.cpp:28:10: error: module //cuttlefish/host/libs/config:data_image does not directly depend on a module exporting 'absl/strings/match.h', which is part of indirectly-used module abseil-cpp+//absl/strings:strings
     28 | #include "absl/strings/match.h"
        |          ^
  1 error generated.

It's possible to reproduce these locally by running bazel test //... anywhere under the base/cvd directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants