Bug 246969

Summary: graphics/libdrm: fix build on GCC architectures
Product: Ports & Packages Reporter: Piotr Kubaj <pkubaj>
Component: Individual Port(s)Assignee: freebsd-x11 (Nobody) <x11>
Status: Closed FIXED    
Severity: Affects Only Me CC: manu, x11
Priority: --- Flags: bugzilla: maintainer-feedback? (x11)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch
pkubaj: maintainer-approval? (x11)
patch with upstream MR commit none

Description Piotr Kubaj freebsd_committer freebsd_triage 2020-06-04 08:44:24 UTC
Created attachment 215215 [details]
patch

sys/types.h needs to be included for u_int.

Error:
In file included from ../xf86drmMode.c:46:
/usr/include/sys/sysctl.h:1101: error: expected declaration specifiers or '...' before 'u_int'
../xf86drmMode.c: In function 'drmModeRevokeLease':
../xf86drmMode.c:1578: warning: declaration of 'revoke' shadows a global declaration

Happens when building with GCC.
Comment 1 Emmanuel Vadot freebsd_committer freebsd_triage 2020-06-04 09:16:43 UTC
Created attachment 215216 [details]
patch with upstream MR commit

I've submitted a MR upstream and use that instead.
If it's good for you I'll commit this.
Comment 2 Piotr Kubaj freebsd_committer freebsd_triage 2020-06-04 09:28:13 UTC
Yes, it works. Please commit that.
Comment 3 commit-hook freebsd_committer freebsd_triage 2020-06-04 09:36:59 UTC
A commit references this bug:

Author: manu
Date: Thu Jun  4 09:36:39 UTC 2020
New revision: 537878
URL: https://svnweb.freebsd.org/changeset/ports/537878

Log:
  libdrm: Fix build on GCC based arch

  PR:		246969
  Reported by:	pkubaj

Changes:
  head/graphics/libdrm/Makefile
  head/graphics/libdrm/distinfo
Comment 4 commit-hook freebsd_committer freebsd_triage 2020-06-06 09:23:32 UTC
A commit references this bug:

Author: manu
Date: Sat Jun  6 09:23:13 UTC 2020
New revision: 538079
URL: https://svnweb.freebsd.org/changeset/ports/538079

Log:
   mesa*: Add platforms option

  Mesa support different platforms, add some options that control which one
  we are enabling and build the -Dplatform meson arg based on that.
  Keep the WAYLAND option for now but if you're using it switch to the
  newly created PLATFORM_WAYLAND.

  PR:		246969
  Approved by:	zeising (x11@ implicit)

Changes:
  head/graphics/mesa-dri/Makefile
  head/graphics/mesa-libs/Makefile