[gnome-continuous-yocto/gnomeostree-3.28-rocko: 7893/8267] go: Fix build with -buildmode=pie



commit c1dce31ced7dda38730555a6adf6b81446281a11
Author: Khem Raj <raj khem gmail com>
Date:   Fri Sep 22 18:05:29 2017 -0700

    go: Fix build with -buildmode=pie
    
    (From OE-Core rev: f080dafbbba31686fddb0867cadb8dfe273632f3)
    
    Signed-off-by: Khem Raj <raj khem gmail com>
    Signed-off-by: Richard Purdie <richard purdie linuxfoundation org>

 meta/recipes-devtools/go/go-1.9.inc                |    1 +
 ...dmode-pie-forces-external-linking-mode-on.patch |   47 ++++++++++++++++++++
 2 files changed, 48 insertions(+), 0 deletions(-)
---
diff --git a/meta/recipes-devtools/go/go-1.9.inc b/meta/recipes-devtools/go/go-1.9.inc
index 0b0aca3..65adaa8 100644
--- a/meta/recipes-devtools/go/go-1.9.inc
+++ b/meta/recipes-devtools/go/go-1.9.inc
@@ -14,6 +14,7 @@ SRC_URI += "\
         file://0006-make.bash-add-GOTOOLDIR_BOOTSTRAP-environment-variab.patch \
         file://0007-ld-add-soname-to-shareable-objects.patch \
         file://0008-make.bash-add-GOHOSTxx-indirection-for-cross-canadia.patch \
+        file://0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch \
 "
 SRC_URI[main.md5sum] = "da2d44ea384076efec43ee1f8b7d45d2"
 SRC_URI[main.sha256sum] = "a4ab229028ed167ba1986825751463605264e44868362ca8e7accc8be057e993"
diff --git a/meta/recipes-devtools/go/go-1.9/0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch 
b/meta/recipes-devtools/go/go-1.9/0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch
new file mode 100644
index 0000000..aa5fcfd
--- /dev/null
+++ b/meta/recipes-devtools/go/go-1.9/0009-cmd-go-buildmode-pie-forces-external-linking-mode-on.patch
@@ -0,0 +1,47 @@
+From aae44527c8065d54f6acaf87c82cba1ac96fae59 Mon Sep 17 00:00:00 2001
+From: Ian Lance Taylor <iant golang org>
+Date: Fri, 18 Aug 2017 17:46:03 -0700
+Subject: [PATCH] cmd/go: -buildmode=pie forces external linking mode on all
+ systems
+
+The go tool assumed that -buildmode=pie implied internal linking on
+linux-amd64. However, that was changed by CL 36417 for issue #18968.
+
+Fixes #21452
+
+Change-Id: I8ed13aea52959cc5c53223f4c41ba35329445545
+Reviewed-on: https://go-review.googlesource.com/57231
+Run-TryBot: Ian Lance Taylor <iant golang org>
+TryBot-Result: Gobot Gobot <gobot golang org>
+Reviewed-by: Avelino <t avelino xxx>
+Reviewed-by: Rob Pike <r golang org>
+---
+Upstream-Status: Backport
+Signed-off-by: Khem Raj <raj khem gmail com>
+
+ src/cmd/go/internal/load/pkg.go | 7 ++++---
+ 1 file changed, 4 insertions(+), 3 deletions(-)
+
+diff --git a/src/cmd/go/internal/load/pkg.go b/src/cmd/go/internal/load/pkg.go
+index 2660d3f..d40773b 100644
+--- a/src/cmd/go/internal/load/pkg.go
++++ b/src/cmd/go/internal/load/pkg.go
+@@ -954,11 +954,12 @@ func (p *Package) load(stk *ImportStack, bp *build.Package, err error) *Package
+ 
+       if cfg.BuildContext.CgoEnabled && p.Name == "main" && !p.Goroot {
+               // Currently build modes c-shared, pie (on systems that do not
+-              // support PIE with internal linking mode), plugin, and
+-              // -linkshared force external linking mode, as of course does
++              // support PIE with internal linking mode (currently all
++              // systems: issue #18968)), plugin, and -linkshared force
++              // external linking mode, as of course does
+               // -ldflags=-linkmode=external. External linking mode forces
+               // an import of runtime/cgo.
+-              pieCgo := cfg.BuildBuildmode == "pie" && (cfg.BuildContext.GOOS != "linux" || 
cfg.BuildContext.GOARCH != "amd64")
++              pieCgo := cfg.BuildBuildmode == "pie"
+               linkmodeExternal := false
+               for i, a := range cfg.BuildLdflags {
+                       if a == "-linkmode=external" {
+-- 
+2.14.1
+


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]