On Wed, 2018-08-08 at 19:45 -0400, Cole Robinson wrote: > And clarify the comments to instead mention the package names > that provide these files Please split this into two patches - one that tweaks the existing comments and one that adds new entries. [...] > - # Nightly firmware repo x86/OVMF > + # Nightly edk2.git-ovmf-x64 > LOADERS="/usr/share/edk2.git/ovmf-x64/OVMF_CODE-pure-efi.fd:/usr/share/edk2.git/ovmf-x64/OVMF_VARS-pure-efi.fd" > - # Nightly firmware repo aarch64/AAVMF > + # Nightly edk2.git-ovmf-ia32 > + LOADERS="/usr/share/edk2.git/ovmf-ia32/OVMF_CODE-pure-efi.fd:/usr/share/edk2.git/ovmf-ia32/OVMF_VARS-pure-efi.fd" I'm pretty sure this won't work ;) > + # Nightly edk2.git-aarch64 > LOADERS="$LOADERS:/usr/share/edk2.git/aarch64/QEMU_EFI-pflash.raw:/usr/share/edk2.git/aarch64/vars-template-pflash.raw" It's kind of insane that the format for this is, and I quote, [...] list of pairs of <loader>:<nvram> paths. Both pairs and list items are separated by a colon. Of course it's too late to change it now, but I couldn't let such a perfect opportunity to complain about it just pass me by :) -- Andrea Bolognani / Red Hat / Virtualization -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
