Bug 226605

Summary: graphics/glfw: add WAYLAND option
Product: Ports & Packages Reporter: Greg V <greg>
Component: Individual Port(s)Assignee: Tobias Kortkamp <tobik>
Status: Closed FIXED    
Severity: Affects Only Me CC: koalative, tobik
Priority: --- Flags: bugzilla: maintainer-feedback? (koalative)
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on: 232323    
Bug Blocks:    
Attachments:
Description Flags
glfw-wayland.patch none

Description Greg V 2018-03-14 15:28:49 UTC
Created attachment 191496 [details]
glfw-wayland.patch

GLFW supports Wayland, this patch adds an option to choose the Wayland backend.

Unfortunately, unlike most toolkits, backend choice is not dynamic in GLFW. So Linux distros ship separate packages, e.g.:

https://www.archlinux.org/packages/community/x86_64/glfw-wayland/
https://www.archlinux.org/packages/community/x86_64/glfw-x11/

and users can install one.

So just adding a non-default option here. Eventually should be a flavor I guess.
Comment 1 commit-hook freebsd_committer 2018-03-28 18:48:44 UTC
A commit references this bug:

Author: tobik
Date: Wed Mar 28 18:47:52 UTC 2018
New revision: 465837
URL: https://svnweb.freebsd.org/changeset/ports/465837

Log:
  graphics/glfw: Add WAYLAND option

  - Reset maintainer after the third consecutive timeout

  PR:		226605
  Submitted by:	greg@unrelenting.technology (based on)
  Approved by:	koalative@gmail.com (maintainer timeout, 2 weeks)

Changes:
  head/graphics/glfw/Makefile
  head/graphics/glfw/files/patch-src_wl__init.c
Comment 2 Tobias Kortkamp freebsd_committer 2018-03-28 18:59:47 UTC
(In reply to Greg V from comment #0)
>  Unfortunately, unlike most toolkits, backend choice is not dynamic in GLFW. So Linux distros ship separate packages, e.g.:

This sounds like something upstream should fix rather than letting all
downstream packagers deal with it in their own way.  Anyway let's
worry about that later and land it now rather than letting this bug
sit for many weeks or months...

Committed with many changes since the initial patch was missing a lot
of dependencies and didn't build in Poudriere as is.
Comment 3 Greg V 2018-03-28 20:31:56 UTC
(In reply to Tobias Kortkamp from comment #2)
Yeah, of course they're planning to implement multi backend, they're currently working on loading the libraries dynamically which is a prerequisite.