On Mon, Dec 08, 2014 at 01:04:28PM -0500, Steve Dickson wrote: >On 12/02/2014 07:40 AM, David Härdeman wrote: >> Some of the headers in ./support/include get generated as part of the >> build process (and end up in the build dir hierarchy) and some are >> not (so they stay in the srcdir hierarchy). >> >> That means the build breaks when using a separate build dir. I'm not sure >> what the best way to make sure that -I$(top_srcdir)/support/include is >> defined everywhere is. Consider this a basis for discussion rather than >> a real patch. Someone with more autoconf-fu can surely come up with a better >> solution. > >This one I didn't commit because it caused -I../../support/include to >defined twice. That path gets define ing Makefile.in file in the >DEFAULT_INCLUDES macro. But only if $(top_srcdir) == $(top_builddir) So, for example, if you do: cd nfs-utils mkdir tmp cd tmp ../configure make Then you'd have -I../../support/include and -I../../../support/include? I think this is in line with: http://www.gnu.org/software/automake/manual/html_node/Program-Variables.html See the section on AM_CPPFLAGS... An example from Gutenprint2: AM_CPPFLAGS = -I$(top_srcdir)/include -I$(top_builddir)/include $(LOCAL_CPPFLAGS) $(GNUCFLAGS) Though a more "complete" fix might be to either have a common include file, or a non-recursive make :) -- David Härdeman -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
