maybe condition to the presence of /usr/bin/tic considering that it does not exist in FreeBSD 13 and it won't be MFCed

Judging by the mailing list, these instructions not being in here are causing problems. I will commit this tonight when I get home.

Thanks @bcr! One thing I've been struggling with is I'd like to tell them but haven't liked anything I've come up with is "hey check the version of the package and make sure it matches the version of the kernel".

I've found two small nits. The rest looks god enough to commit I would say. As mentioned by others, we can always fix other things found in a followup commit.

Committed as e86781da6b600e8096a42a6f44b2b7f97ff08251

I've been hacking on this a while, here's what I've got so far.

One problem that keeps me from wanting to merge this, is that the Xorg FreeType Module has not existed in at least 15 years, but I can't really find any information about that to rewrite the relevant areas.

I think you can commit it, if there's an improvement, we can change it later.
Thanks for all the improvements.

Thanks Jim, these are all great!

A few minor edits. Good job on this section.

Here's my current draft, I am not suggesting this is mergable currently,
maybe parts of it are, but here's just what I've got so far in case
anyone is interested in this issue.

Some language and typo fixes.

Next revision will include increased detail about fonts and configuring touchpads with libinput, however I am really hoping to fix this problem at what I perceive to be the root with upstream manpage.

Fixed xrefs

Additional attempting to clarify more of the instructions. It's still WIP, but appreciate any general feedback.
Thanks @bcr for showing me the testing instructions! Proofreading the rendered output is massively easier!

Here's how you can "make" the doc tree:
https://docs.freebsd.org/en/books/fdp-primer/doc-build/

Found a few straggler typos, sorry for the noise.

Removed duplicated warning after testing and proofreading.

Thanks!

LGTM but I'm not a doc committer.

Polite ping

Capitalize beginning of comment, thanks!

improve comment consistency with upstream, except expanded because xkb is already parsed from the name.

I'm intending to improve consistency with setxkbmap and search results, as XKB is already parsed from the name.

Make comment consistent with setxkbmap.

Thanks for the hint! The bad distfile name really bothered me, but I couldn't figure it out on my own.

2023-04-07 (and newer) results for Vulkan on X11 (both Xwayland and Xorg) are too low like during software rendering. Seems to be limited to --fullscreen option. Likely a drm-kmod regression as mesa-devel-21.2.b.284 no longer works (VK_ERROR_DEVICE_LOST) without either ANV_QUEUE_THREAD_DISABLE=1 or mesa@d2ff2b9e4a22 and still exhibits low performance as new versions.

Abandoning as I lack mental energy to respond to possible comments, and I'm probably no longer interested in FreeBSD.
This has nothing to do with this review, this is just accumulated. Feel free to pickup the revision if you are interested.

Abandoning this, since these changes should be superseded by the update of xserver to 1.20.

I believe this patch has been superseeded by the update of xserver to 1.20.

I don't think that Assistant key should be mapped to KEY_RIGHTMETA for following reasons:

1. It is not really a meta key as it does not affect other key meanings like Shift, Alt, Ctrl and Win keys do.
2. According to https://www.win.tue.nl/~aeb/linux/kbd/scancodes-6.html, scancode of 0x58 prefixed with 0xE0 is emitted by pressing of Fn+F12 keys on Logitech and Microsoft keyboards so the best matching evdev event would be KEY_FN_F12.

What's missing to get this in?

Update comment in Mk/Uses/gl.mk to reflect reality.

Updated based on latest set of comments.

In D16774#362352, @antoine wrote:

bsd.port.pre.mk / bsd.port.post.mk must be kept for the ${COMPILER_TYPE} check

bsd.port.pre.mk / bsd.port.post.mk must be kept for the ${COMPILER_TYPE} check