Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] delve: update to 1.8.0.
@ 2022-01-01 16:37 TeddyDD
  2022-01-01 17:00 ` [PR REVIEW] " paper42
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: TeddyDD @ 2022-01-01 16:37 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1215 bytes --]

There is a new pull request by TeddyDD against master on the void-packages repository

https://github.com/TeddyDD/void-packages-1 update-delve
https://github.com/void-linux/void-packages/pull/34799

delve: update to 1.8.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### 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 [skip CI](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, (ARCH-LIBC)
- 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/34799.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-delve-34799.patch --]
[-- Type: text/x-diff, Size: 1033 bytes --]

From 3d193b3a2a2adfc9356f87ae098fb5ac32221760 Mon Sep 17 00:00:00 2001
From: Daniel Lewan <vision360.daniel@gmail.com>
Date: Sat, 1 Jan 2022 17:36:43 +0100
Subject: [PATCH] delve: update to 1.8.0.

---
 srcpkgs/delve/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/delve/template b/srcpkgs/delve/template
index 43e09ec913c0..893a95ecfc75 100644
--- a/srcpkgs/delve/template
+++ b/srcpkgs/delve/template
@@ -1,6 +1,6 @@
 # Template file for 'delve'
 pkgname=delve
-version=1.7.1
+version=1.8.0
 revision=1
 build_style=go
 go_import_path=github.com/go-delve/delve/cmd/dlv
@@ -9,7 +9,7 @@ maintainer="Renato Aguiar <renato@renatoaguiar.net>"
 license="MIT"
 homepage="https://github.com/go-delve/delve"
 distfiles="https://github.com/go-delve/delve/archive/v${version}.tar.gz"
-checksum=0734b86d48291749ba2537fa6c03e4c3451628f630055c877789e0346db8d67d
+checksum=086106a4776fa155bf20c37d27b9caed55255be6359c7f0bda1893d3adbb614e
 
 case "$XBPS_TARGET_MACHINE" in
 	i686*|arm*|ppc*) broken=yes ;;

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PR REVIEW] delve: update to 1.8.0.
  2022-01-01 16:37 [PR PATCH] delve: update to 1.8.0 TeddyDD
  2022-01-01 17:00 ` [PR REVIEW] " paper42
@ 2022-01-01 17:00 ` paper42
  2022-01-01 17:01 ` [PR PATCH] [Updated] " TeddyDD
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: paper42 @ 2022-01-01 17:00 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 427 bytes --]

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/34799#discussion_r777122810

Comment:
According to https://github.com/go-delve/delve/blob/master/pkg/proc/native/support_sentinel.go, only amd64, aarch64 and i686 are supported, so I think we should just restrict archs instead of marking this as broken - `archs="x86_64* i686* aarch64*"` and a comment with the source.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PR REVIEW] delve: update to 1.8.0.
  2022-01-01 16:37 [PR PATCH] delve: update to 1.8.0 TeddyDD
@ 2022-01-01 17:00 ` paper42
  2022-01-01 17:00 ` paper42
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: paper42 @ 2022-01-01 17:00 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 367 bytes --]

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/34799#discussion_r777122815

Comment:
```suggestion
homepage="https://github.com/go-delve/delve"
changelog="https://raw.githubusercontent.com/go-delve/delve/master/CHANGELOG.md"
distfiles="https://github.com/go-delve/delve/archive/v${version}.tar.gz"
```

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PR PATCH] [Updated] delve: update to 1.8.0.
  2022-01-01 16:37 [PR PATCH] delve: update to 1.8.0 TeddyDD
  2022-01-01 17:00 ` [PR REVIEW] " paper42
  2022-01-01 17:00 ` paper42
@ 2022-01-01 17:01 ` TeddyDD
  2022-01-01 17:01 ` [PR REVIEW] " TeddyDD
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: TeddyDD @ 2022-01-01 17:01 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1220 bytes --]

There is an updated pull request by TeddyDD against master on the void-packages repository

https://github.com/TeddyDD/void-packages-1 update-delve
https://github.com/void-linux/void-packages/pull/34799

delve: update to 1.8.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### 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 [skip CI](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, (ARCH-LIBC)
- 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/34799.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-delve-34799.patch --]
[-- Type: text/x-diff, Size: 1171 bytes --]

From 7fd752c15990e7a1f094f3e0abcc0afc25c938bb Mon Sep 17 00:00:00 2001
From: Daniel Lewan <vision360.daniel@gmail.com>
Date: Sat, 1 Jan 2022 17:36:43 +0100
Subject: [PATCH] delve: update to 1.8.0.

---
 srcpkgs/delve/template | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/delve/template b/srcpkgs/delve/template
index 43e09ec913c0..98294e58d266 100644
--- a/srcpkgs/delve/template
+++ b/srcpkgs/delve/template
@@ -1,6 +1,6 @@
 # Template file for 'delve'
 pkgname=delve
-version=1.7.1
+version=1.8.0
 revision=1
 build_style=go
 go_import_path=github.com/go-delve/delve/cmd/dlv
@@ -8,8 +8,9 @@ short_desc="Debugger for the Go programming language"
 maintainer="Renato Aguiar <renato@renatoaguiar.net>"
 license="MIT"
 homepage="https://github.com/go-delve/delve"
