From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34964) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eAJmg-0004k0-R4 for guix-patches@gnu.org; Thu, 02 Nov 2017 14:00:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eAJmd-0008Ss-1C for guix-patches@gnu.org; Thu, 02 Nov 2017 14:00:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:39535) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eAJmc-0008Sn-QZ for guix-patches@gnu.org; Thu, 02 Nov 2017 14:00:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1eAJmc-0003YE-JX for guix-patches@gnu.org; Thu, 02 Nov 2017 14:00:02 -0400 Subject: [bug#29115] [PATCH] gnu: mpv: enable Wayland support. Resent-Message-ID: MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="=_01d0c9a181f899eac747a10a719fe19b" Date: Thu, 02 Nov 2017 18:59:26 +0100 From: Rutger Helling In-Reply-To: <1b19ea3f1c54bff4cba9f320786867f7@mykolab.com> References: <9137d310167b962d1ce3d20339be4fba@mykolab.com> <20171102172442.GA5245@jasmine.lan> <1b19ea3f1c54bff4cba9f320786867f7@mykolab.com> Message-ID: <3c9a4be90b5030067a77468935cdac93@mykolab.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: 29115@debbugs.gnu.org --=_01d0c9a181f899eac747a10a719fe19b Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII On 2017-11-02 18:52, Rutger Helling wrote: > Hey Leo, > > you're right, it works fine without wayland-protocols. Here's an updated patch without it. > > On 2017-11-02 18:24, Leo Famulari wrote: > On Thu, Nov 02, 2017 at 10:23:15AM +0100, Rutger Helling wrote: Hey Guix, > > this patch enables Wayland support for mpv. > From b18aa7cb448116c689306c07fccec32afc84dc4b Mon Sep 17 00:00:00 2001 > From: Rutger Helling > Date: Thu, 2 Nov 2017 10:19:45 +0100 > Subject: [PATCH] gnu: mpv: enable Wayland support. > > * gnu/packages/video.scm (mpv)[inputs]: Add wayland, wayland-protocols and libxkbcommon. > Thanks! > > ("pulseaudio" ,pulseaudio) > ("rsound" ,rsound) > ("waf" ,python-waf) > + ("wayland" ,wayland) > + ("wayland-protocols" ,wayland-protocols) > + ("libxkbcommon", libxkbcommon) > I tried `guix gc --reference $(./pre-inst-env guix build mpv)`, and it > doesn't keep a reference to wayland-protocols. Also, without > wayland-protocols, it does still build for Wayland, according to the > configure phase. > > Do you think we can omit wayland-protocols from the set of inputs? --=_01d0c9a181f899eac747a10a719fe19b Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=UTF-8

On 2017-11-02 18:52, Rutger Helling wrote:

Hey Leo,

you're right, it works fine without wayland-protocols. Here's an updated= patch without it.

On 2017-11-02 18:24, Leo Famulari wrote:

On Thu, Nov 02, 2017 at 10:23:15AM +0100, Rutger Helling wrote:
Hey Guix,

this patch enables Wayland support for mpv= =2E

From b18aa7cb448116c689306c07fccec32afc84dc4b Mon Sep 17 00:00:00 2= 001
From: Rutger Helling <= rhelling@mykolab.com>
Date: Thu, 2 Nov 2017 10:19:45 +0100
Subject: [PATCH] gnu: mpv: enable Wayland support.

* gnu/pa= ckages/video.scm (mpv)[inputs]: Add wayland, wayland-protocols and libxkbco= mmon.

Thanks!

        ("pulseaudio" ,puls= eaudio)
        ("rsound" ,rs= ound)
        ("waf" ,python-= waf)
+       ("wayland" ,wayland)
= +       ("wayland-protocols" ,wayland-protoc= ols)
+       ("libxkbcommon", libxkbcom= mon)

I tried `guix gc --reference $(./pre-inst-env guix build mpv)`, and = it
doesn't keep a reference to wayland-protocols. Also, without
= wayland-protocols, it does still build for Wayland, according to the
= configure phase.

Do you think we can omit wayland-protocols f= rom the set of inputs?



--=_01d0c9a181f899eac747a10a719fe19b--