Re: wayland/sway: build fix and link to the tree

On Tue, Nov 28, 2023 at 03:59:02PM +0100, Sebastien Marie wrote:
> Hi,
>
> The following patch makes wayland/sway to link to the tree (remove
> UNLINKED) and remove the RUN_DEPENDS on wayland/foot (which is still
> UNLINKED).
>
> foot is only need if you set it to be your "preferred terminal". it is a
> runtime configuration.
>
> a meta package (with sway, foot, swaylock, swayidle, …) might be a
> better solution to have a minimal "sway-desktop" working installation.
>
> This way, sway could be installable via package, and ease the testing.
>
> I also added wayland/wayland-protocols as BUILD_DEPENDS due to configure
> error if the package isn't here.
>
> The revision isn't bumped (port not linked), but it might be preferable
> to avoid local installation to plistdb conflict with package (changed
> rdepends). I could add a REVISION=0.
>
> Comments or OK ?
> --
> Sebastien Marie
>
> diff /home/semarie/repos/openbsd/ports
> commit - cd569fb13fcce17ed39e026ce9ee6eb9f4c7b3a6
> path + /home/semarie/repos/openbsd/ports
> blob - 71caf335dc6d6cf044c1bb7679de268326bbd52a
> file + wayland/sway/Makefile
> --- wayland/sway/Makefile
> +++ wayland/sway/Makefile
> @@ -1,5 +1,4 @@
> COMMENT = i3-compatible Wayland compositor
> -UNLINKED = wayland
>
> GH_ACCOUNT = swaywm
> GH_PROJECT = sway
> @@ -23,7 +22,8 @@ WANTLIB += c m
>
> BUILD_DEPENDS = \
> graphics/gdk-pixbuf2 \
> - textproc/scdoc
> + textproc/scdoc \
> + wayland/wayland-protocols
>
> LIB_DEPENDS = devel/json-c \
> devel/pango \
> @@ -34,8 +34,7 @@ LIB_DEPENDS = devel/json-c \
> wayland/wlroots \
> x11/xkbcommon
>
> -RUN_DEPENDS = x11/gnome/themes-extra \
> - wayland/foot
> +RUN_DEPENDS = x11/gnome/themes-extra

Is there a reason for x11/gnome/themes-extra?
Honest question, I know nothing about sway.

--
Antoine