All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tcmode-default.inc: clear PREFERRED_VERSION for llvm
@ 2022-01-16 16:47 kai.kang
  2022-01-16 18:43 ` Khem Raj
  0 siblings, 1 reply; 3+ messages in thread
From: kai.kang @ 2022-01-16 16:47 UTC (permalink / raw)
  To: raj.khem; +Cc: openembedded-core

From: Kai Kang <kai.kang@windriver.com>

When include layer meta-clang which set LLVMVERSION in layer.conf, it
shows warnings:

| WARNING: preferred version 13.0.1 of llvm-native not available (for item llvm-native)
| WARNING: versions of llvm-native available: 12.0.1
| WARNING: preferred version 13.0.1 of llvm not available (for item llvm)
| WARNING: versions of llvm available: 12.0.1
| WARNING: preferred version 13.0.1 of llvm not available (for item llvm-dev)
| WARNING: versions of llvm available: 12.0.1
| WARNING: preferred version 13.0.1 of llvm not available (for item llvm-bugpointpasses)
| WARNING: versions of llvm available: 12.0.1
| WARNING: preferred version 13.0.1 of llvm not available (for item llvm-liboptremarks)
| WARNING: versions of llvm available: 12.0.1
| WARNING: preferred version 13.0.1 of llvm not available (for item llvm-llvmhello)
| WARNING: versions of llvm available: 12.0.1

Because only one version llvm exists, remove the PREFERRED_VERSION
setting for llvm.

Signed-off-by: Kai Kang <kai.kang@windriver.com>
---
 meta/conf/distro/include/tcmode-default.inc | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
index 49e99af8f2..2622b055a2 100644
--- a/meta/conf/distro/include/tcmode-default.inc
+++ b/meta/conf/distro/include/tcmode-default.inc
@@ -79,10 +79,6 @@ PREFERRED_VERSION_go-runtime ?= "${GOVERSION}"
 PREFERRED_VERSION_nativesdk-go ?= "${GOVERSION}"
 PREFERRED_VERSION_nativesdk-go-runtime ?= "${GOVERSION}"
 
-PREFERRED_VERSION_llvm = "${LLVMVERSION}"
-PREFERRED_VERSION_llvm-native = "${LLVMVERSION}"
-PREFERRED_VERSION_nativesdk-llvm = "${LLVMVERSION}"
-
 # Rust toolchain preferred versions:
 
 PREFERRED_VERSION_cargo ?= "${RUSTVERSION}"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] tcmode-default.inc: clear PREFERRED_VERSION for llvm
  2022-01-16 16:47 [PATCH] tcmode-default.inc: clear PREFERRED_VERSION for llvm kai.kang
@ 2022-01-16 18:43 ` Khem Raj
  2022-01-17  3:35   ` Kai
  0 siblings, 1 reply; 3+ messages in thread
From: Khem Raj @ 2022-01-16 18:43 UTC (permalink / raw)
  To: kai.kang; +Cc: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 2214 bytes --]

On Sun, Jan 16, 2022 at 8:47 AM <kai.kang@windriver.com> wrote:

> From: Kai Kang <kai.kang@windriver.com>
>
> When include layer meta-clang which set LLVMVERSION in layer.conf, it
> shows warnings:
>
> | WARNING: preferred version 13.0.1 of llvm-native not available (for item
> llvm-native)
> | WARNING: versions of llvm-native available: 12.0.1
> | WARNING: preferred version 13.0.1 of llvm not available (for item llvm)
> | WARNING: versions of llvm available: 12.0.1
> | WARNING: preferred version 13.0.1 of llvm not available (for item
> llvm-dev)
> | WARNING: versions of llvm available: 12.0.1
> | WARNING: preferred version 13.0.1 of llvm not available (for item
> llvm-bugpointpasses)
> | WARNING: versions of llvm available: 12.0.1
> | WARNING: preferred version 13.0.1 of llvm not available (for item
> llvm-liboptremarks)
> | WARNING: versions of llvm available: 12.0.1
> | WARNING: preferred version 13.0.1 of llvm not available (for item
> llvm-llvmhello)
> | WARNING: versions of llvm available: 12.0.1
>
> Because only one version llvm exists, remove the PREFERRED_VERSION
> setting for llvm.


I think these are PROVIDERS mismatch more than anything between two recipes
so perhaps it’s better to keep that in sync

Fwiw llvm is not heavily used in oe-core perhaps it should be removed

>
>
> Signed-off-by: Kai Kang <kai.kang@windriver.com>
> ---
>  meta/conf/distro/include/tcmode-default.inc | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/meta/conf/distro/include/tcmode-default.inc
> b/meta/conf/distro/include/tcmode-default.inc
> index 49e99af8f2..2622b055a2 100644
> --- a/meta/conf/distro/include/tcmode-default.inc
> +++ b/meta/conf/distro/include/tcmode-default.inc
> @@ -79,10 +79,6 @@ PREFERRED_VERSION_go-runtime ?= "${GOVERSION}"
>  PREFERRED_VERSION_nativesdk-go ?= "${GOVERSION}"
>  PREFERRED_VERSION_nativesdk-go-runtime ?= "${GOVERSION}"
>
> -PREFERRED_VERSION_llvm = "${LLVMVERSION}"
> -PREFERRED_VERSION_llvm-native = "${LLVMVERSION}"
> -PREFERRED_VERSION_nativesdk-llvm = "${LLVMVERSION}"
> -
>  # Rust toolchain preferred versions:
>
>  PREFERRED_VERSION_cargo ?= "${RUSTVERSION}"
> --
> 2.17.1
>
>

[-- Attachment #2: Type: text/html, Size: 2976 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] tcmode-default.inc: clear PREFERRED_VERSION for llvm
  2022-01-16 18:43 ` Khem Raj
