[arch-dev-public] [PATCH] fixed detection of CONSOLEFONT and enabled all *gz fonts

[Thomas Bächler]

Am 11.02.2010 22:08, schrieb Ionut Biru:
> Hi,
> First the current implementation doesn't work since CONSOLEFONT is
> always zero. Second i don't know anything about fonts and why does the
> first commiter only enabled pfsu.gz fonts but i tested with a psf.gz
> font and is working fine and i decided to add any *.gz font.
> 
> Comments?

If you had posted your patch inline (as git intended it) I could comment
on it directly, but now thunderbird won't quote it.

First, you introduced a broken if [ ... ] statement in the first hunk
that will cause an error (missing space).

Second, the original commiter intended the console font to be set in
mkinitcpio.conf, so he can remove the consolefont setting in rc.conf.
Apparently his system boots faster this way.

Third, using a * in finding the file may return several files.
Furthermore, the * won't be expanded because it is quoted.

Did you test this patch at all?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 262 bytes
Desc: OpenPGP digital signature
URL: <http://mailman.archlinux.org/pipermail/arch-dev-public/attachments/20100212/a9dcac1d/attachment.bin>