-
Notifications
You must be signed in to change notification settings - Fork 7
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
Update to Debian 11 #176
base: master
Are you sure you want to change the base?
Update to Debian 11 #176
Conversation
Vagrantfile
Outdated
echo deb http://httpredir.debian.org/debian/ bullseye main > /etc/apt/sources.list | ||
echo deb-src http://httpredir.debian.org/debian/ bullseye main >> /etc/apt/sources.list | ||
# echo deb http://security.debian.org/debian-security bullseye/updates main >> /etc/apt/sources.list | ||
# echo deb-src http://security.debian.org/debian-security bullseye/updates main >> /etc/apt/sources.list |
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.
Any particular reason the security updates are commented out?
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.
No not really, I guess I just didn't think I needed them for just building this. I'll enable them again, probably a good thing to have.
@@ -29,13 +31,15 @@ Vagrant.configure(2) do |config| | |||
nasm \ | |||
psmisc \ | |||
qemu \ | |||
gcc-arm-none-eabi \ |
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.
Interesting... Are you running this on an M1/M2/M3-based Mac perhaps? Or some other ARM-based machine?
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.
(ah, was already present elsewhere from before.. 🙂)
Vagrantfile
Outdated
sudo DEBIAN_FRONTEND=noninteractive apt-get install -y gcc-arm-none-eabi | ||
#echo deb http://httpredir.debian.org/debian/ unstable main >> /etc/apt/sources.list | ||
#sudo apt-get update | ||
#sudo DEBIAN_FRONTEND=noninteractive apt-get install -y gcc-arm-none-eabi |
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.
Perhaps we can remove both the commented out code and the comment before? Since the package is now available (in newer Debian versions).
Looks good @Gapstare, thanks for the effort! 🌟 I gave some initial feedback now. I think we can merge this once we've addressed these points. We'll have to live with the |
Ok, how about now? 🙂 |
Update to use Debian 11 to build with Vagrant. This breaks modplay functionality.