RE: [PATCH 2/2] netxen: include ipv6.h (fixes build failure)

From: Dhananjay Phadke
Date: Fri Jan 16 2009 - 14:21:44 EST


My bad, I copy-pasted the --to --cc list to git command line and missed out the original Sender in that email.

Double apologies to Ingo, first for introducing the build error and other for fixing without credit to reporter :-(.

Thanks Ingo, Dave.

-Dhananjay

-----Original Message-----
From: David Miller [mailto:davem@xxxxxxxxxxxxx]
Sent: Friday, January 16, 2009 11:05
To: Dhananjay Phadke
Cc: netdev@xxxxxxxxxxxxxxx; akpm@xxxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; mingo@xxxxxxx
Subject: Re: [PATCH 2/2] netxen: include ipv6.h (fixes build failure)

From: Dhananjay Phadke <dhananjay@xxxxxxxxxx>
Date: Fri, 16 Jan 2009 10:28:04 -0800

> Fixes a build error in absence of CONFIG_IPV6:
>
> drivers/net/netxen/netxen_nic_main.c:1189: error: implicit declaration of function 'ipv6_hdr'
> drivers/net/netxen/netxen_nic_main.c:1189: error: invalid type argument of '->'
>
> Signed-off-by: Dhananjay Phadke <dhananjay@xxxxxxxxxx>

Gee, thanks for not even mentioning Ingo having reported this problem
in your commit message. :-((

I've fixed this when committing.

You didn't even CC: him on this patch posting, what gives?


No virus found in this incoming message.
Checked by AVG.
Version: 7.5.526 / Virus Database: 270.10.6/1889 - Release Date: 1/12/2009 20:18


No virus found in this outgoing message.
Checked by AVG.
Version: 7.5.526 / Virus Database: 270.10.6/1889 - Release Date: 1/12/2009 20:18

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/