On Tue, 2019-04-30 at 07:38 +0200, Michał Górny wrote: > Hi, > > Here's my proposed EAPI 7 support for llvm.eclass. I think it should > conceptually work with cross but I haven't tested it. The main problem > is that we have multiple distinct ways of building LLVM, and AFAIU > when using CMake modules, you should be able to get away without having > LLVM in CBUILD. However, when using llvm-config you obviously need > it CBUILD-executable. > > To support both scenarios, get_llvm_prefix is now provided with '-b' > and '-d' options (matching those to has_version). Using them, you can > choose the behavior suitable for your package. > > I've left the default at '-d'. I don't really know which is better > for the generic cross case but either way should work for non-cross > setups. > > -- > Best regards, > Michał Górny > > Michał Górny (4): > llvm.eclass: Remove unnecessary '_rc' from < example > llvm.eclass: Update examples for newer LLVM versions > llvm.eclass: Add EAPI 7 API to get_llvm_prefix > llvm.eclass: Enable EAPI 7 > > eclass/llvm.eclass | 65 +++++++++++++++++++++++++++++++++++++--------- > 1 file changed, 53 insertions(+), 12 deletions(-) >
Merged. -- Best regards, Michał Górny
signature.asc
Description: This is a digitally signed message part