[PATCH v2 07/39] efi: Add a script for building and testing U-Boot on UEFI

[Simon Glass]

Hi Heinrich,

On Mon, 25 Oct 2021 at 13:40, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
>
>
> On 9/25/21 02:30, Simon Glass wrote:
> > It is quite complicating to run U-Boot on qemu since we have four
> > different builds and they must use different versions of qemu and the
> > UEFI binaries.
> >
> > Add a script to help.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
>
> This patch seems not to work:
>
> $ scripts/build-efi.sh -wsPr
> Packaging efi-x86_app32
> cp: cannot stat '/tmp/b/efi-x86_app32/u-boot-app.efi': No such file or
> director
>
> The scripts lacks a step to build the EFI binary.
>
> If you expect the user to build it, you will find it in the current
> directory and not in some arbitrary subdirectory of /tmp.

Did you read the top of the script? It does not do the build, just the
package/run bit
>
> Temporary directories should be created with 'mktemp -d'. Using fixed
> paths (e.g. /tmp/b/) may lead to conflicts.

I'll make it a variable in v3. But we cannot use mktemp-d as the build
needs to stay around for multiple runs of this script.

Regards,
Simon