dmabuf is now supported without an upstream obs patch, and my earlier
obs changes fix obs on all architectures.
---
v2: rebase

 testing/wlrobs/APKBUILD | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/testing/wlrobs/APKBUILD b/testing/wlrobs/APKBUILD
index 7431e62265..3edd554cb4 100644
--- a/testing/wlrobs/APKBUILD
+++ b/testing/wlrobs/APKBUILD
@@ -4,9 +4,9 @@ pkgver=1.0
 pkgrel=1
 pkgdesc="obs-studio plugin that allows you to screen capture on wlroots"
 url="https://hg.sr.ht/~scoopta/wlrobs"
-arch="x86 x86_64" # obs-stduio not provided on other platforms
+arch="all"
 license="GPL-3.0-only"
-makedepends="meson ninja obs-studio-dev wayland-dev"
+makedepends="meson ninja obs-studio-dev wayland-dev libdrm-dev mesa-dev"
 source="https://dev.alpinelinux.org/archive/wlrobs/wlrobs-$pkgver.tar.gz "
 builddir="$srcdir/$pkgname-v$pkgver"
 options="!check" # No tests provided
-- 
2.34.1

Fails on s390x and armhf due to obs-studio(-dev) not packaged. lgtm otherwise

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_206366

I have a separate patch which should enable obs-studio on these arches.

Drew DeVault <sir@cmpwn.com> replied via email:

```
I have a separate patch which should enable obs-studio on these arches.

```

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_206368

Depends on !29571.

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_206524

Sorry to bother you @mailinglist-bot,

but we've detected that this merge request hasn't seen any recent activity. If you need help or want to discuss your approach with developers you can ping `@maribu`. You can also ask on IRC on `#alpine-devel` on irc.oftc.net. If no further activity occurs in this MR, Alpine developers may close it in the future.

Thanks for your contribution.

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_215443

On Sat, 15 Jan 2022 10:26:17 +0000, Drew DeVault wrote:
>  license="GPL-3.0-only"
> -makedepends="meson ninja obs-studio-dev wayland-dev"
> +makedepends="meson ninja obs-studio-dev wayland-dev libdrm-dev mesa-dev"

How is this related to enabling wlrobs on more arches? libdrm and mesa are needed only on non-x86_64? o.O

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_215458

On Sat, 15 Jan 2022 10:26:17 +0000, Drew DeVault wrote:
>  license="GPL-3.0-only"
> -makedepends="meson ninja obs-studio-dev wayland-dev"
> +makedepends="meson ninja obs-studio-dev wayland-dev libdrm-dev mesa-dev"

It's not. This commit does two things: enables more arches, and enables the dmabuf backend (which adds the dependencies).

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_215491

On Sat, 15 Jan 2022 10:26:17 +0000, Drew DeVault wrote:
>  license="GPL-3.0-only"
> -makedepends="meson ninja obs-studio-dev wayland-dev"
> +makedepends="meson ninja obs-studio-dev wayland-dev libdrm-dev mesa-dev"

Maybe it makes sense to split the two changes into one commit each to make it more obvious that adding `libdrm-dev` and `mesa-dev` is needed to also enable the dmabuf backend and not related to enabling it for other arches? (IMO they could still both be part of the same MR.)

But then again, the MR discussion now also serves as documentation for this.

In any case: I am still strongly in favor of getting this in :)

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_215513

On Sat, 15 Jan 2022 10:26:17 +0000, Drew DeVault wrote:
>  license="GPL-3.0-only"
> -makedepends="meson ninja obs-studio-dev wayland-dev"
> +makedepends="meson ninja obs-studio-dev wayland-dev libdrm-dev mesa-dev"

Superseded by this: https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/30934

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/29582#note_215515