[Lldb-commits] [lldb] r133463 - /lldb/trunk/source/Interpreter/Makefile

Peter Collingbourne peter at pcc.me.uk
Wed Jun 22 15:45:08 PDT 2011


On Tue, Jun 21, 2011 at 11:31:18PM -0600, Charles Davis wrote:
> On 6/20/11 1:07 PM, Peter Collingbourne wrote:
> > Author: pcc
> > Date: Mon Jun 20 14:07:07 2011
> > New Revision: 133463
> > 
> > URL: http://llvm.org/viewvc/llvm-project?rev=133463&view=rev
> > Log:
> > Add dependency tracking/clean rule to interpreter makefile
> Unfortunately, this change doesn't work on Darwin because...
> > @@ -32,11 +32,19 @@
> [...]
> > +SWIG_DEPEND_OPTIONS = -MMD -MF "$(PROJ_OBJ_DIR)/$*.cpp.d.tmp" \
> > +                      -MT "$*.cpp" -MT "$(PROJ_OBJ_DIR)/$*.cpp.d"
> ...Darwin's swig (as of Snow Leopard) doesn't support -MT. And it's
> unlikely ever to, because Apple hates and fears the GPLv3.

That's too bad.  But it seems that removing the -MT options isn't
much of a loss here, as we only lose the .d target, which doesn't
seem to be used or needed (I based this part of the Makefile on LLVM's
Makefile.rules, and it isn't entirely clear whether the .d target is
required there either).

I removed -MT in r133660.

Thanks,
-- 
Peter



More information about the lldb-commits mailing list