-
Notifications
You must be signed in to change notification settings - Fork 621
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow overriding qemu binary ($QEMU_SYSTEM_X86_64, $QEMU_SYSTEM_AARCH64) #80
Conversation
69ff2e7
to
866b602
Compare
I think it would be good if Maybe write a custom |
BTW, providing options to qemu for debugging without recompiling lima is the main point of this change for me. For providing the location of an alternate qemu binary |
Passing args is also possible (only for debugging purpose), e.g., "QEMU_SYSTEM_X86_64=/opt/qemu/bin/qemu-system-x86_64 -snapshot" . Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
866b602
to
75443f8
Compare
I'm not sure we want to support overriding these flags, as most of them are configurable via the YAML, though I guess it wouldn't hurt to support overriding A workaround is to specify a custom script like this via the env. #!/bin/sh
set -eu
exec qemu-system-x86_64 (echo $@ | sed -e 's/-cpu [^ ]* /-cpu SOMETHING /') |
e.g. QEMU_SYSTEM_X86_64="qemu-system-x86_64 -cpu host" Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
added commit |
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Also tested with QEMU_SYSTEM_X86_64="qemu-system-x86_64 -display cocoa" limactl start --tty=false
and worked as expected.
Passing args is also possible (only for debugging purpose), e.g., "QEMU_SYSTEM_X86_64=/opt/qemu/bin/qemu-system-x86_64 -snapshot" .
Fix #27
Fix #14