[PATCH xserver 2/2] xfree86: add support for Cygwin
Mark Kettenis
mark.kettenis at xs4all.nl
Wed Mar 28 23:33:42 PDT 2012
> Date: Wed, 28 Mar 2012 23:05:51 -0500
> From: "Yaakov (Cygwin/X)" <yselkowitz at users.sourceforge.net>
>
> On 2012-03-28 22:19, Alan Coopersmith wrote:
> > On 03/27/12 10:16 PM, Yaakov (Cygwin/X) wrote:
> >> +
> >> +i2c/libi2c.la:
> >> + $(MAKE) -C $(@D) $(@F)
> >> +
> >> +dixmods/libdixmods.la:
> >> + $(MAKE) -C $(@D) $(@F)
> >> +
> >> +dixmods/libxorgxkb.la:
> >> + $(MAKE) -C $(@D) $(@F)
> >
> > Should those be $(MAKE) $(AM_MAKEFLAGS) ... ? That's what most automake calls
> > seem to use.
> > Also -C is a gnu make specific flag, but I think we still require GNU make
> > to build the server due to other issues anyway.
The server builds fine with (Open)BSD make, and we'd like to keep it
that way. And it doesn't support -C.
>
> $(MAKE) -C is already used in the top-level Makefile, hw/Makefile, and
> hw/kdrive/Makefile. But now that you mention it, I see that I forgot
> the $(AM_V_at).
It's only used for the "relink" target, which isn't actually used for
a normal build. And it bears a "gross hack" comment in the toplevel
Makefile. Can you solve this in a different way? Or perhaps explain
the problem you're trying to solve? Including "." in SUBDIRS seems
like a gross hack to me as well.
More information about the xorg-devel
mailing list