[Lldb-commits] [PATCH] D43837: Move option parsing out of the Args class

[Zachary Turner via Phabricator via lldb-commits]

zturner added a comment.

In https://reviews.llvm.org/D43837#1022554, @jingham wrote:

> Okay, that sounds good then.  Will you enforce the rule about the Utilities directory socially or by some mechanism?


If you mean the rule that Utility can't depend on anything else, I think it's enforced implicitly.  because we have a `UtilityTests` unit test that only links against `Utility` and nothing else.  If anything else gets brought in, this binary will fail to link.

That said, a while back I wrote a python script (checked in under `lldb/scripts`) which dumps the set of header dependencies for every project.  It would perhaps be a useful exercise to write a lit test that enforces this by dumping the deps of `Utility` and ensuring that they're empty.


https://reviews.llvm.org/D43837