Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] signify: update to 31.
@ 2022-08-10 19:55 mtboehlke
  2022-08-10 20:20 ` [PR PATCH] [Updated] " mtboehlke
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: mtboehlke @ 2022-08-10 19:55 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mtboehlke/void-packages signify
https://github.com/void-linux/void-packages/pull/38590

signify: update to 31.
Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file

<!-- 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, (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/38590.patch is attached

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

From 54de71e15538bef8df52010261817358844904c3 Mon Sep 17 00:00:00 2001
From: Mat Boehlke <mtboehlke@gmail.com>
Date: Wed, 10 Aug 2022 14:41:18 -0500
Subject: [PATCH] signify: update to 31.

Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file
---
 srcpkgs/signify/template | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/signify/template b/srcpkgs/signify/template
index 5b3c6ce62911..5a6600364e17 100644
--- a/srcpkgs/signify/template
+++ b/srcpkgs/signify/template
@@ -1,27 +1,29 @@
 # Template file for 'signify'
 pkgname=signify
 reverts="20141230_3 20141230_2 20141230_1"
-version=30
+version=31
 revision=1
 build_style=gnu-makefile
 make_build_args="BZERO=bundled"
 hostmakedepends="pkg-config"
-makedepends="libbsd-devel"
+makedepends="libbsd-devel libmd-devel"
+checkdepends="tar"
 short_desc="OpenBSD cryptographic signing and verification tool"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="ISC"
 homepage="http://www.tedunangst.com/flak/post/signify"
 distfiles="https://github.com/aperezdc/signify/releases/download/v${version}/signify-${version}.tar.xz"
-checksum=f68406c3085ef902e85500e6c0b90e4c3f56347e5efffc0da7b6fb47803c8686
+checksum=1155fd9eeed4a8aa20476b2333d251953ec5d52338d943a770db5b78dd8d2b74
 
 pre_build() {
 	# Allow building with musl
 		cp ${FILESDIR}/void.h .
 		for file in blf.h sha2.h ohash.h; do
-			sed -e '1 i\#include "void.h"' -i $file
+			vsed -e '1 i\#include "void.h"' -i $file
 		done
+
 	# Create license file
-		head -n 16 signify.c > LICENSE
+		sed -E '3,15 !d;s/^ [*] ?//' signify.c > LICENSE
 }
 
 post_install() {

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

* Re: [PR PATCH] [Updated] signify: update to 31.
  2022-08-10 19:55 [PR PATCH] signify: update to 31 mtboehlke
@ 2022-08-10 20:20 ` mtboehlke
  2022-08-10 21:02 ` mtboehlke
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: mtboehlke @ 2022-08-10 20:20 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mtboehlke/void-packages signify
https://github.com/void-linux/void-packages/pull/38590

signify: update to 31.
Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file

<!-- 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 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, (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/38590.patch is attached

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

From 1dcae0a08211d361c106fbe99d3e9f11363391b8 Mon Sep 17 00:00:00 2001
From: Mat Boehlke <mtboehlke@gmail.com>
Date: Wed, 10 Aug 2022 14:41:18 -0500
Subject: [PATCH] signify: update to 31.

Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file
---
 srcpkgs/signify/template | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/signify/template b/srcpkgs/signify/template
index 5b3c6ce62911..443588dc010d 100644
--- a/srcpkgs/signify/template
+++ b/srcpkgs/signify/template
@@ -1,27 +1,29 @@
 # Template file for 'signify'
 pkgname=signify
 reverts="20141230_3 20141230_2 20141230_1"
-version=30
+version=31
 revision=1
 build_style=gnu-makefile
 make_build_args="BZERO=bundled"
 hostmakedepends="pkg-config"
-makedepends="libbsd-devel"
+makedepends="libbsd-devel libmd-devel"
+checkdepends="tar"
 short_desc="OpenBSD cryptographic signing and verification tool"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="ISC"
-homepage="http://www.tedunangst.com/flak/post/signify"
+homepage="https://www.tedunangst.com/flak/post/signify"
 distfiles="https://github.com/aperezdc/signify/releases/download/v${version}/signify-${version}.tar.xz"
-checksum=f68406c3085ef902e85500e6c0b90e4c3f56347e5efffc0da7b6fb47803c8686
+checksum=1155fd9eeed4a8aa20476b2333d251953ec5d52338d943a770db5b78dd8d2b74
 
 pre_build() {
 	# Allow building with musl
 		cp ${FILESDIR}/void.h .
 		for file in blf.h sha2.h ohash.h; do
-			sed -e '1 i\#include "void.h"' -i $file
+			vsed -e '1 i\#include "void.h"' -i $file
 		done
+
 	# Create license file
-		head -n 16 signify.c > LICENSE
+		sed -E '3,15 !d;s/^ [*] ?//' signify.c > LICENSE
 }
 
 post_install() {

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

* Re: [PR PATCH] [Updated] signify: update to 31.
  2022-08-10 19:55 [PR PATCH] signify: update to 31 mtboehlke
  2022-08-10 20:20 ` [PR PATCH] [Updated] " mtboehlke
