[OE-core] [PATCH v2 1/1] go-runtime: prevent host leakage into target objects
Saul Wold
sgw at linux.intel.com
Wed Oct 4 17:01:37 UTC 2017
On Wed, 2017-10-04 at 16:01 +0000, Matt Madison wrote:
> When building for a target whose architecture matches
> the build host's, the second pass through make.bash
> to build the shareable runtime would also overwrite
> the target's static cgo library with host-compatibile
> binaries.
>
> Fix this by running the host-side build once and
> target-only passes of make.bash twice, for static
> and shareable. This ensures that what gets installed
> is target-compatible.
>
> Also fix an issue with x86-64 targets running MUSL by
> removing the pre-built (for glibc) objects for the
> race detector runtime before building.
>
Please add the [YOCTO #12136] back to denote it's being fixed here.
> Signed-off-by: Matt Madison <matt at madison.systems>
Yup, that worked this time, thanks!
Tested-by: Saul Wold <sgw at linux.intel.com>
> ---
> meta/recipes-devtools/go/go-runtime.inc | 19 +++++++++++++++----
> 1 file changed, 15 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-devtools/go/go-runtime.inc b/meta/recipes-
> devtools/go/go-runtime.inc
> index 934d1aa..f181dc7 100644
> --- a/meta/recipes-devtools/go/go-runtime.inc
> +++ b/meta/recipes-devtools/go/go-runtime.inc
> @@ -15,7 +15,13 @@ export CC_FOR_TARGET="${CC}"
> export CXX_FOR_TARGET="${CXX}"
> export GOROOT_OVERRIDE = "1"
>
> -do_configure[noexec] = "1"
> +do_configure() {
> + :
> +}
> +
> +do_configure_libc-musl() {
> + rm -f ${S}/src/runtime/race/*.syso
> +}
>
> do_compile() {
> export GOBIN="${B}/bin"
> @@ -23,9 +29,13 @@ do_compile() {
> rm -rf ${GOBIN} ${B}/pkg
> mkdir ${GOBIN}
> cd src
> - GO_FLAGS="" ./make.bash
> + ./make.bash --host-only
> + cp ${B}/pkg/tool/${BUILD_GOTUPLE}/go_bootstrap ${B}
> + rm -rf ${B}/pkg/${TARGET_GOTUPLE}
> + ./make.bash --target-only
> if [ -n "${GO_DYNLINK}" ]; then
> - GO_FLAGS="-buildmode=shared" GO_LDFLAGS="-extldflags
> \"${LDFLAGS}\"" ./make.bash
> + cp ${B}/go_bootstrap ${B}/pkg/tool/${BUILD_GOTUPLE}
> + GO_FLAGS="-buildmode=shared" GO_LDFLAGS="-extldflags
> \"${LDFLAGS}\"" ./make.bash --target-only
> fi
> cd ${B}
> }
> @@ -41,8 +51,9 @@ do_install() {
> rm -rf ${D}${libdir}/go/pkg/obj
> rm -rf ${D}${libdir}/go/pkg/bootstrap
> find src -mindepth 1 -maxdepth 1 -type d | while read
> srcdir; do
> - [ "$srcdir" = "./cmd" ] || cp --
> preserve=mode,timestamps -R $srcdir ${D}${libdir}/go/src/
> + cp --preserve=mode,timestamps -R $srcdir
> ${D}${libdir}/go/src/
> done
> + rm -f ${D}${libdir}/go/src/cmd/dist/dist
> }
>
> # Remove test binaries that cannot be relocated
> --
> 2.7.4
>
More information about the Openembedded-core
mailing list