All of lore.kernel.org
 help / color / mirror / Atom feed
* Bug found and fixed. Patch is attached to the letter.
@ 2012-12-04 16:51 Vladimir Testov
  2012-12-10 10:49 ` Vladimir 'φ-coder/phcoder' Serbinenko
  0 siblings, 1 reply; 2+ messages in thread
From: Vladimir Testov @ 2012-12-04 16:51 UTC (permalink / raw)
  To: grub-devel

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

Hello!

I was playing with grub-mkfont utility, trying to make console font with 
selected size (10 actually).
And there were too much space between strings. (Also, I had to keep utf-8 
symbols that are not ascii)
So I faced that -c\--asce option of grub-mkfont does not work at all. This 
patch fixes the problem.
Now `grub-mkconfig -s 10 -c 9 -d 2 -a -o ... ...` makes the font I need.

Hope it will be useful.

And hope it will be added to upstream (as well as my previous patch).

-- 
With best regards,
_______________________________
Vladimir Testov, ROSA Laboratory.
www.rosalab.ru

[-- Attachment #2: grub2-mkfont-fix.patch --]
[-- Type: text/x-patch, Size: 407 bytes --]

diff -Naur grub-2.00/util/grub-mkfont.c grub-new/util/grub-mkfont.c
--- grub-2.00/util/grub-mkfont.c	2012-03-10 16:17:57.000000000 +0400
+++ grub-new/util/grub-mkfont.c	2012-12-04 18:00:35.463252107 +0400
@@ -1101,7 +1101,7 @@
       arguments->font_info.desc = strtoul (arg, NULL, 0);
       break;
 
-    case 'e':
+    case 'c':
       arguments->font_info.asce = strtoul (arg, NULL, 0);
       break;
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Bug found and fixed. Patch is attached to the letter.
  2012-12-04 16:51 Bug found and fixed. Patch is attached to the letter Vladimir Testov
@ 2012-12-10 10:49 ` Vladimir 'φ-coder/phcoder' Serbinenko
  0 siblings, 0 replies; 2+ messages in thread
From: Vladimir 'φ-coder/phcoder' Serbinenko @ 2012-12-10 10:49 UTC (permalink / raw)
  To: grub-devel

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

On 04.12.2012 17:51, Vladimir Testov wrote:

> Hello!
> 
> I was playing with grub-mkfont utility, trying to make console font with 
> selected size (10 actually).
> And there were too much space between strings. (Also, I had to keep utf-8 
> symbols that are not ascii)
> So I faced that -c\--asce option of grub-mkfont does not work at all. This 
> patch fixes the problem.
> Now `grub-mkconfig -s 10 -c 9 -d 2 -a -o ... ...` makes the font I need.
> 

Committed, thanks.

> Hope it will be useful.
> 
> And hope it will be added to upstream (as well as my previous patch).
> 
> 
> 
> 
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel



-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-12-10 10:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-04 16:51 Bug found and fixed. Patch is attached to the letter Vladimir Testov
2012-12-10 10:49 ` Vladimir 'φ-coder/phcoder' Serbinenko

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.