X-Original-To: alpine-aports@mail.alpinelinux.org Delivered-To: alpine-aports@mail.alpinelinux.org Received: from mail.alpinelinux.org (dallas-a1.alpinelinux.org [127.0.0.1]) by mail.alpinelinux.org (Postfix) with ESMTP id 4F3ADDC0D6B for ; Mon, 22 Feb 2016 15:02:52 +0000 (UTC) Received: from mail-lb0-f170.google.com (mail-lb0-f170.google.com [209.85.217.170]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mail.alpinelinux.org (Postfix) with ESMTPS id 5E649DC0D68 for ; Mon, 22 Feb 2016 15:02:48 +0000 (UTC) Received: by mail-lb0-f170.google.com with SMTP id x1so83442899lbj.3 for ; Mon, 22 Feb 2016 07:02:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-type:content-transfer-encoding; bh=4HzH60a3ef3q+7/lMJZ7gcoA+VuFT9LhLKhqTzP6vFI=; b=CEs5gJfY3QYR9AacREqvDwEOsgxO2ymw8Seh9v7SWHf5Lil0SPod4DXeLSaCxucJrR Vum2PwAXZjnolhI5B6Hs/DxY/dkTWF6fYsuNJsfKz1Ku5PUq5eWDYz8xAvMTZVGeGoIf SwU664yXHP+gcwL96gIFm9JZCoK57gTuQam11JANqZ0d+rvHpgAX8piRPE7sSgPQSYAy BlWpsuUqUIBnQaf4J04qTwggc7lmuelw1EisYiEeCHrCB+2pbkyc0+79v1zMrpxWXT5o zdv7BwQDbmIl6hlIx9Ik09CK/skoh8FSbnkjmWme0IsQE3BBp3Pikt8dGlBMIbmplu1r Z9hA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :in-reply-to:references:mime-version:content-type :content-transfer-encoding; bh=4HzH60a3ef3q+7/lMJZ7gcoA+VuFT9LhLKhqTzP6vFI=; b=IbNtRwQVF+8ggGHZCjcd1ertgAwOjKmR1l839WhyK6MwIqdEN7+E5tI/XtDwNmr3Pw xKUbkQTxuceGOmkFQ0DWs+j3W6uIMj51c+dLZVy6r30CPdwy8+pB1sRHtQhMGuLTxwA9 gdlbmVPRvRIxpp5UKJYqAreZmjxmv5xKpyJ2CMmYoKn4IEoA6Xa6IIhY528udml6+Cnn o3MnuOeJCp2g0S+OnwmYmQimTbHsvl0mRB6ngmvcNgFyFLX2xr7yGETYBzIb3WT2vC37 6tzyPB70MEypBJubMSnkdLvwPkIl1FRRzPR1rz+0OjaFMY500Um8JpN4WAHfdcVYkxjt KEDg== X-Gm-Message-State: AG10YORXkX1muu4on92ujyZXG4VDYD9SZCfOwce+MKFlgh0u0dek5Ix+12EMkIru0BVWDg== X-Received: by 10.112.130.41 with SMTP id ob9mr10202582lbb.81.1456153364137; Mon, 22 Feb 2016 07:02:44 -0800 (PST) Received: from vostro.util.wtbts.net ([2001:1bc8:101:f402:21a:9fff:fe0c:4022]) by smtp.gmail.com with ESMTPSA id a14sm3443581lfe.7.2016.02.22.07.02.43 (version=TLSv1/SSLv3 cipher=OTHER); Mon, 22 Feb 2016 07:02:43 -0800 (PST) Sender: =?UTF-8?Q?Timo_Ter=C3=A4s?= Date: Mon, 22 Feb 2016 17:02:41 +0200 From: Timo Teras To: Nathan Johnson Cc: alpine-aports@lists.alpinelinux.org, Michael Nikitochkin Subject: Re: [alpine-aports] [PATCH] community/go: update to go 1.6 Message-ID: <20160222170241.77a421ae@vostro.util.wtbts.net> In-Reply-To: <1455921565-314-1-git-send-email-nathan@nathanjohnson.info> References: <1455921565-314-1-git-send-email-nathan@nathanjohnson.info> X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.28; x86_64-alpine-linux-musl) X-Mailinglist: alpine-aports Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Virus-Scanned: ClamAV using ClamSMTP Hi, I applied but reverted it. Seems it does not work properly on x86, but the build fails. Perhaps go now needs pax marking on x86? Or it's something else? Any ideas? I also added Michael to cc as almost similar patch came from him. Perhaps you can rework this together, or coordinate who looks at this? Additionally, is there some difference in the 1.6 and 1.6.0 tarballs? Thanks, Timo On Fri, 19 Feb 2016 22:39:25 +0000 Nathan Johnson wrote: > --- > community/go/APKBUILD | 28 ++++++------- > community/go/default-sc-getpw-r-size-max.patch | 57 > -------------------------- > community/go/no-pic.patch | 2 +- 3 files > changed, 13 insertions(+), 74 deletions(-) delete mode 100644 > community/go/default-sc-getpw-r-size-max.patch >=20 > diff --git a/community/go/APKBUILD b/community/go/APKBUILD > index 7db7178..bdc53d5 100644 > --- a/community/go/APKBUILD > +++ b/community/go/APKBUILD > @@ -1,9 +1,9 @@ > # Contributor: S=C3=B6ren Tempel > # Maintainer: Eivind Uggedal > pkgname=3Dgo > -pkgver=3D1.5.3 > +pkgver=3D1.6 > # This should be the latest commit on the corresponding release > branch -_toolsver=3D"d02228d1857b9f49cd0252788516ff5584266eb6" > +_toolsver=3D"c887be1b2ebd11663d4bf2fbca508c449172339e" > pkgrel=3D0 > pkgdesc=3D"Go programming language compiler" > url=3D"http://www.golang.org/" > @@ -17,8 +17,7 @@ install=3D"" > subpackages=3D"$pkgname-doc $pkgname-tools $pkgname-cross" > source=3D"http://golang.org/dl/go${pkgver}.src.tar.gz > $pkgname-tools-$pkgver.tar.gz::https://github.com/golang/tools/archive/= ${_toolsver}.tar.gz > - no-pic.patch > - default-sc-getpw-r-size-max.patch" > + no-pic.patch" > =20 > # NOTE: building go for x86 with grsec kernel requires: > # sysctl -w kernel.modify_ldt=3D1 > @@ -136,15 +135,12 @@ cross() { > done > } > =20 > -md5sums=3D"80a0eac7ab750b01b3f7096a1d4667b8 go1.5.3.src.tar.gz > -c64b28ba5df8499bcc19adc995da991a go-tools-1.5.3.tar.gz > -b303cb36017c5f1303b1567b31b48de3 no-pic.patch > -12a36502180b9801f8d9e60824563c92 default-sc-getpw-r-size-max.patch" > -sha256sums=3D"754e06dab1c31ab168fc9db9e32596734015ea9e24bc44cae7f237f417= ce4efe > go1.5.3.src.tar.gz > -80db6fc563df50eb924dbf4f96cef287fc6b89cc8b98595c1dac3c80a118f290 > go-tools-1.5.3.tar.gz > -973144c68d10cdc740501ad814841912bae7c3936480fabb3f7fe092ba82ba7f > no-pic.patch > -70f8f9b3ad6b8639db2b401ae14747a6e8d762443ca054c358a5aa4815df7d1f > default-sc-getpw-r-size-max.patch" > -sha512sums=3D"b348832ea7cb095cf793960cf9f4da5e9f6ca8de7b5262bb819de97efe= a4835643e30a18efd4a82d01ea3854a1905af9e134271716c4b68c9d2a8f6e15f65ff0 > go1.5.3.src.tar.gz > -80813c2da74c7b25fa2cfd15b2505757e7f466469ac701cb22d190dc83a607f091c4dea2= 07a2e371e53d7a1f5459e7b53e5eea53c95565e958a6eb7fcb212359 > go-tools-1.5.3.tar.gz > -e21084dc07a74c7b6437ae22034b62e3a255692d32dba767c15efdb123d3159c03a02437= a41350e512eaa2478a7afbd8b409317a779c56ee4939512d060d57fa > no-pic.patch > -d59c977341b9ab15664b4b0c4f3df9e74ce48ffbd59594a7abb5a0907469f9d362c4e558= 606dd4b7cd102c165dd53027bae6aa1c48623c2105461dcad4cee14d > default-sc-getpw-r-size-max.patch" > +md5sums=3D"e67833ea37fbc002fbe38efe6c1bcd98 go1.6.src.tar.gz > +733a96b59562ed84ff552542f16a3ab3 go-tools-1.6.tar.gz > +efa8ac40fa8429f7cd6f23eca05a0a0f no-pic.patch" > +sha256sums=3D"a96cce8ce43a9bf9b2a4c7d470bc7ee0cb00410da815980681c8353218= dcf146 > go1.6.src.tar.gz > +955e5a119babad356d9cf00cdaaf3c27648d4451109b467c872fad8cc94d5b56 > go-tools-1.6.tar.gz > +1eb60ded9035e9692e4c17b2bb6e9c073e21261eaafa56ee0528d5d2c43a936d > no-pic.patch" > +sha512sums=3D"59e9d72a80558fd5e3f176e068897a45333b36e35f6c00393647941a70= e741168e65941b6059397378020c3b78ec3471a48809682f7efd97cf33eec6325fc3e8 > go1.6.src.tar.gz > +955def7cc3e5ceb5d8e47477400007d32c25f3bc2764f04a0451bd0235af507e63a0966d= 201a345f51ac764471da8620b5357d1099ec39e2422bd04d25fd71b8 > go-tools-1.6.tar.gz > +51e4c3ee0cda98090db5d00468072bcbf5f0a1a429dd8dedddccaa1467b285f724b012be= 96d329566b195470bea0aaf4b464e8ba6d2029b0dd42678a7a14b663 > no-pic.patch" diff --git > a/community/go/default-sc-getpw-r-size-max.patch > b/community/go/default-sc-getpw-r-size-max.patch deleted file mode > 100644 index 77be27d..0000000 --- > a/community/go/default-sc-getpw-r-size-max.patch +++ /dev/null @@ > -1,57 +0,0 @@ -From cb867d2fd64adc851f82be3c6eb6e38ec008930b Mon Sep > 17 00:00:00 2001 -From: Dominik Honnef -Date: > Sun, 21 Jun 2015 20:07:29 +0200 -Subject: [PATCH] os/user: don't > depend on _SC_GETPW_R_SIZE_MAX on Linux - -Even Linux systems may not > have _SC_GETPW_R_SIZE_MAX if using a -different libc than glibc (e.g. > musl). Instead of having special-cases -for the BSDs, handle -1 > correctly by always using a default buffer size. - -Fixes #11319. - > -Change-Id: I8b1b260eb9830e6dbe7667f3f33d115ae4de4ce8 -Reviewed-on: > https://go-review.googlesource.com/13772 -Reviewed-by: Brad > Fitzpatrick -Run-TryBot: Brad Fitzpatrick > ---- > - src/os/user/lookup_unix.go | 19 ++++++++----------- > - 1 file changed, 8 insertions(+), 11 deletions(-) > - > -diff --git a/src/os/user/lookup_unix.go b/src/os/user/lookup_unix.go > -index f4f603e..e8a1eb1 100644 > ---- a/src/os/user/lookup_unix.go > -+++ b/src/os/user/lookup_unix.go > -@@ -9,7 +9,6 @@ package user > -=20 > - import ( > - "fmt" > -- "runtime" > - "strconv" > - "strings" > - "syscall" > -@@ -55,17 +54,15 @@ func lookupUnix(uid int, username string, > lookupByName bool) (*User, error) { > - var pwd C.struct_passwd > - var result *C.struct_passwd > -=20 > -- var bufSize C.long > -- if runtime.GOOS =3D=3D "dragonfly" || runtime.GOOS =3D=3D > "freebsd" { -- // DragonFly and FreeBSD do not have > _SC_GETPW_R_SIZE_MAX -- // and just return -1. So > just use the same -- // size that Linux returns. > -+ bufSize :=3D C.sysconf(C._SC_GETPW_R_SIZE_MAX) > -+ if bufSize =3D=3D -1 { > -+ // DragonFly and FreeBSD do not have > _SC_GETPW_R_SIZE_MAX. -+ // Additionally, not all > Linux systems have it, either. For -+ // example, the > musl libc returns -1. > - bufSize =3D 1024 > -- } else { > -- bufSize =3D C.sysconf(C._SC_GETPW_R_SIZE_MAX) > -- if bufSize <=3D 0 || bufSize > 1<<20 { > -- return nil, fmt.Errorf("user: unreasonable > _SC_GETPW_R_SIZE_MAX of %d", bufSize) -- } > -+ } > -+ if bufSize <=3D 0 || bufSize > 1<<20 { > -+ return nil, fmt.Errorf("user: unreasonable > _SC_GETPW_R_SIZE_MAX of %d", bufSize) > - } > - buf :=3D C.malloc(C.size_t(bufSize)) > - defer C.free(buf) > diff --git a/community/go/no-pic.patch b/community/go/no-pic.patch > index 5a4e396..db1f809 100644 > --- a/community/go/no-pic.patch > +++ b/community/go/no-pic.patch > @@ -2,7 +2,7 @@ diff --git a/src/cmd/link/internal/ld/lib.go > b/src/cmd/link/internal/ld/lib.go index 8ccbec9dd634..4e96bfadc260 > 100644 --- a/src/cmd/link/internal/ld/lib.go > +++ b/src/cmd/link/internal/ld/lib.go > -@@ -1071,6 +1071,11 @@ func hostlink() { > +@@ -1194,6 +1194,11 @@ func hostlink() { > argv =3D append(argv, peimporteddlls()...) > } > =20 --- Unsubscribe: alpine-aports+unsubscribe@lists.alpinelinux.org Help: alpine-aports+help@lists.alpinelinux.org ---