On 31/08/2021 17:50, John Arbuckle wrote:

Currently the file qemu_vga.ndrv is not copied into the /build/pc-bios folder. 
This makes all video resolution choices disappear from a PowerPC Mac OS guest. 
This patch has the qemu_vga.ndrv file copied into the build/pc-bios folder 
giving users back their video resolution choices.

Signed-off-by: John Arbuckle <programmingk...@gmail.com>
---
  configure | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/configure b/configure
index 9a79a004d7..281577e46f 100755
--- a/configure
+++ b/configure
@@ -5056,7 +5056,9 @@ for bios_file in \
      $source_path/pc-bios/openbios-* \
      $source_path/pc-bios/u-boot.* \
      $source_path/pc-bios/edk2-*.fd.bz2 \
-    $source_path/pc-bios/palcode-*
+    $source_path/pc-bios/palcode-* \
+    $source_path/pc-bios/qemu_vga.ndrv
+
  do
      LINKS="$LINKS pc-bios/$(basename $bios_file)"
  done

I think the patch looks correct, however the commit message and subject aren't quite right. How about something like:

[PATCH] configure: add missing pc-bios/qemu_vga.ndrv symlink in build tree

Ensure that a link to pc-bios/qemu_vga.ndrv is added to the build tree, otherwise the optional MacOS client driver will not be loaded by OpenBIOS when launching QEMU directly from the build directory.


ATB,

Mark.

Reply via email to