* [PR PATCH] go: update to 1.19, add tests
@ 2022-08-06 9:44 subnut
2022-08-06 9:44 ` subnut
` (7 more replies)
0 siblings, 8 replies; 12+ messages in thread
From: subnut @ 2022-08-06 9:44 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1239 bytes --]
There is a new pull request by subnut against master on the void-packages repository
https://github.com/subnut/void-packages go1.19
https://github.com/void-linux/void-packages/pull/38498
go: update to 1.19, add tests
<!-- Uncomment relevant sections and delete options which are not applicable -->
#### Testing the changes
- I tested the changes in this PR: **briefly**
<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->
<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
#### Local build testing
- I built this PR locally for my native architecture, (x86_64-glibc)
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
- aarch64-musl
- armv7l
- armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/38498.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-go1.19-38498.patch --]
[-- Type: text/x-diff, Size: 1266 bytes --]
From 5ba4d1caeae5e157b72d9459511e227c0f3252cc Mon Sep 17 00:00:00 2001
From: Subhaditya Nath <sn03.general@gmail.com>
Date: Sat, 6 Aug 2022 15:12:02 +0530
Subject: [PATCH] go: update to 1.19, add tests
---
srcpkgs/go/template | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/go/template b/srcpkgs/go/template
index 2db64b795ed8..dba229f32fa7 100644
--- a/srcpkgs/go/template
+++ b/srcpkgs/go/template
@@ -1,17 +1,18 @@
# Template file for 'go'
pkgname=go
-version=1.18.3
+version=1.19
revision=1
create_wrksrc=yes
build_wrksrc=go
hostmakedepends="go1.12-bootstrap"
+checkdepends="clang gcc-fortran"
short_desc="Go Programming Language"
maintainer="Michael Aldridge <maldridge@voidlinux.org>"
license="BSD-3-Clause"
homepage="https://go.dev/"
changelog="https://go.dev/doc/devel/release.html"
distfiles="https://go.dev/dl/go${version}.src.tar.gz"
-checksum=0012386ddcbb5f3350e407c679923811dbd283fcdc421724931614a842ecbc2d
+checksum=9419cc70dc5a2523f29a77053cafff658ed21ef3561d9b6b020280ebceab28b9
nostrip=yes
noverifyrdeps=yes
@@ -47,6 +48,10 @@ do_build() {
bash make.bash -v
}
+do_check() {
+ ./bin/go tool dist test -v -run '!^go_test:(os|cmd/link/internal/ld|net)$'
+}
+
do_install() {
local bindir
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
@ 2022-08-06 9:44 ` subnut
2022-08-06 9:49 ` paper42
` (6 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: subnut @ 2022-08-06 9:44 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 170 bytes --]
New comment by subnut on void-packages repository
https://github.com/void-linux/void-packages/pull/38498#issuecomment-1207184409
Comment:
Please review @the-maldridge
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
2022-08-06 9:44 ` subnut
@ 2022-08-06 9:49 ` paper42
2022-08-06 11:04 ` [PR PATCH] [Updated] " subnut
` (5 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: paper42 @ 2022-08-06 9:49 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 161 bytes --]
New comment by paper42 on void-packages repository
https://github.com/void-linux/void-packages/pull/38498#issuecomment-1207184952
Comment:
duplicate of #38474
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PR PATCH] [Updated] go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
2022-08-06 9:44 ` subnut
2022-08-06 9:49 ` paper42
@ 2022-08-06 11:04 ` subnut
2022-08-06 11:11 ` subnut
` (4 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: subnut @ 2022-08-06 11:04 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1244 bytes --]
There is an updated pull request by subnut against master on the void-packages repository
https://github.com/subnut/void-packages go1.19
https://github.com/void-linux/void-packages/pull/38498
go: update to 1.19, add tests
<!-- Uncomment relevant sections and delete options which are not applicable -->
#### Testing the changes
- I tested the changes in this PR: **briefly**
<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->
<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
#### Local build testing
- I built this PR locally for my native architecture, (x86_64-glibc)
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
- aarch64-musl
- armv7l
- armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/38498.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-go1.19-38498.patch --]
[-- Type: text/x-diff, Size: 3503 bytes --]
From eed49a318f5d7f3b88e57fa63532d4fc5ab29596 Mon Sep 17 00:00:00 2001
From: Subhaditya Nath <sn03.general@gmail.com>
Date: Sat, 6 Aug 2022 15:12:02 +0530
Subject: [PATCH] go: update to 1.19, add tests
A few tests need to be skipped, but otherwise tests run perfectly fine.
---
srcpkgs/go/files/fix-musl-tests.patch | 13 +++++++++++++
srcpkgs/go/template | 21 +++++++++++++++++++--
2 files changed, 32 insertions(+), 2 deletions(-)
create mode 100644 srcpkgs/go/files/fix-musl-tests.patch
diff --git a/srcpkgs/go/files/fix-musl-tests.patch b/srcpkgs/go/files/fix-musl-tests.patch
new file mode 100644
index 000000000000..7b35c5b0dcd3
--- /dev/null
+++ b/srcpkgs/go/files/fix-musl-tests.patch
@@ -0,0 +1,13 @@
+--- a/src/syscall/syscall_linux_test.go
++++ b/src/syscall/syscall_linux_test.go
+@@ -530,10 +530,6 @@ func TestSetuidEtc(t *testing.T) {
+ {call: "Setgid(1)", fn: func() error { return syscall.Setgid(1) }, filter: "Gid:", expect: "\t1\t1\t1\t1"},
+ {call: "Setgid(0)", fn: func() error { return syscall.Setgid(0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
+
+- {call: "Setgroups([]int{0,1,2,3})", fn: func() error { return syscall.Setgroups([]int{0, 1, 2, 3}) }, filter: "Groups:", expect: "\t0 1 2 3"},
+- {call: "Setgroups(nil)", fn: func() error { return syscall.Setgroups(nil) }, filter: "Groups:", expect: ""},
+- {call: "Setgroups([]int{0})", fn: func() error { return syscall.Setgroups([]int{0}) }, filter: "Groups:", expect: "\t0"},
+-
+ {call: "Setregid(101,0)", fn: func() error { return syscall.Setregid(101, 0) }, filter: "Gid:", expect: "\t101\t0\t0\t0"},
+ {call: "Setregid(0,102)", fn: func() error { return syscall.Setregid(0, 102) }, filter: "Gid:", expect: "\t0\t102\t102\t102"},
+ {call: "Setregid(0,0)", fn: func() error { return syscall.Setregid(0, 0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
diff --git a/srcpkgs/go/template b/srcpkgs/go/template
index 2db64b795ed8..65b36190089d 100644
--- a/srcpkgs/go/template
+++ b/srcpkgs/go/template
@@ -1,17 +1,18 @@
# Template file for 'go'
pkgname=go
-version=1.18.3
+version=1.19
revision=1
create_wrksrc=yes
build_wrksrc=go
hostmakedepends="go1.12-bootstrap"
+checkdepends="clang gcc-fortran"
short_desc="Go Programming Language"
maintainer="Michael Aldridge <maldridge@voidlinux.org>"
license="BSD-3-Clause"
homepage="https://go.dev/"
changelog="https://go.dev/doc/devel/release.html"
distfiles="https://go.dev/dl/go${version}.src.tar.gz"
-checksum=0012386ddcbb5f3350e407c679923811dbd283fcdc421724931614a842ecbc2d
+checksum=9419cc70dc5a2523f29a77053cafff658ed21ef3561d9b6b020280ebceab28b9
nostrip=yes
noverifyrdeps=yes
@@ -31,12 +32,24 @@ if [ "$CROSS_BUILD" ]; then
fi
fi
+post_extract() {
+ if [ "$XBPS_LIBC" = musl ]; then
+ patch "$patch_args" <"${FILESDIR}/fix-musl-tests.patch"
+ fi
+}
+
do_build() {
unset GCC CC CXX LD CFLAGS
# FIXME: work around leaking go build-style vars when built as a
# dependency
unset CGO_CXXFLAGS CGO_CFLAGS CGO_ENABLED
+ # Workaround for go-1.19 compilation error on i686
+ # runtime/cgo(.text): unknown symbol __stack_chk_fail_local in pcrel
+ if [ "$_goarch" = 386 ]; then
+ export CGO_CFLAGS="-fno-stack-protector"
+ fi
+
export GOROOT_BOOTSTRAP="/usr/lib/go1.12"
export GOROOT=$PWD
export GOROOT_FINAL="/usr/lib/go"
@@ -47,6 +60,10 @@ do_build() {
bash make.bash -v
}
+do_check() {
+ ./bin/go tool dist test -v -run '!^go_test:(os|cmd/link/internal/ld|net)$'
+}
+
do_install() {
local bindir
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PR PATCH] [Updated] go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
` (2 preceding siblings ...)
2022-08-06 11:04 ` [PR PATCH] [Updated] " subnut
@ 2022-08-06 11:11 ` subnut
2022-08-06 11:18 ` subnut
` (3 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: subnut @ 2022-08-06 11:11 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1244 bytes --]
There is an updated pull request by subnut against master on the void-packages repository
https://github.com/subnut/void-packages go1.19
https://github.com/void-linux/void-packages/pull/38498
go: update to 1.19, add tests
<!-- Uncomment relevant sections and delete options which are not applicable -->
#### Testing the changes
- I tested the changes in this PR: **briefly**
<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->
<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
#### Local build testing
- I built this PR locally for my native architecture, (x86_64-glibc)
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
- aarch64-musl
- armv7l
- armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/38498.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-go1.19-38498.patch --]
[-- Type: text/x-diff, Size: 3509 bytes --]
From 0e6f563e732299d28a43a6d365267a83f4a3a4d1 Mon Sep 17 00:00:00 2001
From: Subhaditya Nath <sn03.general@gmail.com>
Date: Sat, 6 Aug 2022 15:12:02 +0530
Subject: [PATCH] go: update to 1.19, add tests
A few tests need to be skipped, but otherwise tests run perfectly fine.
---
srcpkgs/go/files/fix-musl-tests.patch | 13 +++++++++++++
srcpkgs/go/template | 21 +++++++++++++++++++--
2 files changed, 32 insertions(+), 2 deletions(-)
create mode 100644 srcpkgs/go/files/fix-musl-tests.patch
diff --git a/srcpkgs/go/files/fix-musl-tests.patch b/srcpkgs/go/files/fix-musl-tests.patch
new file mode 100644
index 000000000000..6f19ba9a15e2
--- /dev/null
+++ b/srcpkgs/go/files/fix-musl-tests.patch
@@ -0,0 +1,13 @@
+--- a/go/src/syscall/syscall_linux_test.go
++++ b/go/src/syscall/syscall_linux_test.go
+@@ -530,10 +530,6 @@ func TestSetuidEtc(t *testing.T) {
+ {call: "Setgid(1)", fn: func() error { return syscall.Setgid(1) }, filter: "Gid:", expect: "\t1\t1\t1\t1"},
+ {call: "Setgid(0)", fn: func() error { return syscall.Setgid(0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
+
+- {call: "Setgroups([]int{0,1,2,3})", fn: func() error { return syscall.Setgroups([]int{0, 1, 2, 3}) }, filter: "Groups:", expect: "\t0 1 2 3"},
+- {call: "Setgroups(nil)", fn: func() error { return syscall.Setgroups(nil) }, filter: "Groups:", expect: ""},
+- {call: "Setgroups([]int{0})", fn: func() error { return syscall.Setgroups([]int{0}) }, filter: "Groups:", expect: "\t0"},
+-
+ {call: "Setregid(101,0)", fn: func() error { return syscall.Setregid(101, 0) }, filter: "Gid:", expect: "\t101\t0\t0\t0"},
+ {call: "Setregid(0,102)", fn: func() error { return syscall.Setregid(0, 102) }, filter: "Gid:", expect: "\t0\t102\t102\t102"},
+ {call: "Setregid(0,0)", fn: func() error { return syscall.Setregid(0, 0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
diff --git a/srcpkgs/go/template b/srcpkgs/go/template
index 2db64b795ed8..65b36190089d 100644
--- a/srcpkgs/go/template
+++ b/srcpkgs/go/template
@@ -1,17 +1,18 @@
# Template file for 'go'
pkgname=go
-version=1.18.3
+version=1.19
revision=1
create_wrksrc=yes
build_wrksrc=go
hostmakedepends="go1.12-bootstrap"
+checkdepends="clang gcc-fortran"
short_desc="Go Programming Language"
maintainer="Michael Aldridge <maldridge@voidlinux.org>"
license="BSD-3-Clause"
homepage="https://go.dev/"
changelog="https://go.dev/doc/devel/release.html"
distfiles="https://go.dev/dl/go${version}.src.tar.gz"
-checksum=0012386ddcbb5f3350e407c679923811dbd283fcdc421724931614a842ecbc2d
+checksum=9419cc70dc5a2523f29a77053cafff658ed21ef3561d9b6b020280ebceab28b9
nostrip=yes
noverifyrdeps=yes
@@ -31,12 +32,24 @@ if [ "$CROSS_BUILD" ]; then
fi
fi
+post_extract() {
+ if [ "$XBPS_LIBC" = musl ]; then
+ patch "$patch_args" <"${FILESDIR}/fix-musl-tests.patch"
+ fi
+}
+
do_build() {
unset GCC CC CXX LD CFLAGS
# FIXME: work around leaking go build-style vars when built as a
# dependency
unset CGO_CXXFLAGS CGO_CFLAGS CGO_ENABLED
+ # Workaround for go-1.19 compilation error on i686
+ # runtime/cgo(.text): unknown symbol __stack_chk_fail_local in pcrel
+ if [ "$_goarch" = 386 ]; then
+ export CGO_CFLAGS="-fno-stack-protector"
+ fi
+
export GOROOT_BOOTSTRAP="/usr/lib/go1.12"
export GOROOT=$PWD
export GOROOT_FINAL="/usr/lib/go"
@@ -47,6 +60,10 @@ do_build() {
bash make.bash -v
}
+do_check() {
+ ./bin/go tool dist test -v -run '!^go_test:(os|cmd/link/internal/ld|net)$'
+}
+
do_install() {
local bindir
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PR PATCH] [Updated] go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
` (3 preceding siblings ...)
2022-08-06 11:11 ` subnut
@ 2022-08-06 11:18 ` subnut
2022-08-06 11:20 ` subnut
` (2 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: subnut @ 2022-08-06 11:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1244 bytes --]
There is an updated pull request by subnut against master on the void-packages repository
https://github.com/subnut/void-packages go1.19
https://github.com/void-linux/void-packages/pull/38498
go: update to 1.19, add tests
<!-- Uncomment relevant sections and delete options which are not applicable -->
#### Testing the changes
- I tested the changes in this PR: **briefly**
<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->
<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
#### Local build testing
- I built this PR locally for my native architecture, (x86_64-glibc)
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
- aarch64-musl
- armv7l
- armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/38498.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-go1.19-38498.patch --]
[-- Type: text/x-diff, Size: 3662 bytes --]
From 39b379298c8d3bee285053d866ba365474bd1956 Mon Sep 17 00:00:00 2001
From: Subhaditya Nath <sn03.general@gmail.com>
Date: Sat, 6 Aug 2022 15:12:02 +0530
Subject: [PATCH] go: update to 1.19, add tests
A few tests need to be skipped, but otherwise tests run perfectly fine.
---
srcpkgs/go/patches/fix-musl-tests.patch | 16 ++++++++++++++++
srcpkgs/go/template | 21 +++++++++++++++++++--
2 files changed, 35 insertions(+), 2 deletions(-)
create mode 100644 srcpkgs/go/patches/fix-musl-tests.patch
diff --git a/srcpkgs/go/patches/fix-musl-tests.patch b/srcpkgs/go/patches/fix-musl-tests.patch
new file mode 100644
index 000000000000..0ae3a48aeb83
--- /dev/null
+++ b/srcpkgs/go/patches/fix-musl-tests.patch
@@ -0,0 +1,16 @@
+These tests fail on musl because /proc/../status doesn't contain the "Groups"
+field, because musl-libc lacks the getgroups() syscall.
+
+--- a/go/src/syscall/syscall_linux_test.go
++++ b/go/src/syscall/syscall_linux_test.go
+@@ -530,10 +530,6 @@ func TestSetuidEtc(t *testing.T) {
+ {call: "Setgid(1)", fn: func() error { return syscall.Setgid(1) }, filter: "Gid:", expect: "\t1\t1\t1\t1"},
+ {call: "Setgid(0)", fn: func() error { return syscall.Setgid(0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
+
+- {call: "Setgroups([]int{0,1,2,3})", fn: func() error { return syscall.Setgroups([]int{0, 1, 2, 3}) }, filter: "Groups:", expect: "\t0 1 2 3"},
+- {call: "Setgroups(nil)", fn: func() error { return syscall.Setgroups(nil) }, filter: "Groups:", expect: ""},
+- {call: "Setgroups([]int{0})", fn: func() error { return syscall.Setgroups([]int{0}) }, filter: "Groups:", expect: "\t0"},
+-
+ {call: "Setregid(101,0)", fn: func() error { return syscall.Setregid(101, 0) }, filter: "Gid:", expect: "\t101\t0\t0\t0"},
+ {call: "Setregid(0,102)", fn: func() error { return syscall.Setregid(0, 102) }, filter: "Gid:", expect: "\t0\t102\t102\t102"},
+ {call: "Setregid(0,0)", fn: func() error { return syscall.Setregid(0, 0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
diff --git a/srcpkgs/go/template b/srcpkgs/go/template
index 2db64b795ed8..65b36190089d 100644
--- a/srcpkgs/go/template
+++ b/srcpkgs/go/template
@@ -1,17 +1,18 @@
# Template file for 'go'
pkgname=go
-version=1.18.3
+version=1.19
revision=1
create_wrksrc=yes
build_wrksrc=go
hostmakedepends="go1.12-bootstrap"
+checkdepends="clang gcc-fortran"
short_desc="Go Programming Language"
maintainer="Michael Aldridge <maldridge@voidlinux.org>"
license="BSD-3-Clause"
homepage="https://go.dev/"
changelog="https://go.dev/doc/devel/release.html"
distfiles="https://go.dev/dl/go${version}.src.tar.gz"
-checksum=0012386ddcbb5f3350e407c679923811dbd283fcdc421724931614a842ecbc2d
+checksum=9419cc70dc5a2523f29a77053cafff658ed21ef3561d9b6b020280ebceab28b9
nostrip=yes
noverifyrdeps=yes
@@ -31,12 +32,24 @@ if [ "$CROSS_BUILD" ]; then
fi
fi
+post_extract() {
+ if [ "$XBPS_LIBC" = musl ]; then
+ patch "$patch_args" <"${FILESDIR}/fix-musl-tests.patch"
+ fi
+}
+
do_build() {
unset GCC CC CXX LD CFLAGS
# FIXME: work around leaking go build-style vars when built as a
# dependency
unset CGO_CXXFLAGS CGO_CFLAGS CGO_ENABLED
+ # Workaround for go-1.19 compilation error on i686
+ # runtime/cgo(.text): unknown symbol __stack_chk_fail_local in pcrel
+ if [ "$_goarch" = 386 ]; then
+ export CGO_CFLAGS="-fno-stack-protector"
+ fi
+
export GOROOT_BOOTSTRAP="/usr/lib/go1.12"
export GOROOT=$PWD
export GOROOT_FINAL="/usr/lib/go"
@@ -47,6 +60,10 @@ do_build() {
bash make.bash -v
}
+do_check() {
+ ./bin/go tool dist test -v -run '!^go_test:(os|cmd/link/internal/ld|net)$'
+}
+
do_install() {
local bindir
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PR PATCH] [Updated] go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
` (4 preceding siblings ...)
2022-08-06 11:18 ` subnut
@ 2022-08-06 11:20 ` subnut
2022-08-06 14:57 ` [PR PATCH] [Closed]: " the-maldridge
2022-08-06 21:39 ` 0x5c
7 siblings, 0 replies; 12+ messages in thread
From: subnut @ 2022-08-06 11:20 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1244 bytes --]
There is an updated pull request by subnut against master on the void-packages repository
https://github.com/subnut/void-packages go1.19
https://github.com/void-linux/void-packages/pull/38498
go: update to 1.19, add tests
<!-- Uncomment relevant sections and delete options which are not applicable -->
#### Testing the changes
- I tested the changes in this PR: **briefly**
<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->
<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
#### Local build testing
- I built this PR locally for my native architecture, (x86_64-glibc)
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
- aarch64-musl
- armv7l
- armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/38498.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-go1.19-38498.patch --]
[-- Type: text/x-diff, Size: 3397 bytes --]
From 3a5ae16d3436a44ee66e8dff1eb18fc1d132fc26 Mon Sep 17 00:00:00 2001
From: Subhaditya Nath <sn03.general@gmail.com>
Date: Sat, 6 Aug 2022 15:12:02 +0530
Subject: [PATCH] go: update to 1.19, add tests
A few tests need to be skipped, but otherwise tests run perfectly fine.
---
srcpkgs/go/patches/fix-musl-tests.patch | 16 ++++++++++++++++
srcpkgs/go/template | 15 +++++++++++++--
2 files changed, 29 insertions(+), 2 deletions(-)
create mode 100644 srcpkgs/go/patches/fix-musl-tests.patch
diff --git a/srcpkgs/go/patches/fix-musl-tests.patch b/srcpkgs/go/patches/fix-musl-tests.patch
new file mode 100644
index 000000000000..0ae3a48aeb83
--- /dev/null
+++ b/srcpkgs/go/patches/fix-musl-tests.patch
@@ -0,0 +1,16 @@
+These tests fail on musl because /proc/../status doesn't contain the "Groups"
+field, because musl-libc lacks the getgroups() syscall.
+
+--- a/go/src/syscall/syscall_linux_test.go
++++ b/go/src/syscall/syscall_linux_test.go
+@@ -530,10 +530,6 @@ func TestSetuidEtc(t *testing.T) {
+ {call: "Setgid(1)", fn: func() error { return syscall.Setgid(1) }, filter: "Gid:", expect: "\t1\t1\t1\t1"},
+ {call: "Setgid(0)", fn: func() error { return syscall.Setgid(0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
+
+- {call: "Setgroups([]int{0,1,2,3})", fn: func() error { return syscall.Setgroups([]int{0, 1, 2, 3}) }, filter: "Groups:", expect: "\t0 1 2 3"},
+- {call: "Setgroups(nil)", fn: func() error { return syscall.Setgroups(nil) }, filter: "Groups:", expect: ""},
+- {call: "Setgroups([]int{0})", fn: func() error { return syscall.Setgroups([]int{0}) }, filter: "Groups:", expect: "\t0"},
+-
+ {call: "Setregid(101,0)", fn: func() error { return syscall.Setregid(101, 0) }, filter: "Gid:", expect: "\t101\t0\t0\t0"},
+ {call: "Setregid(0,102)", fn: func() error { return syscall.Setregid(0, 102) }, filter: "Gid:", expect: "\t0\t102\t102\t102"},
+ {call: "Setregid(0,0)", fn: func() error { return syscall.Setregid(0, 0) }, filter: "Gid:", expect: "\t0\t0\t0\t0"},
diff --git a/srcpkgs/go/template b/srcpkgs/go/template
index 2db64b795ed8..0a356b840413 100644
--- a/srcpkgs/go/template
+++ b/srcpkgs/go/template
@@ -1,17 +1,18 @@
# Template file for 'go'
pkgname=go
-version=1.18.3
+version=1.19
revision=1
create_wrksrc=yes
build_wrksrc=go
hostmakedepends="go1.12-bootstrap"
+checkdepends="clang gcc-fortran"
short_desc="Go Programming Language"
maintainer="Michael Aldridge <maldridge@voidlinux.org>"
license="BSD-3-Clause"
homepage="https://go.dev/"
changelog="https://go.dev/doc/devel/release.html"
distfiles="https://go.dev/dl/go${version}.src.tar.gz"
-checksum=0012386ddcbb5f3350e407c679923811dbd283fcdc421724931614a842ecbc2d
+checksum=9419cc70dc5a2523f29a77053cafff658ed21ef3561d9b6b020280ebceab28b9
nostrip=yes
noverifyrdeps=yes
@@ -37,6 +38,12 @@ do_build() {
# dependency
unset CGO_CXXFLAGS CGO_CFLAGS CGO_ENABLED
+ # Workaround for go-1.19 compilation error on i686
+ # runtime/cgo(.text): unknown symbol __stack_chk_fail_local in pcrel
+ if [ "$_goarch" = 386 ]; then
+ export CGO_CFLAGS="-fno-stack-protector"
+ fi
+
export GOROOT_BOOTSTRAP="/usr/lib/go1.12"
export GOROOT=$PWD
export GOROOT_FINAL="/usr/lib/go"
@@ -47,6 +54,10 @@ do_build() {
bash make.bash -v
}
+do_check() {
+ ./bin/go tool dist test -v -run '!^go_test:(os|cmd/link/internal/ld|net)$'
+}
+
do_install() {
local bindir
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PR PATCH] [Closed]: go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
` (5 preceding siblings ...)
2022-08-06 11:20 ` subnut
@ 2022-08-06 14:57 ` the-maldridge
2022-08-06 21:39 ` 0x5c
7 siblings, 0 replies; 12+ messages in thread
From: the-maldridge @ 2022-08-06 14:57 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1091 bytes --]
There's a closed pull request on the void-packages repository
go: update to 1.19, add tests
https://github.com/void-linux/void-packages/pull/38498
Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->
#### Testing the changes
- I tested the changes in this PR: **briefly**
<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->
<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
#### Local build testing
- I built this PR locally for my native architecture, (x86_64-glibc)
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
- aarch64-musl
- armv7l
- armv6l-musl
-->
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: go: update to 1.19, add tests
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
` (6 preceding siblings ...)
2022-08-06 14:57 ` [PR PATCH] [Closed]: " the-maldridge
@ 2022-08-06 21:39 ` 0x5c
7 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-08-06 21:39 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 246 bytes --]
New comment by 0x5c on void-packages repository
https://github.com/void-linux/void-packages/pull/38498#issuecomment-1207286572
Comment:
hum, this appears to be a superset of the changes in https://github.com/void-linux/void-packages/pull/38474
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: go: update to 1.19, add tests
2022-08-07 5:03 [PR PATCH] " subnut
2022-08-07 20:49 ` paper42
2022-08-08 1:14 ` the-maldridge
@ 2022-08-08 2:36 ` subnut
2 siblings, 0 replies; 12+ messages in thread
From: subnut @ 2022-08-08 2:36 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 161 bytes --]
New comment by subnut on void-packages repository
https://github.com/void-linux/void-packages/pull/38510#issuecomment-1207588263
Comment:
@the-maldridge Done.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: go: update to 1.19, add tests
2022-08-07 5:03 [PR PATCH] " subnut
2022-08-07 20:49 ` paper42
@ 2022-08-08 1:14 ` the-maldridge
2022-08-08 2:36 ` subnut
2 siblings, 0 replies; 12+ messages in thread
From: the-maldridge @ 2022-08-08 1:14 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 262 bytes --]
New comment by the-maldridge on void-packages repository
https://github.com/void-linux/void-packages/pull/38510#issuecomment-1207541726
Comment:
@subnut this looks fine with the exception of the tests. Please apply the requested changes, then good for merge.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: go: update to 1.19, add tests
2022-08-07 5:03 [PR PATCH] " subnut
@ 2022-08-07 20:49 ` paper42
2022-08-08 1:14 ` the-maldridge
2022-08-08 2:36 ` subnut
2 siblings, 0 replies; 12+ messages in thread
From: paper42 @ 2022-08-07 20:49 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 198 bytes --]
New comment by paper42 on void-packages repository
https://github.com/void-linux/void-packages/pull/38510#issuecomment-1207483087
Comment:
this PR seems to be in the best shape, cc @the-maldridge
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2022-08-08 2:36 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-06 9:44 [PR PATCH] go: update to 1.19, add tests subnut
2022-08-06 9:44 ` subnut
2022-08-06 9:49 ` paper42
2022-08-06 11:04 ` [PR PATCH] [Updated] " subnut
2022-08-06 11:11 ` subnut
2022-08-06 11:18 ` subnut
2022-08-06 11:20 ` subnut
2022-08-06 14:57 ` [PR PATCH] [Closed]: " the-maldridge
2022-08-06 21:39 ` 0x5c
2022-08-07 5:03 [PR PATCH] " subnut
2022-08-07 20:49 ` paper42
2022-08-08 1:14 ` the-maldridge
2022-08-08 2:36 ` subnut
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).