In article <e1qyqfe-0006ir...@dinsdale.python.org>, benjamin.peterson <python-check...@python.org> wrote:
> http://hg.python.org/cpython/rev/0d0ca6f95d9d > changeset: 70879:0d0ca6f95d9d > user: Benjamin Peterson <benja...@python.org> > date: Sun Jun 19 17:17:30 2011 -0500 > summary: > remove duplicate -I > > files: > Makefile.pre.in | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > > diff --git a/Makefile.pre.in b/Makefile.pre.in > --- a/Makefile.pre.in > +++ b/Makefile.pre.in > @@ -73,7 +73,7 @@ > # Both CPPFLAGS and LDFLAGS need to contain the shell's value for setup.py > to > # be able to build extension modules using the directories specified in the > # environment variables > -PY_CPPFLAGS= -I. -IInclude -I$(srcdir)/Include $(CONFIGURE_CPPFLAGS) > $(CPPFLAGS) > +PY_CPPFLAGS= -I. -IInclude $(CONFIGURE_CPPFLAGS) $(CPPFLAGS) > PY_LDFLAGS= $(CONFIGURE_LDFLAGS) $(LDFLAGS) > NO_AS_NEEDED= @NO_AS_NEEDED@ > LDLAST= @LDLAST@ I think someone has already mentioned that this change breaks builds with a source directory outside the build directory and that breaks my OS X installer builds. Please revert or make the following change which works for me (but not tested elsewhere): -PY_CPPFLAGS= -I. -IInclude $(CONFIGURE_CPPFLAGS) $(CPPFLAGS) +PY_CPPFLAGS= -I. -I$(srcdir)/Include $(CONFIGURE_CPPFLAGS) $(CPPFLAGS) -- Ned Deily, n...@acm.org _______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
