Oliver Kurz changed bug 1135865
What Removed Added
CC   xlai@suse.com
Flags   needinfo?(xlai@suse.com)

Comment # 16 on bug 1135865 from
Nothing I changed, had been introduced in tests originally in
1630d35cb for some PXE based tests. Are you arguing that xvideo does not exist
or is wrong?

I took the opportunity to "fix" that and created
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/7703
for the code change with the patch

```
--- a/lib/bootloader_setup.pm
+++ b/lib/bootloader_setup.pm
@@ -385,11 +385,9 @@ sub bootmenu_default_params {
         # gfxpayload variable replaced vga option in grub2
         if (!is_jeos && !is_caasp && (check_var('ARCH', 'i586') ||
check_var('ARCH', 'x86_64'))) {
             push @params, "vga=791";
-            if (check_var("INSTALL_TO_OTHERS", 1) || !$args{pxe}) {
-                push @params, "video=1024x768-16";
-            } else {
-                push @params, "xvideo=1024x768";
-            }
+            my $video = 'video=1024x768';
+            $video .= '-16' if check_var('QEMUVGA', 'cirrus');
+            push @params, $video;
         }

     }
```

so it should always be `video=1024x768` unless on "cirrus" where we add the
"-16".

@xlai as you initially introduced the "xvideo=" parameter which apparently does
not even exist for Linux could you please crosscheck the above change and/or
the PR?


You are receiving this mail because: