This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Untangle linking of dependencies
ClosedPublic

Authored by JDevlieghere on Mar 28 2019, 4:50 PM.

Details

Summary

The utility library shouldn't depend on curses, libedit or python since it uses none of them. Move the first two to libCore where they are actually used.

Diff Detail

Repository
rLLDB LLDB

Event Timeline

JDevlieghere created this revision.Mar 28 2019, 4:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 28 2019, 4:50 PM
Herald added a subscriber: mgorny. · View Herald Transcript

And move python to the ScriptInterpreter

I agree Utility is not the right place for this, but I am not sure about Core either.

For curses, that's probably fine as an interim solution, because the usage is in IOHandler.

However, for libedit, I'd expect this stuff to be in Host, since that's where we have the Editline.h wrapper. Core can then inherit anything it needs from there since it's higher up in the dependency chain. Is there any reason why that couldn't work?

This revision was not accepted when it landed; it landed in state Needs Review.Mar 29 2019, 10:45 AM
This revision was automatically updated to reflect the committed changes.