Bug#561593: background image config/handling (plus password handling)

[Alexander Kurtz]

forcemerge 605705 561593
thanks

Am Freitag, den 18.12.2009, 15:07 +0100 schrieb debian at x.ray.net:
> Package: grub-pc
> Version: 1.98~20091210-1
> Tags: patch
> 
> hi!
> 
> i just received the grub-pc upgrade, to find out that /etc/grub/05_debian_theme now sources /usr/share/desktop-base/grub_background.sh, obviously expecting the grub background image ($WALLPAPER) and grub menu colors ($COLOR_NORMAL and $COLOR_HIGHLIGHT) to be set in there. but grub_background.sh doesn't exist. neither as a file, nor as something i was able to find documented somewhere in the package.
The package desktop-base started providing the file in version 6.0.0:

	desktop-base (6.0.0) unstable; urgency=low
	[...]
	   * grub:
	     - add SpaceFun grub theme.
	     - add a grub_background.sh which will override settings in grub and
	       correctly set the default theme.
	[...]
	 -- Yves-Alexis Perez <corsac at debian.org>  Mon, 15 Nov 2010 19:58:48 +0100

> also, for configuration issues, i guess variables like these are (and should be) usually set in /etc/default/grub (like the readme says. and while we're at it, all variables in there have the prefix GRUB_, which is generally a very good idea i guess)?
No, these variables are not part of GRUB's configuration but belong to
desktop-base. GRUB merely uses them to find out what should be the
default background image.

> i don't understand the rationale of /usr/share/desktop-base/grub_background.sh, but even if it's a good idea as some sort of hook, then even if WALLPAPER was set in /etc/default/grub, it would be overwritten in any case. to prevent this, i guess the best solution would be to remove the hardcoded default values from that file, and simply set them as default values in /etc/default/grub resp. /usr/share/grub/default/grub.
I think this part of your bug report is a duplicate of #605705[1]. If
you want to set a custom wallpaper nowadays, you can either simply copy
it to /boot/grub/ or specify it in /etc/default/grub for example like
this:
	GRUB_BACKGROUND="/path/to/myfile.png"

> while i was at it i also added the following variables to be able to configure grub-menu-passwords:
> 
> GRUB_SUPERUSERS="user1 user2 ..."
> GRUB_PASSWORD_user1="cryptstring"
> ...
> 
> (can be configured in /etc/default/grub, changed 00_header and grub-mkconfig to handle them)
Please file a separate bug for this issue if you still think it's
relevant.

Best regards

Alexander Kurtz
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/pkg-grub-devel/attachments/20110102/a08bbe45/attachment.pgp>