-
Notifications
You must be signed in to change notification settings - Fork 153
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
Fix presets #281
Fix presets #281
Conversation
This fixes debian-pi#253, fixes debian-pi#277 and fixes debian-pi#279.
If you want to load kernel modules, this package is needed (afaik).
cdebootstrap_cmdline="$cdebootstrap_cmdline,${kernel_meta_package}" | ||
fi | ||
# minimal | ||
minimal_packages="fake-hwclock,ifupdown,net-tools,ntp,openssh-server" |
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.
I'd add ${base_packages}
into the list here, not in the include parameter
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.
although I'm not against it like it is, not to be misunderstood 😇
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.
I had it like that at first, but changed it into what it is now. Don't recall right now what the reasoning of it was ...
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.
cool
This PR re-implements the presets configuration so that a kernel is always installed as well as
cpufrequtils
to fully utilize the CPU.This fixes #253, #277, #279 and #271.