search

kubesail / pibox-os

📦💻 The Official PiBox Operating System
https://pibox.io
54 stars 19 forks source link

update update-framebuffer.sh (for standalone use) #8

Closed iiidefix closed 2 years ago

iiidefix commented 2 years ago

remove unnecessary installation of lvm2 and parted instead install required font

iiidefix commented 2 years ago

additionally update README.md to only run update-framebuffer.sh

erulabs commented 2 years ago

Hey @iiidefix! Thanks for the PR. lvm2 and parted are required for the diskStats function (which powers our soon-to-be-released storage interface on https://kubesail.com). Would you mind re-adding those deps? Also, the chown line needs to be left in place.

iiidefix commented 2 years ago

Hey @erulabs! To be honest the diskStats feature feels misplaced in a frame buffer daemon. I'd love to see that functionality in a different service.

erulabs commented 2 years ago

Hey @iiidefix You're not at all wrong about that, although because the framebuffer is deeply tied in with the agent, we may wind up re-naming this project "pibox-host-daemon" or something more general. Possibly we should split it into two services - trying to Keep It Simple to start out.

iiidefix commented 2 years ago

Sorry I don't understand the requirements for kubesail.

erulabs commented 2 years ago

Sorry for the headache @iiidefix - I'll go ahead and add fonts-piboto to the installation packages. We'll get this cleaned up in the future for standalone use :)