Reviewed-by: Konstantin Kostiuk <kkost...@redhat.com>

On Thu, Jan 11, 2024 at 10:18 PM Nick Briggs <nicholas.h.bri...@gmail.com>
wrote:

> Solaris has net/if_arp.h and netinet/if_ether.h rather than net/ethernet.h,
> but does not define ETHER_ADDR_LEN, instead providing ETHERADDRL.
>
> Signed-off-by: Nick Briggs <nicholas.h.bri...@gmail.com>
> ---
>  qga/commands-posix.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/qga/commands-posix.c b/qga/commands-posix.c
> index 6169bbf7a0..26008db497 100644
> --- a/qga/commands-posix.c
> +++ b/qga/commands-posix.c
> @@ -45,9 +45,12 @@
>  #include <arpa/inet.h>
>  #include <sys/socket.h>
>  #include <net/if.h>
> -#if defined(__NetBSD__) || defined(__OpenBSD__)
> +#if defined(__NetBSD__) || defined(__OpenBSD__) || defined(CONFIG_SOLARIS)
>  #include <net/if_arp.h>
>  #include <netinet/if_ether.h>
> +#if !defined(ETHER_ADDR_LEN) && defined(ETHERADDRL)
> +#define ETHER_ADDR_LEN ETHERADDRL
> +#endif
>  #else
>  #include <net/ethernet.h>
>  #endif
> --
> 2.31.1
>
>

Reply via email to