Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 815361 - lldb doesn't need clang[xml]?
Summary: lldb doesn't need clang[xml]?
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: LLVM support project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-29 15:14 UTC by Alex Xu (Hello71)
Modified: 2021-10-04 13:29 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Xu (Hello71) 2021-09-29 15:14:49 UTC
maybe it was true in some old version, but looking at llvm 12.0.1, clang[xml] only enables c-index-test, which seems not used by lldb. libxml2 seems more important for lldb, used for gdb remote and plist parsing. neither of these seem critical though, and doesn't need to pull in clang[xml], only libxml2 directly.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2021-10-04 13:21:44 UTC
You are probably correct.  I don't really recall why we did that but I suppose LLDB might have used clang's variables for determining whether libxml2 was available.  Nowadays it checks for it itself.
Comment 2 Larry the Git Cow gentoo-dev 2021-10-04 13:29:36 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f2d9718baa48a98e90754834b467dc61ad2675f0

commit f2d9718baa48a98e90754834b467dc61ad2675f0
Author:     Michał Górny <mgorny@gentoo.org>
AuthorDate: 2021-10-04 13:25:29 +0000
Commit:     Michał Górny <mgorny@gentoo.org>
CommitDate: 2021-10-04 13:25:29 +0000

    dev-util/lldb: Make xml optional and depend on libxml2 directly
    
    Closes: https://bugs.gentoo.org/815361
    Signed-off-by: Michał Górny <mgorny@gentoo.org>

 dev-util/lldb/lldb-12.0.1.ebuild      | 6 ++++--
 dev-util/lldb/lldb-13.0.0.9999.ebuild | 6 ++++--
 dev-util/lldb/lldb-13.0.0.ebuild      | 6 ++++--
 dev-util/lldb/lldb-14.0.0.9999.ebuild | 6 ++++--
 4 files changed, 16 insertions(+), 8 deletions(-)