freebsd stable - Dec 2015 - make tinderbox broken with LINT-NOINET on ref10-amd64.freebsd.org (if_gif compilation issues)

On 14.12.15 19:51, Mariusz Zaborski wrote:
> On Mon, Dec 14, 2015 at 06:31:52AM -0800, NGie Cooper wrote:
>> Hi Andrey, Just a heads up that LINT-NOINET seems to be broken on
>> amd64/i386 with if_gif(4) on stable/10. Could you please fix this? 
>> Thanks, -NGie
> Hi,
> 
> I already found the problem we need to MFC r276224 and r277130.
I don't think r277130 is related to this problem.
I have this patch and I'm trying to build with them.

-- 
WBR, Andrey V. Elsukov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: modules.diff
Type: text/x-patch
Size: 1031 bytes
Desc: not available
URL:
<http://lists.freebsd.org/pipermail/freebsd-stable/attachments/20151214/a0c9e72b/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 538 bytes
Desc: OpenPGP digital signature
URL:
<http://lists.freebsd.org/pipermail/freebsd-stable/attachments/20151214/a0c9e72b/attachment.sig>

On Mon, Dec 14, 2015 at 07:55:06PM +0300, Andrey V. Elsukov
wrote:
> On 14.12.15 19:51, Mariusz Zaborski wrote:
> > On Mon, Dec 14, 2015 at 06:31:52AM -0800, NGie Cooper wrote:
> >> Hi Andrey, Just a heads up that LINT-NOINET seems to be broken on
> >> amd64/i386 with if_gif(4) on stable/10. Could you please fix this?
> >> Thanks, -NGie
> > Hi,
> > 
> > I already found the problem we need to MFC r276224 and r277130.
> 
> I don't think r277130 is related to this problem.
> I have this patch and I'm trying to build with them.
Not with this but after fixing r276224 it will crash in other place which
r277130 is fixing. :)
> 
> -- 
> WBR, Andrey V. Elsukov
> Index: stable/10/sys/modules/if_gif/Makefile
> ==================================================================> ---
stable/10/sys/modules/if_gif/Makefile	(revision 292213)
> +++ stable/10/sys/modules/if_gif/Makefile	(working copy)
> @@ -5,7 +5,7 @@
>  .PATH: ${.CURDIR}/../../net ${.CURDIR}/../../netinet
${.CURDIR}/../../netinet6
>  
>  KMOD=	if_gif
> -SRCS=	if_gif.c in_gif.c opt_inet.h opt_inet6.h opt_mrouting.h
> +SRCS=	if_gif.c opt_inet.h opt_inet6.h opt_mrouting.h
>  
>  .if !defined(KERNBUILDDIR)
>  opt_inet.h:
> @@ -15,10 +15,11 @@ opt_inet.h:
>  opt_inet6.h:
>  	echo "#define INET6 1" > ${.TARGET}
>  .endif
> -
> -opt_mrouting.h:
> -	echo "#define MROUTING 1" > ${.TARGET}
>  .else
> +OPT_INET!= cat ${KERNBUILDDIR}/opt_inet.h; echo
> +.if empty(OPT_INET)
> +MK_INET_SUPPORT=no
> +.endif
>  OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h; echo
>  .if empty(OPT_INET6)
>  MK_INET6_SUPPORT= no
> @@ -25,6 +26,10 @@ MK_INET6_SUPPORT= no
>  .endif
>  .endif
>  
> +.if ${MK_INET_SUPPORT} != "no"
> +SRCS+=  in_gif.c
> +.endif
> +
>  .if ${MK_INET6_SUPPORT} != "no"
>  SRCS+=	in6_gif.c
>  .endif



-- 
Mariusz Zaborski	
oshogbo//vx		| http://oshogbo.vexillium.org
FreeBSD commiter	| https://freebsd.org
Software developer	| http://wheelsystems.com
If it's not broken, let's fix it till it is!!1