unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Rutger Helling <rhelling@mykolab.com>
To: 33935@debbugs.gnu.org
Subject: [bug#33935] PATCH] gnu: sdl2: Fix Wayland support.
Date: Mon, 31 Dec 2018 11:17:00 +0100	[thread overview]
Message-ID: <20181231101700.38c9867e@mykolab.com> (raw)


[-- Attachment #1.1: Type: text/plain, Size: 543 bytes --]

Hi Guix,

I've discovered that our SDL2 unfortunately does not correctly support
Wayland at the moment. I found while running Xonotic I could still kill
the window with xkill, even with the SDL_VIDEODRIVER=wayland variable
set. I unset the DISPLAY variable and tried running it that way and
found that it gave an error and refused to start.

This patch should fix the support. I can now run 'DISPLAY=""
SDL_VIDEODRIVER=wayland xonotic' and it'll run. The window cannot be
killed by xkill, which means it's running Wayland natively.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-sdl2-Fix-Wayland-support.patch --]
[-- Type: text/x-patch, Size: 1398 bytes --]

From 8861047922636a8a7cb10ca4a753c16c896b1ff9 Mon Sep 17 00:00:00 2001
From: Rutger Helling <rhelling@mykolab.com>
Date: Mon, 31 Dec 2018 11:07:05 +0100
Subject: [PATCH] gnu: sdl2: Fix Wayland support.

* gnu/packages/sdl.scm (sdl2)[propagated-inputs]: Add "wayland-protocols".
[inputs]: Remove "wayland-protocols".
---
 gnu/packages/sdl.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/sdl.scm b/gnu/packages/sdl.scm
index 619892572..e272c0f8d 100644
--- a/gnu/packages/sdl.scm
+++ b/gnu/packages/sdl.scm
@@ -116,6 +116,9 @@ joystick, and graphics hardware.")
        ((#:configure-flags flags)
         `(append '("--disable-wayland-shared")
                  ,flags))))
+    (propagated-inputs
+     (append `(("wayland-protocols" ,wayland-protocols))
+             (package-propagated-inputs sdl)))
     (inputs
      ;; SDL2 needs to be built with ibus support otherwise some systems
      ;; experience a bug where input events are doubled.
@@ -126,8 +129,7 @@ joystick, and graphics hardware.")
                ("glib" ,glib)
                ("ibus" ,ibus)
                ("libxkbcommon" ,libxkbcommon)
-               ("wayland" ,wayland)
-               ("wayland-protocols" ,wayland-protocols))
+               ("wayland" ,wayland))
              (package-inputs sdl)))
     (license bsd-3)))
 
-- 
2.20.1


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

             reply	other threads:[~2018-12-31 10:18 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-31 10:17 Rutger Helling [this message]
2018-12-31 10:26 ` bug#33935: PATCH] gnu: sdl2: Fix Wayland support Rutger Helling

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181231101700.38c9867e@mykolab.com \
    --to=rhelling@mykolab.com \
    --cc=33935@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).