Bug 815361

Summary:	lldb doesn't need clang[xml]?
Product:	Gentoo Linux	Reporter:	Alex Xu (Hello71) <alex_y_xu>
Component:	Current packages	Assignee:	LLVM support project <llvm>
Status:	RESOLVED FIXED
Severity:	normal	CC:	mgorny
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

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

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(-)