| 1 |
On Tue, 2019-04-30 at 07:38 +0200, Michał Górny wrote: |
| 2 |
> Hi, |
| 3 |
> |
| 4 |
> Here's my proposed EAPI 7 support for llvm.eclass. I think it should |
| 5 |
> conceptually work with cross but I haven't tested it. The main problem |
| 6 |
> is that we have multiple distinct ways of building LLVM, and AFAIU |
| 7 |
> when using CMake modules, you should be able to get away without having |
| 8 |
> LLVM in CBUILD. However, when using llvm-config you obviously need |
| 9 |
> it CBUILD-executable. |
| 10 |
> |
| 11 |
> To support both scenarios, get_llvm_prefix is now provided with '-b' |
| 12 |
> and '-d' options (matching those to has_version). Using them, you can |
| 13 |
> choose the behavior suitable for your package. |
| 14 |
> |
| 15 |
> I've left the default at '-d'. I don't really know which is better |
| 16 |
> for the generic cross case but either way should work for non-cross |
| 17 |
> setups. |
| 18 |
> |
| 19 |
> -- |
| 20 |
> Best regards, |
| 21 |
> Michał Górny |
| 22 |
> |
| 23 |
> Michał Górny (4): |
| 24 |
> llvm.eclass: Remove unnecessary '_rc' from < example |
| 25 |
> llvm.eclass: Update examples for newer LLVM versions |
| 26 |
> llvm.eclass: Add EAPI 7 API to get_llvm_prefix |
| 27 |
> llvm.eclass: Enable EAPI 7 |
| 28 |
> |
| 29 |
> eclass/llvm.eclass | 65 +++++++++++++++++++++++++++++++++++++--------- |
| 30 |
> 1 file changed, 53 insertions(+), 12 deletions(-) |
| 31 |
> |
| 32 |
|
| 33 |
Merged. |
| 34 |
|
| 35 |
-- |
| 36 |
Best regards, |
| 37 |
Michał Górny |
Archives