Message ID | bug-109587-502@http.bugs.freedesktop.org/ (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [Bug,109587] "xf86drm: Add drmIsMaster()" commit breaks X server builds | expand |
https://bugs.freedesktop.org/show_bug.cgi?id=109587 Eero Tamminen <eero.t.tamminen@intel.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #1 from Eero Tamminen <eero.t.tamminen@intel.com> --- Fixed by: ----------------------------------------------------- commit eba6609b7bee5a56fa5334992b1e6f3aef8d244a Author: Eric Engestrom <eric.engestrom@intel.com> Date: Fri Feb 8 14:46:07 2019 +0000 xf86drm: fix return type for drmIsMaster() Xserver has struct members named `bool`, which means the last commit breaks its build with errors like this: error: two or more data types in declaration specifiers Bool bool; ^ Fix this by making it return a 0/1 integer, with the same semantic as the boolean it was before. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=109587 Fixes: 17dfe3ac93217b43f93b "xf86drm: Add drmIsMaster()" Cc: Christopher James Halse Rogers <christopher.halse.rogers@canonical.com> Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> ----------------------------------------------------- I hope somebody fixes also those X server xf86Opt.h & winconfig.h headers soon as even some of X server code itself is using stdbool.h: ----------------------------------------------------- $ git grep stdbool.h hw/xquartz/GL/capabilities.h:#include <stdbool.h> hw/xquartz/mach-startup/bundle-main.c:#include <stdbool.h> hw/xwin/winmultiwindowicons.c:#include <stdbool.h> test/damage/primitives.c:#include <stdbool.h> test/simple-xinit.c:#include <stdbool.h> -----------------------------------------------------
https://bugs.freedesktop.org/show_bug.cgi?id=109587 Michel Dänzer <michel@daenzer.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pedretti.fabio@gmail.com --- Comment #2 from Michel Dänzer <michel@daenzer.net> --- *** Bug 109678 has been marked as a duplicate of this bug. ***
--- a/xf86drm.h +++ b/xf86drm.h @@ -37,6 +37,7 @@ #include <stdarg.h> #include <sys/types.h> #include <stdint.h> +#include <stdbool.h> ----------------------------------------------- Breaks X server build: ----------------------------------------------- FAILED: hw/xfree86/dri2/xorg_dri2@sta/dri2.c.o