On Sun, Oct 19, 2014 at 09:25:51AM +0200, Matthieu Herrb wrote:
> On Thu, Oct 16, 2014 at 08:14:16PM +0200, Mark Kettenis wrote:
> > Hi Marc,
> >
> > Is there a reason why conditional includes (sinclude/-include) aren't
> > enabled in OpenBSD?
> >
> > I'm asking because the Xorg people now use it in one of the xserver
> > Makefile. We could of course try to convince them to revert the
> > change they made. But it is a somewhat useful feature.
>
> So, now that espie@ has committed the bit, ok to remove the local
> change that made the inclusion unconditionnal (and slightly broke make
> clean in xserver) ?
>
> Index: Makefile.am
> ===================================================================
> RCS file: /cvs/OpenBSD/xenocara/xserver/hw/xfree86/Makefile.am,v
> retrieving revision 1.9
> diff -u -r1.9 Makefile.am
> --- Makefile.am 27 Sep 2014 17:53:01 -0000 1.9
> +++ Makefile.am 19 Oct 2014 07:23:22 -0000
> @@ -138,7 +138,7 @@
> $(AM_V_GEN)CPP='$(CPP)' AWK='$(AWK)' $(SHELL) $(srcdir)/sdksyms.sh
> $(top_srcdir) $(CFLAGS) $(AM_CFLAGS) $(AM_CPPFLAGS)
>
> SDKSYMS_DEP = sdksyms.dep
> -include $(SDKSYMS_DEP)
> +-include $(SDKSYMS_DEP)
>
> i2c/libi2c.la:
> $(AM_V_at)cd i2c && $(MAKE) libi2c.la
> Index: Makefile.in
> ===================================================================
> RCS file: /cvs/OpenBSD/xenocara/xserver/hw/xfree86/Makefile.in,v
> retrieving revision 1.29
> diff -u -r1.29 Makefile.in
> --- Makefile.in 27 Sep 2014 17:53:01 -0000 1.29
> +++ Makefile.in 19 Oct 2014 07:23:22 -0000
> @@ -1116,7 +1116,7 @@
>
> sdksyms.dep sdksyms.c: sdksyms.sh
> $(AM_V_GEN)CPP='$(CPP)' AWK='$(AWK)' $(SHELL) $(srcdir)/sdksyms.sh
> $(top_srcdir) $(CFLAGS) $(AM_CFLAGS) $(AM_CPPFLAGS)
> -include $(SDKSYMS_DEP)
> +-include $(SDKSYMS_DEP)
>
> i2c/libi2c.la:
> $(AM_V_at)cd i2c && $(MAKE) libi2c.la
>
> --
Wait for a bit until new snaps have the new make, then sure.