[PATCH] add support for GRUB_GFXMODE variable

[PATCH] add support for GRUB_GFXMODE variable

[Felix Zielcke]

[-- Attachment #1: Type: text/plain, Size: 299 bytes --]

Here's a patch to add a GRUB_GFXMODE variable for update-grub
to set the gfxmode. 


2008-07-31  Felix Zielcke  <fzielcke@z-51.de>
        
	* util/update-grub.in (GRUB_GFXMODE): Export variable.
	* util/grub.d/00_header.in: Allow the administrator to change default 
	gfxmode via ${GRUB_GFXMODE}.


[-- Attachment #2: gfxmode.diff --]
[-- Type: text/x-patch, Size: 1472 bytes --]

Index: util/grub.d/00_header.in
===================================================================
--- util/grub.d/00_header.in	(Revision 1754)
+++ util/grub.d/00_header.in	(Arbeitskopie)
@@ -31,6 +31,7 @@
 
 if [ "x${GRUB_DEFAULT}" = "x" ] ; then GRUB_DEFAULT=0 ; fi
 if [ "x${GRUB_TIMEOUT}" = "x" ] ; then GRUB_TIMEOUT=5 ; fi
+if [ "x${GRUB_GFXMODE}" = "x" ] ; then GRUB_GFXMODE=640x480 ; fi
 
 cat << EOF
 set default=${GRUB_DEFAULT}
@@ -43,7 +44,7 @@
     prepare_grub_to_access_device `${grub_probe} --target=device ${GRUB_FONT_PATH}`
     cat << EOF
 if font `make_system_path_relative_to_its_root ${GRUB_FONT_PATH}` ; then
-  set gfxmode=640x480
+  set gfxmode=${GRUB_GFXMODE}
   insmod gfxterm
   insmod vbe
   terminal gfxterm
Index: util/update-grub.in
===================================================================
--- util/update-grub.in	(Revision 1754)
+++ util/update-grub.in	(Arbeitskopie)
@@ -164,7 +164,7 @@
 export GRUB_DEVICE GRUB_DEVICE_UUID GRUB_DEVICE_BOOT GRUB_DEVICE_BOOT_UUID GRUB_FS GRUB_FONT_PATH GRUB_PRELOAD_MODULES
 
 # These are optional, user-defined variables.
-export GRUB_DEFAULT GRUB_TIMEOUT GRUB_DISTRIBUTOR GRUB_CMDLINE_LINUX GRUB_CMDLINE_LINUX_DEFAULT GRUB_TERMINAL GRUB_SERIAL_COMMAND GRUB_DISABLE_LINUX_UUID
+export GRUB_DEFAULT GRUB_TIMEOUT GRUB_DISTRIBUTOR GRUB_CMDLINE_LINUX GRUB_CMDLINE_LINUX_DEFAULT GRUB_TERMINAL GRUB_SERIAL_COMMAND GRUB_DISABLE_LINUX_UUID GRUB_GFXMODE
 
 rm -f ${grub_cfg}.new
 exec > ${grub_cfg}.new

Re: [PATCH] add support for GRUB_GFXMODE variable

[Felix Zielcke]

Hello,

seems like nobody responded to this one :)

Am Donnerstag, den 31.07.2008, 17:35 +0200 schrieb Felix Zielcke:
> Here's a patch to add a GRUB_GFXMODE variable for update-grub
> to set the gfxmode. 
> 
> 
> 2008-07-31  Felix Zielcke  <fzielcke@z-51.de>
>         
> 	* util/update-grub.in (GRUB_GFXMODE): Export variable.
> 	* util/grub.d/00_header.in: Allow the administrator to change default 
> 	gfxmode via ${GRUB_GFXMODE}.

Re: [PATCH] add support for GRUB_GFXMODE variable

[Marco Gerards]

Hi,

Felix Zielcke <fzielcke@z-51.de> writes:

> seems like nobody responded to this one :)
>
> Am Donnerstag, den 31.07.2008, 17:35 +0200 schrieb Felix Zielcke:
>> Here's a patch to add a GRUB_GFXMODE variable for update-grub
>> to set the gfxmode. 
>> 
>> 
>> 2008-07-31  Felix Zielcke  <fzielcke@z-51.de>
>>         
>> 	* util/update-grub.in (GRUB_GFXMODE): Export variable.
>> 	* util/grub.d/00_header.in: Allow the administrator to change default 
>> 	gfxmode via ${GRUB_GFXMODE}.

This is Roberts domain ;-)

--
Marco

Re: [PATCH] add support for GRUB_GFXMODE variable

[Felix Zielcke]

Am Mittwoch, den 13.08.2008, 23:26 +0200 schrieb Marco Gerards:
> >> 2008-07-31  Felix Zielcke  <fzielcke@z-51.de>
> >>         
> >> 	* util/update-grub.in (GRUB_GFXMODE): Export variable.
> >> 	* util/grub.d/00_header.in: Allow the administrator to change default 
> >> 	gfxmode via ${GRUB_GFXMODE}.
> 
> This is Roberts domain ;-)
> 

Hello Robert :)
*ping*
I still haven't hear anything from you,
does this look okay or not?

Re: [PATCH] add support for GRUB_GFXMODE variable

[Robert Millan]

On Fri, Aug 22, 2008 at 12:46:16PM +0200, Felix Zielcke wrote:
> Am Mittwoch, den 13.08.2008, 23:26 +0200 schrieb Marco Gerards:
> > >> 2008-07-31  Felix Zielcke  <fzielcke@z-51.de>
> > >>         
> > >> 	* util/update-grub.in (GRUB_GFXMODE): Export variable.
> > >> 	* util/grub.d/00_header.in: Allow the administrator to change default 
> > >> 	gfxmode via ${GRUB_GFXMODE}.
> > 
> > This is Roberts domain ;-)
> > 
> 
> Hello Robert :)
> *ping*
> I still haven't hear anything from you,
> does this look okay or not?

Seems fine.

-- 
Robert Millan

  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all."

Re: [PATCH] add support for GRUB_GFXMODE variable

[Felix Zielcke]

Committed.

Am Freitag, den 22.08.2008, 14:53 +0200 schrieb Robert Millan:
> On Fri, Aug 22, 2008 at 12:46:16PM +0200, Felix Zielcke wrote:
> > Am Mittwoch, den 13.08.2008, 23:26 +0200 schrieb Marco Gerards:
> > > >> 2008-07-31  Felix Zielcke  <fzielcke@z-51.de>
> > > >>         
> > > >> 	* util/update-grub.in (GRUB_GFXMODE): Export variable.
> > > >> 	* util/grub.d/00_header.in: Allow the administrator to change default 
> > > >> 	gfxmode via ${GRUB_GFXMODE}.
> > > 
> > > This is Roberts domain ;-)
> > > 
> > 
> > Hello Robert :)
> > *ping*
> > I still haven't hear anything from you,
> > does this look okay or not?
> 
> Seems fine.
>