Thanks for the patch, pushed to git now. Currently blocked from rawhide until we fix https://bugzilla.redhat.com/show_bug.cgi?id=992180 though.

Oh, damn, this needs to build on the stone-age 32bit arch? This is still
useful and used?

If that is really needed, we should try to fix up the build, at the
moment, only 64bit builds are supported. Please let me know, I can take
a look ...

I don't know why ARM was added at all to the target arches. In dist-git, it appeared in commit

commit 1516367bfab219109804409e5300ecdd1bcb4254
Author: Dan Horák <dan>
Date:   Thu May 23 08:16:31 2013 +0200

    set ExclusiveArch

without any explanation at all.

+ExclusiveArch: %{ix86} x86_64 %{arm}

(In reply to Laszlo Ersek from comment #3)
> I don't know why ARM was added at all to the target arches. In dist-git, it
> appeared in commit
> 
> commit 1516367bfab219109804409e5300ecdd1bcb4254
> Author: Dan Horák <dan>
> Date:   Thu May 23 08:16:31 2013 +0200
> 
>     set ExclusiveArch
> 
> without any explanation at all.
> 
> +ExclusiveArch: %{ix86} x86_64 %{arm}

Dan works on fedora s390. edk2 likely didn't build there, so he added the ExclusiveArch, but just included every other architecture. So the package may not make sense for arm.

Long term, if we build OVMF, edk2 needs to only be compiled for x86, but the package needs to be available on all architectures (so qemu-system-x86 is available on all architectures). To do that we need to cross-compile, and that makes the package noarch which means we can drop the ExclusiveArch handling.

The EFI shell should be in a noarch subpackage.  If Kay wants both 32-bit and 64-bit shell binaries, he should build the ShellPkg twice.  So in fact I suggest reverting both of Kay's patches.

(In reply to Paolo Bonzini from comment #5)
> The EFI shell should be in a noarch subpackage.  If Kay wants both 32-bit
> and 64-bit shell binaries, he should build the ShellPkg twice.  So in fact I
> suggest reverting both of Kay's patches.

I'm trying to understand. Why should efi-shell be inside a noarch package? is it an external tool, or something run inside the guest? My understanding is that we only do 'compile for x86 and stuff in a noarch package' for things that we pass into a guest from the qemu command line or similar (maybe that covers efi-shell but I'm ignorant here and googling didn't explain things).

The EFI shell is not needed inside the guest, since the OVMF ROM already includes it.  My guest is that Kay wants an easy way to get an EFI shell for his computer.

It has to be noarch because it doesn't include any Linux binaries.  You could even use a 64-bit shell on a 32-bit installation, that would make sense.

This bug appears to have been reported against 'rawhide' during the Fedora 20 development cycle.
Changing version to '20'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora20

This message is a reminder that Fedora 20 is nearing its end of life.
Approximately 4 (four) weeks from now Fedora will stop maintaining
and issuing updates for Fedora 20. It is Fedora's policy to close all
bug reports from releases that are no longer maintained. At that time
this bug will be closed as EOL if it remains open with a Fedora  'version'
of '20'.

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, simply change the 'version' 
to a later Fedora version.

Thank you for reporting this issue and we are sorry that we were not 
able to fix it before Fedora 20 is end of life. If you would still like 
to see this bug fixed and are able to reproduce it against a later version 
of Fedora, you are encouraged  change the 'version' to a later Fedora 
version prior this bug is closed as described in the policy above.

Although we aim to fix as many bugs as possible during every release's 
lifetime, sometimes those efforts are overtaken by events. Often a 
more recent Fedora release includes newer upstream software that fixes 
bugs or makes them obsolete.

Still relevant with latest packages

Gerd, I see edk2-ovmf has /usr/share/edk2/ovmf/Shell.efi , does that cover this request?

(In reply to Cole Robinson from comment #11)
> Gerd, I see edk2-ovmf has /usr/share/edk2/ovmf/Shell.efi , does that cover
> this request?

Yes.

Thanks. This is in f24/rawhide then. Will probably push to f23 eventually