Github messages for voidlinux
 help / color / mirror / Atom feed
* [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).