+changelog="https://raw.githubusercontent.com/go-delve/delve/master/CHANGELOG.md"
 distfiles="https://github.com/go-delve/delve/archive/v${version}.tar.gz"
-checksum=0734b86d48291749ba2537fa6c03e4c3451628f630055c877789e0346db8d67d
+checksum=086106a4776fa155bf20c37d27b9caed55255be6359c7f0bda1893d3adbb614e
 
 case "$XBPS_TARGET_MACHINE" in
 	i686*|arm*|ppc*) broken=yes ;;

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PR REVIEW] delve: update to 1.8.0.
  2022-01-01 16:37 [PR PATCH] delve: update to 1.8.0 TeddyDD
                   ` (2 preceding siblings ...)
  2022-01-01 17:01 ` [PR PATCH] [Updated] " TeddyDD
@ 2022-01-01 17:01 ` TeddyDD
  2022-01-01 17:07 ` [PR PATCH] [Updated] " TeddyDD
  2022-01-01 17:15 ` [PR PATCH] [Merged]: " paper42
  5 siblings, 0 replies; 7+ messages in thread
From: TeddyDD @ 2022-01-01 17:01 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 153 bytes --]

New review comment by TeddyDD on void-packages repository

https://github.com/void-linux/void-packages/pull/34799#discussion_r777122958

Comment:
Thanks

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PR PATCH] [Updated] delve: update to 1.8.0.
  2022-01-01 16:37 [PR PATCH] delve: update to 1.8.0 TeddyDD
                   ` (3 preceding siblings ...)
  2022-01-01 17:01 ` [PR REVIEW] " TeddyDD
@ 2022-01-01 17:07 ` TeddyDD
  2022-01-01 17:15 ` [PR PATCH] [Merged]: " paper42
  5 siblings, 0 replies; 7+ messages in thread
From: TeddyDD @ 2022-01-01 17:07 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1220 bytes --]

There is an updated pull request by TeddyDD against master on the void-packages repository

https://github.com/TeddyDD/void-packages-1 update-delve
https://github.com/void-linux/void-packages/pull/34799

delve: update to 1.8.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### 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 [skip CI](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, (ARCH-LIBC)
- 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/34799.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-delve-34799.patch --]
[-- Type: text/x-diff, Size: 1327 bytes --]

From 6cdcb6fdd086ed4254ecea9970c9af20dd008c3f Mon Sep 17 00:00:00 2001
From: Daniel Lewan <vision360.daniel@gmail.com>
Date: Sat, 1 Jan 2022 17:36:43 +0100
Subject: [PATCH] delve: update to 1.8.0.

---
 srcpkgs/delve/template | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/delve/template b/srcpkgs/delve/template
index 43e09ec913c0..d5f8945fc880 100644
--- a/srcpkgs/delve/template
+++ b/srcpkgs/delve/template
@@ -1,19 +1,18 @@
 # Template file for 'delve'
 pkgname=delve
-version=1.7.1
+version=1.8.0
 revision=1
+# https://github.com/go-delve/delve/blob/master/pkg/proc/native/support_sentinel.go
+archs="x86_64* i686* aarch64*"
 build_style=go
 go_import_path=github.com/go-delve/delve/cmd/dlv
 short_desc="Debugger for the Go programming language"
 maintainer="Renato Aguiar <renato@renatoaguiar.net>"
 license="MIT"
 homepage="https://github.com/go-delve/delve"
+changelog="https://raw.githubusercontent.com/go-delve/delve/master/CHANGELOG.md"
 distfiles="https://github.com/go-delve/delve/archive/v${version}.tar.gz"
-checksum=0734b86d48291749ba2537fa6c03e4c3451628f630055c877789e0346db8d67d
-
-case "$XBPS_TARGET_MACHINE" in
-	i686*|arm*|ppc*) broken=yes ;;
-esac
+checksum=086106a4776fa155bf20c37d27b9caed55255be6359c7f0bda1893d3adbb614e
 
 post_install() {
 	vlicense LICENSE

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PR PATCH] [Merged]: delve: update to 1.8.0.
  2022-01-01 16:37 [PR PATCH] delve: update to 1.8.0 TeddyDD
                   ` (4 preceding siblings ...)
  2022-01-01 17:07 ` [PR PATCH] [Updated] " TeddyDD
@ 2022-01-01 17:15 ` paper42
  5 siblings, 0 replies; 7+ messages in thread
From: paper42 @ 2022-01-01 17:15 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1057 bytes --]

There's a merged pull request on the void-packages repository

delve: update to 1.8.0.
https://github.com/void-linux/void-packages/pull/34799

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### 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 [skip CI](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, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-01-01 17:15 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-01 16:37 [PR PATCH] delve: update to 1.8.0 TeddyDD
2022-01-01 17:00 ` [PR REVIEW] " paper42
2022-01-01 17:00 ` paper42
2022-01-01 17:01 ` [PR PATCH] [Updated] " TeddyDD
2022-01-01 17:01 ` [PR REVIEW] " TeddyDD
2022-01-01 17:07 ` [PR PATCH] [Updated] " TeddyDD
2022-01-01 17:15 ` [PR PATCH] [Merged]: " paper42

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).