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

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to