@ 2022-01-17  3:35   ` Kai
  0 siblings, 0 replies; 3+ messages in thread
From: Kai @ 2022-01-17  3:35 UTC (permalink / raw)
  To: Khem Raj; +Cc: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 2739 bytes --]

On 1/17/22 2:43 AM, Khem Raj wrote:
>
>
> On Sun, Jan 16, 2022 at 8:47 AM <kai.kang@windriver.com 
> <mailto:kai.kang@windriver.com>> wrote:
>
>     From: Kai Kang <kai.kang@windriver.com
>     <mailto:kai.kang@windriver.com>>
>
>     When include layer meta-clang which set LLVMVERSION in layer.conf, it
>     shows warnings:
>
>     | WARNING: preferred version 13.0.1 of llvm-native not available
>     (for item llvm-native)
>     | WARNING: versions of llvm-native available: 12.0.1
>     | WARNING: preferred version 13.0.1 of llvm not available (for
>     item llvm)
>     | WARNING: versions of llvm available: 12.0.1
>     | WARNING: preferred version 13.0.1 of llvm not available (for
>     item llvm-dev)
>     | WARNING: versions of llvm available: 12.0.1
>     | WARNING: preferred version 13.0.1 of llvm not available (for
>     item llvm-bugpointpasses)
>     | WARNING: versions of llvm available: 12.0.1
>     | WARNING: preferred version 13.0.1 of llvm not available (for
>     item llvm-liboptremarks)
>     | WARNING: versions of llvm available: 12.0.1
>     | WARNING: preferred version 13.0.1 of llvm not available (for
>     item llvm-llvmhello)
>     | WARNING: versions of llvm available: 12.0.1
>
>     Because only one version llvm exists, remove the PREFERRED_VERSION
>     setting for llvm.
>
>
> I think these are PROVIDERS mismatch more than anything between two 
> recipes so perhaps it’s better to keep that in sync

My doubt is that it seems only a single llvm recipe exists in oe-core 
around all the layers including meta-clang.

Regards,
Kai

>
> Fwiw llvm is not heavily used in oe-core perhaps it should be removed
>
>
>
>     Signed-off-by: Kai Kang <kai.kang@windriver.com
>     <mailto:kai.kang@windriver.com>>
>     ---
>      meta/conf/distro/include/tcmode-default.inc | 4 ----
>      1 file changed, 4 deletions(-)
>
>     diff --git a/meta/conf/distro/include/tcmode-default.inc
>     b/meta/conf/distro/include/tcmode-default.inc
>     index 49e99af8f2..2622b055a2 100644
>     --- a/meta/conf/distro/include/tcmode-default.inc
>     +++ b/meta/conf/distro/include/tcmode-default.inc
>     @@ -79,10 +79,6 @@ PREFERRED_VERSION_go-runtime ?= "${GOVERSION}"
>      PREFERRED_VERSION_nativesdk-go ?= "${GOVERSION}"
>      PREFERRED_VERSION_nativesdk-go-runtime ?= "${GOVERSION}"
>
>     -PREFERRED_VERSION_llvm = "${LLVMVERSION}"
>     -PREFERRED_VERSION_llvm-native = "${LLVMVERSION}"
>     -PREFERRED_VERSION_nativesdk-llvm = "${LLVMVERSION}"
>     -
>      # Rust toolchain preferred versions:
>
>      PREFERRED_VERSION_cargo ?= "${RUSTVERSION}"
>     -- 
>     2.17.1
>

-- 
Kai Kang
Wind River Linux


[-- Attachment #2: Type: text/html, Size: 4850 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-01-17  3:35 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-01-16 16:47 [PATCH] tcmode-default.inc: clear PREFERRED_VERSION for llvm kai.kang
2022-01-16 18:43 ` Khem Raj
2022-01-17  3:35   ` Kai

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.