[PATCH:libdrm] include <sys/sysmacros.h> directly for major() and minor()

Emil Velikov emil.l.velikov at gmail.com
Wed Jul 8 09:19:59 PDT 2015


Hi Alan

On 7 July 2015 at 01:45, Alan Coopersmith <alan.coopersmith at oracle.com> wrote:
> Linux seems to pick this up via another header, but Solaris needs it
> explictly included, or we get undefined symbol errors for major & minor.
>
A slightly pedandic suggestion:
Can we use sys/types.h + sys/mkdev.h as the manpage suggests. We
already do so in xf86drm.c

Thanks
Emil


More information about the dri-devel mailing list