search

broozar / installDesktopFreeBSD

FreeBSD desktop installer
GNU General Public License v3.0
98 stars 23 forks source link

sed -> sysrc #2

Closed nicholasbernstein closed 3 years ago

nicholasbernstein commented 3 years ago

I was looking through your script as inspiration for a similar one I put together which installs various window environments, and noticed that you were editing the rc.conf directly, and the kld_list part could blow away other modules listed to be loaded, so I changed it to use sysrc kld_list+="module" and updated the earlier section where you turn stuff like moused on to use sysrc as well since I was in there.

nicholasbernstein commented 3 years ago

np, it looked like you didn't want it enabled because it's deprecated. I didn't want to mess with your logic, just thought sysrc was nicer.