All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] distro: default: Change Go default version to 1.9
@ 2018-04-11 14:32 Otavio Salvador
  2018-04-11 15:50 ` Khem Raj
  0 siblings, 1 reply; 2+ messages in thread
From: Otavio Salvador @ 2018-04-11 14:32 UTC (permalink / raw)
  To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador

For the time being, there is a serious bug[1] in Go 1.10 when it comes to
use the shared runtime support which cases problems in multiple projects.

 1. https://github.com/golang/go/issues/24640

It is still unclear if the problem arises from a bug inside the
compiler itself or it makes a real problem more visible. Either way,
using 1.10 as default seems to be a risk so we are changing back to
1.9 for now.

Refs: [YOCTO: #12631]

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---

 meta/conf/distro/include/tcmode-default.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
index 7862edb6333..00399ae820c 100644
--- a/meta/conf/distro/include/tcmode-default.inc
+++ b/meta/conf/distro/include/tcmode-default.inc
@@ -72,7 +72,7 @@ PREFERRED_VERSION_nativesdk-qemu ?= "${QEMUVERSION}"
 # Setup suitable toolchain flags
 require conf/distro/include/as-needed.inc
 
-GOVERSION ?= "1.10%"
+GOVERSION ?= "1.9%"
 PREFERRED_VERSION_virtual/${TARGET_PREFIX}go ?= "${GOVERSION}"
 PREFERRED_VERSION_go-cross-${TARGET_ARCH} ?= "${GOVERSION}"
 PREFERRED_VERSION_go-crosssdk-${SDK_ARCH} ?= "${GOVERSION}"
-- 
2.17.0



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

* Re: [PATCH] distro: default: Change Go default version to 1.9
  2018-04-11 14:32 [PATCH] distro: default: Change Go default version to 1.9 Otavio Salvador
@ 2018-04-11 15:50 ` Khem Raj
  0 siblings, 0 replies; 2+ messages in thread
From: Khem Raj @ 2018-04-11 15:50 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: OpenEmbedded Core Mailing List

On Wed, Apr 11, 2018 at 7:32 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> For the time being, there is a serious bug[1] in Go 1.10 when it comes to
> use the shared runtime support which cases problems in multiple projects.
>
>  1. https://github.com/golang/go/issues/24640
>
> It is still unclear if the problem arises from a bug inside the
> compiler itself or it makes a real problem more visible. Either way,
> using 1.10 as default seems to be a risk so we are changing back to
> 1.9 for now.

agreed LGTM

>
> Refs: [YOCTO: #12631]
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>
>  meta/conf/distro/include/tcmode-default.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
> index 7862edb6333..00399ae820c 100644
> --- a/meta/conf/distro/include/tcmode-default.inc
> +++ b/meta/conf/distro/include/tcmode-default.inc
> @@ -72,7 +72,7 @@ PREFERRED_VERSION_nativesdk-qemu ?= "${QEMUVERSION}"
>  # Setup suitable toolchain flags
>  require conf/distro/include/as-needed.inc
>
> -GOVERSION ?= "1.10%"
> +GOVERSION ?= "1.9%"
>  PREFERRED_VERSION_virtual/${TARGET_PREFIX}go ?= "${GOVERSION}"
>  PREFERRED_VERSION_go-cross-${TARGET_ARCH} ?= "${GOVERSION}"
>  PREFERRED_VERSION_go-crosssdk-${SDK_ARCH} ?= "${GOVERSION}"
> --
> 2.17.0
>


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

end of thread, other threads:[~2018-04-11 15:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-11 14:32 [PATCH] distro: default: Change Go default version to 1.9 Otavio Salvador
2018-04-11 15:50 ` Khem Raj

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.