-
-
Notifications
You must be signed in to change notification settings - Fork 147
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
Linux support #152
Linux support #152
Conversation
Thanks a lot @franklefebvre for contributing this. I'll take a moment during this week to review it and do some QA. |
VirtualCore/Source/Virtualization/Helpers/LinuxVirtualMachineConfigurationHelper.swift
Outdated
Show resolved
Hide resolved
I don't see any major issues with the changes made in the PR, only minor tweaks we can do over time. However, I'm having trouble testing this. I'm getting an immediate shutdown of the Linux VM with "EFI Boot Options returned. Exiting VM" briefly shown on the display. Perhaps I got the wrong ISO? The one I tried was Future improvements (that I plan on doing myself, but wouldn't mind someone else PR'ing first :P)
|
Thank you @insidegui for reviewing my PR.
|
Duh, I totally missed that distinction 🤦♂️ Will try it out again with that one 😄 |
This is good to go for now, and I'm already working on some of the improvements I mentioned :) |
Oh and thanks again @franklefebvre for this fantastic contribution 🥳 |
Tested Fedora 38 workstation arm on my macbook m1 and it works well. Thanks for adding Linux! @franklefebvre @insidegui |
This update allows to install and boot from some ARM-based Linux distributions (tested successfully with many versions of Ubuntu -- couldn't make it work with CentOS so far.)
The option is visible only when available (macOS 13+)
To install a Linux system:
Some classes/structs need to be renamed (VBMacConfiguration etc). I chose to keep them unmodified for now in order not to add too much noise to this MR.