Re: git: 4a9d156ccc - main - Add a section on using wayland with FreeBSD: - Note the basic setup; - Cover basic configuration of wayfire, sway, and hikari; - Add information about using Xwayland; - Discuss useful utilities such as ly and vnc

From: Daniel Ebdrup Jensen <debdrup_at_FreeBSD.org>
Date: Sun, 20 Feb 2022 12:46:51 UTC
On Sun, Feb 20, 2022 at 01:53:33AM +0000, Tom Rhodes wrote:
>The branch main has been updated by trhodes:
>
>URL: https://cgit.FreeBSD.org/doc/commit/?id=4a9d156ccc8d24d4226d793dbdcc520276c43084
>
>commit 4a9d156ccc8d24d4226d793dbdcc520276c43084
>Author:     Tom Rhodes <trhodes@FreeBSD.org>
>AuthorDate: 2022-02-20 01:47:07 +0000
>Commit:     Tom Rhodes <trhodes@FreeBSD.org>
>CommitDate: 2022-02-20 01:47:07 +0000
>
>    Add a section on using wayland with FreeBSD:
>     - Note the basic setup;
>     - Cover basic configuration of wayfire, sway, and hikari;
>     - Add information about using Xwayland;
>     - Discuss useful utilities such as ly and vnc
>
>    Reviewed by:            several
>    Requested by:           several
>    Differential revision:  D32463
>---
[SNIP]

Hi Tom,

      First off, thanks for making this addition, it's very useful!

      I've only noticed one little thing:
      The note about nvidia drivers for Wayland needs an extra set of =
      in order to render properly on [1].

      I'm sorry I didn't notice it in the review phase, but may I
      recommend you add the docs group in the future? I don't receive
      mail on the Doc Committers group.

      Also, it appears you included the wrong differential, the right one
      is [2].

Yours,
Daniel Ebdrup Jensen

1: https://docs.freebsd.org/en/books/handbook/x11/#x-wayland
2: https://reviews.freebsd.org/D32436