@ 2022-08-10 21:02 ` mtboehlke
  2022-08-10 23:01 ` [PR REVIEW] " classabbyamp
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: mtboehlke @ 2022-08-10 21:02 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mtboehlke/void-packages signify
https://github.com/void-linux/void-packages/pull/38590

signify: update to 31.
Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file

<!-- 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 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, (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/38590.patch is attached

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

From b8d7d6acba0b183b169c59f06e7448b7ec6d58d3 Mon Sep 17 00:00:00 2001
From: Mat Boehlke <mtboehlke@gmail.com>
Date: Wed, 10 Aug 2022 14:41:18 -0500
Subject: [PATCH] signify: update to 31.

Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file

Also add changelog
---
 srcpkgs/signify/template | 15 +++++++++------
 srcpkgs/signify/update   |  1 -
 2 files changed, 9 insertions(+), 7 deletions(-)
 delete mode 100644 srcpkgs/signify/update

diff --git a/srcpkgs/signify/template b/srcpkgs/signify/template
index 5b3c6ce62911..512fc66f35ac 100644
--- a/srcpkgs/signify/template
+++ b/srcpkgs/signify/template
@@ -1,27 +1,30 @@
 # Template file for 'signify'
 pkgname=signify
 reverts="20141230_3 20141230_2 20141230_1"
-version=30
+version=31
 revision=1
 build_style=gnu-makefile
 make_build_args="BZERO=bundled"
 hostmakedepends="pkg-config"
-makedepends="libbsd-devel"
+makedepends="libbsd-devel libmd-devel"
+checkdepends="tar"
 short_desc="OpenBSD cryptographic signing and verification tool"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="ISC"
-homepage="http://www.tedunangst.com/flak/post/signify"
+homepage="https://www.tedunangst.com/flak/post/signify"
+changelog="https://raw.githubusercontent.com/aperezdc/signify/master/CHANGELOG.md"
 distfiles="https://github.com/aperezdc/signify/releases/download/v${version}/signify-${version}.tar.xz"
-checksum=f68406c3085ef902e85500e6c0b90e4c3f56347e5efffc0da7b6fb47803c8686
+checksum=1155fd9eeed4a8aa20476b2333d251953ec5d52338d943a770db5b78dd8d2b74
 
 pre_build() {
 	# Allow building with musl
 		cp ${FILESDIR}/void.h .
 		for file in blf.h sha2.h ohash.h; do
-			sed -e '1 i\#include "void.h"' -i $file
+			vsed -e '1 i\#include "void.h"' -i $file
 		done
+
 	# Create license file
-		head -n 16 signify.c > LICENSE
+		sed -E '3,15 !d;s/^ [*] ?//' signify.c > LICENSE
 }
 
 post_install() {
diff --git a/srcpkgs/signify/update b/srcpkgs/signify/update
deleted file mode 100644
index 97579dd59c04..000000000000
--- a/srcpkgs/signify/update
+++ /dev/null
@@ -1 +0,0 @@
-pkgname="${pkgname}-portable"

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

* Re: [PR REVIEW] signify: update to 31.
  2022-08-10 19:55 [PR PATCH] signify: update to 31 mtboehlke
  2022-08-10 20:20 ` [PR PATCH] [Updated] " mtboehlke
  2022-08-10 21:02 ` mtboehlke
@ 2022-08-10 23:01 ` classabbyamp
  2022-08-11  0:06 ` [PR PATCH] [Updated] " mtboehlke
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: classabbyamp @ 2022-08-10 23:01 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/38590#discussion_r942981116

Comment:
can you remove the extra tab indents here

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

* Re: [PR PATCH] [Updated] signify: update to 31.
  2022-08-10 19:55 [PR PATCH] signify: update to 31 mtboehlke
                   ` (2 preceding siblings ...)
  2022-08-10 23:01 ` [PR REVIEW] " classabbyamp
@ 2022-08-11  0:06 ` mtboehlke
  2022-08-11  0:07 ` [PR REVIEW] " mtboehlke
  2022-08-11  0:33 ` [PR PATCH] [Merged]: " classabbyamp
  5 siblings, 0 replies; 7+ messages in thread
From: mtboehlke @ 2022-08-11  0:06 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mtboehlke/void-packages signify
https://github.com/void-linux/void-packages/pull/38590

signify: update to 31.
Edit: I also added changelog and removed the UPDATE file, which doesn't seem correct anymore.

Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file

<!-- 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 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, (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/38590.patch is attached

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

From c721aa888fd8b8c780b33ce76f28a52eaa5fe3c6 Mon Sep 17 00:00:00 2001
From: Mat Boehlke <mtboehlke@gmail.com>
Date: Wed, 10 Aug 2022 14:41:18 -0500
Subject: [PATCH] signify: update to 31.

Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file

Also add changelog
---
 srcpkgs/signify/template | 21 ++++++++++++---------
 srcpkgs/signify/update   |  1 -
 2 files changed, 12 insertions(+), 10 deletions(-)
 delete mode 100644 srcpkgs/signify/update

diff --git a/srcpkgs/signify/template b/srcpkgs/signify/template
index 5b3c6ce62911..7883f6e23241 100644
--- a/srcpkgs/signify/template
+++ b/srcpkgs/signify/template
@@ -1,27 +1,30 @@
 # Template file for 'signify'
 pkgname=signify
 reverts="20141230_3 20141230_2 20141230_1"
-version=30
+version=31
 revision=1
 build_style=gnu-makefile
 make_build_args="BZERO=bundled"
 hostmakedepends="pkg-config"
-makedepends="libbsd-devel"
+makedepends="libbsd-devel libmd-devel"
+checkdepends="tar"
 short_desc="OpenBSD cryptographic signing and verification tool"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="ISC"
-homepage="http://www.tedunangst.com/flak/post/signify"
+homepage="https://www.tedunangst.com/flak/post/signify"
+changelog="https://raw.githubusercontent.com/aperezdc/signify/master/CHANGELOG.md"
 distfiles="https://github.com/aperezdc/signify/releases/download/v${version}/signify-${version}.tar.xz"
-checksum=f68406c3085ef902e85500e6c0b90e4c3f56347e5efffc0da7b6fb47803c8686
+checksum=1155fd9eeed4a8aa20476b2333d251953ec5d52338d943a770db5b78dd8d2b74
 
 pre_build() {
 	# Allow building with musl
-		cp ${FILESDIR}/void.h .
-		for file in blf.h sha2.h ohash.h; do
-			sed -e '1 i\#include "void.h"' -i $file
-		done
+	cp ${FILESDIR}/void.h .
+	for file in blf.h sha2.h ohash.h; do
+		vsed -e '1 i\#include "void.h"' -i $file
+	done
+
 	# Create license file
-		head -n 16 signify.c > LICENSE
+	sed -E '3,15 !d;s/^ [*] ?//' signify.c > LICENSE
 }
 
 post_install() {
diff --git a/srcpkgs/signify/update b/srcpkgs/signify/update
deleted file mode 100644
index 97579dd59c04..000000000000
--- a/srcpkgs/signify/update
+++ /dev/null
@@ -1 +0,0 @@
-pkgname="${pkgname}-portable"

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

* Re: [PR REVIEW] signify: update to 31.
  2022-08-10 19:55 [PR PATCH] signify: update to 31 mtboehlke
                   ` (3 preceding siblings ...)
  2022-08-11  0:06 ` [PR PATCH] [Updated] " mtboehlke
@ 2022-08-11  0:07 ` mtboehlke
  2022-08-11  0:33 ` [PR PATCH] [Merged]: " classabbyamp
  5 siblings, 0 replies; 7+ messages in thread
From: mtboehlke @ 2022-08-11  0:07 UTC (permalink / raw)
  To: ml

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

New review comment by mtboehlke on void-packages repository

https://github.com/void-linux/void-packages/pull/38590#discussion_r943009475

Comment:
Done

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

* Re: [PR PATCH] [Merged]: signify: update to 31.
  2022-08-10 19:55 [PR PATCH] signify: update to 31 mtboehlke
                   ` (4 preceding siblings ...)
  2022-08-11  0:07 ` [PR REVIEW] " mtboehlke
@ 2022-08-11  0:33 ` classabbyamp
  5 siblings, 0 replies; 7+ messages in thread
From: classabbyamp @ 2022-08-11  0:33 UTC (permalink / raw)
  To: ml

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

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

signify: update to 31.
https://github.com/void-linux/void-packages/pull/38590

Description:
Edit: I also added changelog and removed the UPDATE file, which doesn't seem correct anymore.

Add needed libmd makedepend and tar checkdepend,
use vsed instead of sed for the file patching,
and use sed instead of head to extract a nicer LICENSE file

<!-- 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 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, (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-08-11  0:33 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-10 19:55 [PR PATCH] signify: update to 31 mtboehlke
2022-08-10 20:20 ` [PR PATCH] [Updated] " mtboehlke
2022-08-10 21:02 ` mtboehlke
2022-08-10 23:01 ` [PR REVIEW] " classabbyamp
2022-08-11  0:06 ` [PR PATCH] [Updated] " mtboehlke
2022-08-11  0:07 ` [PR REVIEW] " mtboehlke
2022-08-11  0:33 ` [PR PATCH] [Merged]: " classabbyamp

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