[parted-devel] [PATCH] linux: Include <sys/sysmacros.h> for major() macro.

Sebastian Parschauer sparschauer at suse.de
Thu Apr 5 14:30:44 UTC 2018


On 04.04.2018 18:25, Brian C. Lane wrote:
> From: "Richard W.M. Jones" <rjones at redhat.com>
> 
> Since glibc 2.27 this header is required.
> ---
>  libparted/arch/linux.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libparted/arch/linux.c b/libparted/arch/linux.c
> index 31b98ab..7e86b51 100644
> --- a/libparted/arch/linux.c
> +++ b/libparted/arch/linux.c
> @@ -41,6 +41,7 @@
>  #include <sys/utsname.h>        /* for uname() */
>  #include <scsi/scsi.h>
>  #include <assert.h>
> +#include <sys/sysmacros.h>
>  #ifdef ENABLE_DEVICE_MAPPER
>  #include <libdevmapper.h>
>  #endif
> 

I see, this triggers a warning requesting this change otherwise. Would
look nicer to keep the <sys/*> includes together by inserting two or
even four lines above for keeping alphabetical order but okay.

Acked-by: Sebastian Parschauer <sparschauer at suse.de>



More information about the parted-devel mailing list