/boot/grub or ... desktop-base required in 05_debian_theme path?

Bug #458079 reported by drs305
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
grub2 (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

Binary package hint: grub2

When using a folder other than the defaults, the splash image location is not imported into grub.cfg unless either the "/boot/grub" OR "/usr/share/images/desktop-base" folder is also present in /etc/grub.d/05_debian_theme:

Line 16 approximately:
  for i in {/boot/grub,/usr/share/images/desktop-base}/moreblue-orbit-grub.{png,tga} ; do

This will not import my images into grub.cfg unless I add one of the two folders above. They don't have to be in the assigned order, but at least one of them must be present in the line for the script to import an image from any other location:

WORKS
  for i in {/boot/grub,/usr/share/images/desktop-base,/usr/share/images/grub}/earth.640x480.{png,tga} ; do
  for i in {/boot/grub,/usr/share/images/grub}/earth.640x480.{png,tga} ; do
  for i in {/usr/share/images/desktop-base,/usr/share/images/grub}/earth.640x480.{png,tga} ; do
  for i in {/usr/share/images/grub,/boot/grub}/earth.640x480.{png,tga} ; do

WON'T WORK
  for i in {/usr/share/images/grub}/earth.640x480.{png,tga} ; do

Perhaps the script is written that way for a purpose but I see no reason to have such a constraint on the location of the splash images.

Thanks for a great GRUB 2 product!

drs305

ProblemType: Bug
Architecture: amd64
Date: Thu Oct 22 06:56:52 2009
DistroRelease: Ubuntu 9.10
NonfreeKernelModules: nvidia
Package: grub-pc 1.97~beta4-1ubuntu2
ProcEnviron:
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-14.48-generic
SourcePackage: grub2
Uname: Linux 2.6.31-14-generic x86_64
XsessionErrors:
 (gnome-settings-daemon:2237): GLib-CRITICAL **: g_propagate_error: assertion `src != NULL' failed
 (gnome-settings-daemon:2237): GLib-CRITICAL **: g_propagate_error: assertion `src != NULL' failed
 (polkit-gnome-authentication-agent-1:2290): GLib-CRITICAL **: g_once_init_leave: assertion `initialization_value != 0' failed
 (nautilus:2269): Eel-CRITICAL **: eel_preferences_get_boolean: assertion `preferences_is_initialized ()' failed

Revision history for this message
drs305 (drs305) wrote :
Revision history for this message
Felix Zielcke (fzielcke) wrote : Re: [Bug 458079] [NEW] /boot/grub or ... desktop-base required in 05_debian_theme path?

Am Donnerstag, den 22.10.2009, 11:16 +0000 schrieb drs305:
> Public bug reported:
>
> Binary package hint: grub2
>
> When using a folder other than the defaults, the splash image location
> is not imported into grub.cfg unless either the "/boot/grub" OR
> "/usr/share/images/desktop-base" folder is also present in
> /etc/grub.d/05_debian_theme:
>
> Line 16 approximately:
> for i in {/boot/grub,/usr/share/images/desktop-base}/moreblue-orbit-grub.{png,tga} ; do
>
> This will not import my images into grub.cfg unless I add one of the two
> folders above. They don't have to be in the assigned order, but at least
> one of them must be present in the line for the script to import an
> image from any other location:
>
> WORKS
> for i in {/boot/grub,/usr/share/images/desktop-base,/usr/share/images/grub}/earth.640x480.{png,tga} ; do
> for i in {/boot/grub,/usr/share/images/grub}/earth.640x480.{png,tga} ; do
> for i in {/usr/share/images/desktop-base,/usr/share/images/grub}/earth.640x480.{png,tga} ; do
> for i in {/usr/share/images/grub,/boot/grub}/earth.640x480.{png,tga} ; do
>
> WON'T WORK
> for i in {/usr/share/images/grub}/earth.640x480.{png,tga} ; do
>
> Perhaps the script is written that way for a purpose but I see no reason
> to have such a constraint on the location of the splash images.

If you only use one directory, you don't even need the { }
so this definately should work:

  for i in /usr/share/images/grub/earth.640x480.{png,tga} ; do

If you modify that script and you know if your image is png or tga then
you even can do the same with the {png,tga}

According to bash's man page this seems to be with intent.

>
> Thanks for a great GRUB 2 product!
>

You're welcome :)

--
Felix Zielcke
Proud Debian Maintainer and GNU GRUB developer

Felix Zielcke (fzielcke)
Changed in grub2 (Ubuntu):
status: New → Invalid
Revision history for this message
drs305 (drs305) wrote :

Yes, that works. Mark as invalid.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.