On Fri, Mar 15, 2019 at 03:01:41PM +0000, Stuart Henderson wrote:
> Contrary to the advice in faq 7, it is possible to load a new font if
> you have console on a drm device, but there are restrictions on font
> size that must be followed.
> 
> This builds raw variants of some of the Terminus fonts and adds a readme
> showing how to use them. I've updated to a newer version of Terminus
> while there (I could split into two commits if preferred but it's simple
> enough that it doesn't seem necessary).
> 
> Thanks to fcambus@ for clues on this!

This looks good to me, a few nitpicks inline below.

With those addressed, OK fcambus@.

> Index: pkg/README
> ===================================================================
> RCS file: pkg/README
> diff -N pkg/README
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ pkg/README        15 Mar 2019 14:59:57 -0000
> @@ -0,0 +1,28 @@
> +$OpenBSD: README.template,v 1.6 2018/09/04 13:40:26 ajacoutot Exp $
> +
> ++-------------------------------------------------------------------------------
> +| Running ${PKGSTEM} on OpenBSD
> ++-------------------------------------------------------------------------------
> +
> +Use with wscons(4)
> +==================
> +Raw versions of some Terminus fonts are included; these can be loaded with
> +wsfontload(8) for use in OpenBSD's wscons(4) text-mode console.
> +
> +For consoles on a drm(4) device, a font may only be loaded if it matches
> +the size of the current console font. For displays with a width above
> +1920 and belove 3840 pixels, rasops(9) will select a 16x32 font which
            ^^^^^^

You have a typo here: s/belove/below.

> +can be replaced with one of the "ter-132*" fonts. (Terminus does not
> +have a 32x64 font to match the size used by default on wider displays).
> +
> +This can be loaded and configured like so:
> +
> +$ wsfontload -N ter-132n -w 16 -h 32 /usr/local/share/misc/pcvtfonts/ter-132n
                                        ^^^^^^^^^^

portcheck complains about an hardcoded path here :)

> +$ wsconsctl display.font=ter-132n
> +
> +To return to the default console font:
> +
> +$ wsconsctl display.font="Spleen 16x32"

Both wsfontload and wsconsctl must be run as root, so maybe we should
change $ to # for the prompts?

Reply via email to