[libvirt] [PATCH] Prefix major/minor with gnu_dev_

[Daniel P. Berrange]

On Mon, Sep 05, 2016 at 03:33:36PM +0200, Michal Privoznik wrote:
> In the latest glibc, major() and minor() functions are marked as
> deprecated (glibc commit dbab6577):
> 
>   CC       util/libvirt_util_la-vircgroup.lo
> util/vircgroup.c: In function 'virCgroupGetBlockDevString':
> util/vircgroup.c:768:5: error: '__major_from_sys_types' is deprecated:
>   In the GNU C Library, `major' is defined by <sys/sysmacros.h>.
>   For historical compatibility, it is currently defined by
>   <sys/types.h> as well, but we plan to remove this soon.
>   To use `major', include <sys/sysmacros.h> directly.
>   If you did not intend to use a system-defined macro `major',
>   you should #undef it after including <sys/types.h>.
>   [-Werror=deprecated-declarations]
>      if (virAsprintf(&ret, "%d:%d ", major(sb.st_rdev), minor(sb.st_rdev)) < 0)
>      ^~
> In file included from /usr/include/features.h:397:0,
>                  from /usr/include/bits/libc-header-start.h:33,
>                  from /usr/include/stdio.h:28,
>                  from ../gnulib/lib/stdio.h:43,
>                  from util/vircgroup.c:26:
> /usr/include/sys/sysmacros.h:87:1: note: declared here
>  __SYSMACROS_DEFINE_MAJOR (__SYSMACROS_FST_IMPL_TEMPL)
>  ^
> 
> Applications are supposed to use gnu_dev_major() or
> gnu_dev_minor() respectively.

How does this work on non-gnu systems like *BSD / OS-X / Mingw[1] ?


Regards,
Daniel

[1] Presumably our mingw setup doesn't build the files affected ?
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|