Skip to content
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

FAI-CD: Add Grub2 background image + DejaVuSansMono font. #124

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dzatoah
Copy link

@dzatoah dzatoah commented Jul 3, 2024

Background image: /usr/share/images/desktop-base/desktop-grub.png
Font file: /usr/share/fonts/truetype/dejavu/DejaVuSansMono.ttf

Needed packages: grub2-common desktop-base fonts-dejavu-core

See https://salsa.debian.org/debian-edu/debian-edu-router/-/merge_requests/236 (needs activated salsa account to see)

This patch is based on commit 2f2e63b0dbe932e872772829dd6d4d0213e3db58 (tag: 6.0.1). The current Debian version.

@dzatoah dzatoah force-pushed the mr/add-background-image-and-font-to-grub2-memdisk branch from f0b4a6f to bbe730f Compare July 3, 2024 12:44
bin/fai-cd Outdated Show resolved Hide resolved
bin/fai-cd Outdated Show resolved Hide resolved
bin/fai-cd Outdated Show resolved Hide resolved
bin/fai-cd Outdated Show resolved Hide resolved
bin/fai-cd Outdated Show resolved Hide resolved
bin/fai-cd Outdated Show resolved Hide resolved
@sunweaver
Copy link
Contributor

Please also provide a grub.cfg that can be used with the GFX mode proposed here. I guess we should provide it in addition to the grub.cfg used by default in FAI.

@sunweaver
Copy link
Contributor

@dzatoah Also, this branch does not merge.

@dzatoah
Copy link
Author

dzatoah commented Jul 3, 2024

@dzatoah Also, this branch does not merge.

Please read the description

This patch is based on commit 2f2e63b (tag: 6.0.1). The current Debian version.

@sunweaver
Copy link
Contributor

@dzatoah Also, this branch does not merge.

Please read the description

This patch is based on commit 2f2e63b (tag: 6.0.1). The current Debian version.

please rebase (of course) against latest version here.

@dzatoah dzatoah force-pushed the mr/add-background-image-and-font-to-grub2-memdisk branch from bbe730f to 4d7c5af Compare July 9, 2024 10:25
@dzatoah
Copy link
Author

dzatoah commented Jul 9, 2024

Please have another look @sunweaver

@sunweaver
Copy link
Contributor

@dzatoah lgtm now, @Mrfai can you take a look? For Debian Edu Router we worked on fai-cd supporting desktop-base background images. The background image must be installed to the NFSROOT (correct, @dzatoah?).

@dzatoah
Copy link
Author

dzatoah commented Jul 9, 2024

The background image must be installed to the NFSROOT

@sunweaver

No, the image must be present on the build host, currently.
In our case: mns+-router-artwork provides /usr/share/images/desktop-base/desktop-grub.png via update-alternatives (installed on odin directly). The same with the font file, except it doesn't use update-alternatives.

@dzatoah dzatoah force-pushed the mr/add-background-image-and-font-to-grub2-memdisk branch from 8facb70 to c9e9cc9 Compare September 5, 2024 15:37
@dzatoah
Copy link
Author

dzatoah commented Sep 5, 2024

So I've changed it that you'll need to install fonts-dejavu-core and desktop-base in the NFSROOT, so that the background image and font file can be loaded via Grub2.

…nsMono font to FAI-CD.

Background image: $NFSROOT/usr/share/images/desktop-base/desktop-grub.png
  - Can be changed via: `chroot $NFSROOT update-alternatives --config desktop-grub`
Font file: $NFSROOT/usr/share/fonts/truetype/dejavu/DejaVuSansMono.ttf

Needs 'grub2-common' *on host*, and 'fonts-dejavu-core' and 'desktop-base' packages *inside of NFSROOT*.
@dzatoah dzatoah force-pushed the mr/add-background-image-and-font-to-grub2-memdisk branch from c9e9cc9 to 15f6018 Compare September 5, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants