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 > >