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

Keep /usr/share/pixmaps/fedora-logo-sprite.svg in fedora boot.iso #1340

Closed
KKoukiou opened this issue Aug 22, 2023 · 2 comments · Fixed by #1341
Closed

Keep /usr/share/pixmaps/fedora-logo-sprite.svg in fedora boot.iso #1340

KKoukiou opened this issue Aug 22, 2023 · 2 comments · Fixed by #1341

Comments

@KKoukiou
Copy link
Contributor

The /usr/share/pixmaps/fedora-logo-sprite.svg file contains the best suitable logo for Web UI branding.

Looks like the /usr/share/pixmaps/fedora-logo-sprite.svg file from the fedora-logos package is present on Live media, but missing on boot.iso.

See rhinstaller/anaconda#5046

@VladimirSlavik
Copy link
Contributor

Ideally something like this would be possible in runtime-cleanup.tmpl:

- removefrom ${branding.logos} /usr/share/{kde4,pixmaps}/*
+ removefrom ${branding.logos} /usr/share/{kde4,pixmaps}/* --except /usr/share/pixmaps/fedora-logo-sprite.svg

However, the --allbut option does not do that, and --except does not exist.

I guess one could maneuver around this by being more specific, although it would not be pretty:

- removefrom ${branding.logos} /usr/share/{kde4,pixmaps}/*
+ removefrom ${branding.logos} /usr/share/kde4/*
+ removefrom ${branding.logos} /usr/share/pixmaps/bootloader/*
+ removefrom ${branding.logos} /usr/share/pixmaps/*.png

...which should trim it down to...

$ rpm -ql  fedora-logos | grep '/usr/share/pixmaps' | grep -v png | grep -v bootloader
/usr/share/pixmaps/fedora-logo-sprite.svg
/usr/share/pixmaps/fedora_whitelogo.svg

HTH?

@bcl
Copy link
Contributor

bcl commented Aug 31, 2023

I think that's the best we can do, other than leaving all of pixmaps in there (and that looks like it is about 3M so it's worth trying to trim it down).

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 a pull request may close this issue.

3 participants