[Lldb-commits] [PATCH] D14790: cmake: add -DLLDB_ALLOW_STATIC_BINDINGS=1, defaults off

[Todd Fiala via lldb-commits]

tfiala added a comment.

In http://reviews.llvm.org/D14790#292226, @zturner wrote:

> So you can't install swig on your Linux machines either?
>
> I really can't wrap my head around why we need this.  What is the problem with installing swig, given that we need to install other software already?


Zachary, I appreciate your dislike of the idea.  And I have totally modified my approach to this to minimize its impact so that it is a no-op for everyone else.  And I am happy to remove this when we have an alternative to not require swig.

> The build instructions here [http://lldb.llvm.org/build.html#BuildingLldbOnLinux] say you need to install swig, libedit, python, and ninja.  You need either git or svn too.  Why must we insist on supporting this configuration when it is not necessary?

> 

> I want the CMake build to build one way.


You are being obstructionist if you go with that statement.  You are not impacted by this change, so attempting to shoot it down based on you not liking it for reasons that your team doesn't care about is not a position that makes sense.  If I was forcing you to do something different, then we have something to work through if it caused undue harm.  When I need to get something done, and I'm tweaking it a lot to appease you, and you still come back with "no, I don't like this so, even though I'm not the one who has a requirement for it, I'm going to say I don't want it" seems counter to an supporting multiple groups with different objectives in the same source base.


http://reviews.llvm.org/D14790