Received: from listserv.local (listserv [172.16.8.22]) by nld3-dev1.alpinelinux.org (Postfix) with ESMTP id CF120781089; Tue, 15 Feb 2022 09:14:31 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit To: Drew DeVault From: Marian Buschsieweke Cc: ~alpine/aports@lists.alpinelinux.org In-Reply-To: <164491328032.985.8823973081218967383.gitlab.29582.eff7f82b87fb8be5a21c1f621739fb5016d471e2@listserv.local> Subject: Re: [PATCH v2] testing/wlrobs: enable more arches, dmabuf Date: Tue, 15 Feb 2022 09:14:31 -0000 Message-ID: <164491647181.985.14408121974989573664.gitlab.29582.eff7f82b87fb8be5a21c1f621739fb5016d471e2@listserv.local> 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