* Re: [PR REVIEW] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
@ 2021-07-17 4:00 ` ericonr
2021-07-17 4:00 ` ericonr
` (14 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-07-17 4:00 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 286 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#discussion_r671604568
Comment:
It's uglier, but caddy and tor are using `MAX_OPEN_FILES` instead of `NOFILES`... I think it'd be neat to keep consistency across the repo.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
2021-07-17 4:00 ` [PR REVIEW] " ericonr
@ 2021-07-17 4:00 ` ericonr
2021-07-17 4:00 ` ericonr
` (13 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-07-17 4:00 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 201 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#discussion_r671604635
Comment:
Does it make sense to build with openssl *and* gnutls?
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
2021-07-17 4:00 ` [PR REVIEW] " ericonr
2021-07-17 4:00 ` ericonr
@ 2021-07-17 4:00 ` ericonr
2021-07-17 4:11 ` [PR PATCH] [Updated] " jailbird777
` (12 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-07-17 4:00 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 188 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#issuecomment-881813334
Comment:
Nice work, I think this is close to mergeable !
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Updated] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (2 preceding siblings ...)
2021-07-17 4:00 ` ericonr
@ 2021-07-17 4:11 ` jailbird777
2021-07-17 4:21 ` jailbird777
` (11 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-17 4:11 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1580 bytes --]
There is an updated pull request by jailbird777 against master on the void-packages repository
https://github.com/jailbird777/void-packages dnsdist
https://github.com/void-linux/void-packages/pull/31932
New package: dnsdist-1.6.0
<!-- Mark items with [x] where applicable -->
#### General
- [X] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [X] I built this PR locally for my native architecture, (x86_64-musl)
- [X] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [X] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
This is my second attempt (first one was #28109 ). Sorry for being such a newb, this is my first attempt at xbps packaging. The original PR wanted to use the upstream .conf instead of my sample one, so I went with the upstream one this time. The reason I didn't use it originally was because it's entirely commented out, so it doesn't work out of the box. I wanted a better user experience by default.
A patch file from https://github.com/void-linux/void-packages/pull/31932.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-dnsdist-31932.patch --]
[-- Type: text/x-diff, Size: 2603 bytes --]
From ff80aa30b656ba004059c0baff421a240f7a8323 Mon Sep 17 00:00:00 2001
From: jailbird777 <jailbird@fdf.net>
Date: Mon, 12 Jul 2021 18:42:03 -0500
Subject: [PATCH] New package: dnsdist-1.6.0
---
srcpkgs/dnsdist/files/dnsdist/run | 4 ++++
srcpkgs/dnsdist/template | 37 +++++++++++++++++++++++++++++++
srcpkgs/dnsdist/update | 1 +
3 files changed, 42 insertions(+)
create mode 100644 srcpkgs/dnsdist/files/dnsdist/run
create mode 100644 srcpkgs/dnsdist/template
create mode 100644 srcpkgs/dnsdist/update
diff --git a/srcpkgs/dnsdist/files/dnsdist/run b/srcpkgs/dnsdist/files/dnsdist/run
new file mode 100644
index 000000000000..0a0c14508312
--- /dev/null
+++ b/srcpkgs/dnsdist/files/dnsdist/run
@@ -0,0 +1,4 @@
+#!/bin/sh
+[ -f ./conf ] && . ./conf
+NOFILES=${NOFILES:-2048}
+exec chpst -o $NOFILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
diff --git a/srcpkgs/dnsdist/template b/srcpkgs/dnsdist/template
new file mode 100644
index 000000000000..5a3ccbbc8aa4
--- /dev/null
+++ b/srcpkgs/dnsdist/template
@@ -0,0 +1,37 @@
+# Template file for 'dnsdist'
+pkgname=dnsdist
+version=1.6.0
+revision=1
+build_style=gnu-configure
+configure_args="--with-pic --with-gnu-ld --with-libsodium --with-re2
+ --with-net-snmp --with-libcap --with-libssl --enable-dnscrypt
+ --with-libcrypto=${XBPS_CROSS_BASE}/usr --with-service-user=_dnsdist
+ --with-service-group=_dnsdist --sbindir=/usr/bin --with-gnutls --with-lmdb
+ --enable-dns-over-tls --enable-dns-over-https --sysconfdir=/etc/dnsdist"
+conf_files="/etc/dnsdist/dnsdist.conf"
+hostmakedepends="autoconf automake pkgconf"
+makedepends="libsodium-devel boost-devel re2-devel net-snmp-devel libcap-devel
+ lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel gnutls-devel
+ h2o-devel"
+short_desc="Dynamic DNS loadbalancer"
+maintainer="JailBird <jailbird@fdf.net>"
+license="GPL-2.0-only"
+homepage="https://dnsdist.org/"
+distfiles="https://downloads.powerdns.com/releases/${pkgname}-${version}.tar.bz2"
+checksum=a7783a04d8d4ad2b0168ffaaf85ef95d5f557057b0462280684dd799d0cdd292
+
+system_accounts="_dnsdist"
+
+if [ "$XBPS_WORDSIZE" = 32 ]; then
+ # This can be removed once MUSL is updated to 1.2.x
+ broken="no y2038 support"
+fi
+
+pre_configure() {
+ export LMDB_LIBS="-llmdb"
+}
+
+post_install() {
+ vinstall dnsdistconf.lua 644 etc/dnsdist dnsdist.conf
+ vsv dnsdist
+}
diff --git a/srcpkgs/dnsdist/update b/srcpkgs/dnsdist/update
new file mode 100644
index 000000000000..da8e072d218c
--- /dev/null
+++ b/srcpkgs/dnsdist/update
@@ -0,0 +1 @@
+ignore="*alpha[0-9]* *rc[0-9]* *beta[0-9]*"
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Updated] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (3 preceding siblings ...)
2021-07-17 4:11 ` [PR PATCH] [Updated] " jailbird777
@ 2021-07-17 4:21 ` jailbird777
2021-07-17 4:34 ` jailbird777
` (10 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-17 4:21 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1580 bytes --]
There is an updated pull request by jailbird777 against master on the void-packages repository
https://github.com/jailbird777/void-packages dnsdist
https://github.com/void-linux/void-packages/pull/31932
New package: dnsdist-1.6.0
<!-- Mark items with [x] where applicable -->
#### General
- [X] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [X] I built this PR locally for my native architecture, (x86_64-musl)
- [X] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [X] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
This is my second attempt (first one was #28109 ). Sorry for being such a newb, this is my first attempt at xbps packaging. The original PR wanted to use the upstream .conf instead of my sample one, so I went with the upstream one this time. The reason I didn't use it originally was because it's entirely commented out, so it doesn't work out of the box. I wanted a better user experience by default.
A patch file from https://github.com/void-linux/void-packages/pull/31932.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-dnsdist-31932.patch --]
[-- Type: text/x-diff, Size: 2603 bytes --]
From ff80aa30b656ba004059c0baff421a240f7a8323 Mon Sep 17 00:00:00 2001
From: jailbird777 <jailbird@fdf.net>
Date: Mon, 12 Jul 2021 18:42:03 -0500
Subject: [PATCH] New package: dnsdist-1.6.0
---
srcpkgs/dnsdist/files/dnsdist/run | 4 ++++
srcpkgs/dnsdist/template | 37 +++++++++++++++++++++++++++++++
srcpkgs/dnsdist/update | 1 +
3 files changed, 42 insertions(+)
create mode 100644 srcpkgs/dnsdist/files/dnsdist/run
create mode 100644 srcpkgs/dnsdist/template
create mode 100644 srcpkgs/dnsdist/update
diff --git a/srcpkgs/dnsdist/files/dnsdist/run b/srcpkgs/dnsdist/files/dnsdist/run
new file mode 100644
index 000000000000..0a0c14508312
--- /dev/null
+++ b/srcpkgs/dnsdist/files/dnsdist/run
@@ -0,0 +1,4 @@
+#!/bin/sh
+[ -f ./conf ] && . ./conf
+NOFILES=${NOFILES:-2048}
+exec chpst -o $NOFILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
diff --git a/srcpkgs/dnsdist/template b/srcpkgs/dnsdist/template
new file mode 100644
index 000000000000..5a3ccbbc8aa4
--- /dev/null
+++ b/srcpkgs/dnsdist/template
@@ -0,0 +1,37 @@
+# Template file for 'dnsdist'
+pkgname=dnsdist
+version=1.6.0
+revision=1
+build_style=gnu-configure
+configure_args="--with-pic --with-gnu-ld --with-libsodium --with-re2
+ --with-net-snmp --with-libcap --with-libssl --enable-dnscrypt
+ --with-libcrypto=${XBPS_CROSS_BASE}/usr --with-service-user=_dnsdist
+ --with-service-group=_dnsdist --sbindir=/usr/bin --with-gnutls --with-lmdb
+ --enable-dns-over-tls --enable-dns-over-https --sysconfdir=/etc/dnsdist"
+conf_files="/etc/dnsdist/dnsdist.conf"
+hostmakedepends="autoconf automake pkgconf"
+makedepends="libsodium-devel boost-devel re2-devel net-snmp-devel libcap-devel
+ lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel gnutls-devel
+ h2o-devel"
+short_desc="Dynamic DNS loadbalancer"
+maintainer="JailBird <jailbird@fdf.net>"
+license="GPL-2.0-only"
+homepage="https://dnsdist.org/"
+distfiles="https://downloads.powerdns.com/releases/${pkgname}-${version}.tar.bz2"
+checksum=a7783a04d8d4ad2b0168ffaaf85ef95d5f557057b0462280684dd799d0cdd292
+
+system_accounts="_dnsdist"
+
+if [ "$XBPS_WORDSIZE" = 32 ]; then
+ # This can be removed once MUSL is updated to 1.2.x
+ broken="no y2038 support"
+fi
+
+pre_configure() {
+ export LMDB_LIBS="-llmdb"
+}
+
+post_install() {
+ vinstall dnsdistconf.lua 644 etc/dnsdist dnsdist.conf
+ vsv dnsdist
+}
diff --git a/srcpkgs/dnsdist/update b/srcpkgs/dnsdist/update
new file mode 100644
index 000000000000..da8e072d218c
--- /dev/null
+++ b/srcpkgs/dnsdist/update
@@ -0,0 +1 @@
+ignore="*alpha[0-9]* *rc[0-9]* *beta[0-9]*"
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Updated] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (4 preceding siblings ...)
2021-07-17 4:21 ` jailbird777
@ 2021-07-17 4:34 ` jailbird777
2021-07-17 4:34 ` [PR REVIEW] " jailbird777
` (9 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-17 4:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1580 bytes --]
There is an updated pull request by jailbird777 against master on the void-packages repository
https://github.com/jailbird777/void-packages dnsdist
https://github.com/void-linux/void-packages/pull/31932
New package: dnsdist-1.6.0
<!-- Mark items with [x] where applicable -->
#### General
- [X] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [X] I built this PR locally for my native architecture, (x86_64-musl)
- [X] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [X] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
This is my second attempt (first one was #28109 ). Sorry for being such a newb, this is my first attempt at xbps packaging. The original PR wanted to use the upstream .conf instead of my sample one, so I went with the upstream one this time. The reason I didn't use it originally was because it's entirely commented out, so it doesn't work out of the box. I wanted a better user experience by default.
A patch file from https://github.com/void-linux/void-packages/pull/31932.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-dnsdist-31932.patch --]
[-- Type: text/x-diff, Size: 4546 bytes --]
From ff80aa30b656ba004059c0baff421a240f7a8323 Mon Sep 17 00:00:00 2001
From: jailbird777 <jailbird@fdf.net>
Date: Mon, 12 Jul 2021 18:42:03 -0500
Subject: [PATCH 1/2] New package: dnsdist-1.6.0
---
srcpkgs/dnsdist/files/dnsdist/run | 4 ++++
srcpkgs/dnsdist/template | 37 +++++++++++++++++++++++++++++++
srcpkgs/dnsdist/update | 1 +
3 files changed, 42 insertions(+)
create mode 100644 srcpkgs/dnsdist/files/dnsdist/run
create mode 100644 srcpkgs/dnsdist/template
create mode 100644 srcpkgs/dnsdist/update
diff --git a/srcpkgs/dnsdist/files/dnsdist/run b/srcpkgs/dnsdist/files/dnsdist/run
new file mode 100644
index 000000000000..0a0c14508312
--- /dev/null
+++ b/srcpkgs/dnsdist/files/dnsdist/run
@@ -0,0 +1,4 @@
+#!/bin/sh
+[ -f ./conf ] && . ./conf
+NOFILES=${NOFILES:-2048}
+exec chpst -o $NOFILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
diff --git a/srcpkgs/dnsdist/template b/srcpkgs/dnsdist/template
new file mode 100644
index 000000000000..5a3ccbbc8aa4
--- /dev/null
+++ b/srcpkgs/dnsdist/template
@@ -0,0 +1,37 @@
+# Template file for 'dnsdist'
+pkgname=dnsdist
+version=1.6.0
+revision=1
+build_style=gnu-configure
+configure_args="--with-pic --with-gnu-ld --with-libsodium --with-re2
+ --with-net-snmp --with-libcap --with-libssl --enable-dnscrypt
+ --with-libcrypto=${XBPS_CROSS_BASE}/usr --with-service-user=_dnsdist
+ --with-service-group=_dnsdist --sbindir=/usr/bin --with-gnutls --with-lmdb
+ --enable-dns-over-tls --enable-dns-over-https --sysconfdir=/etc/dnsdist"
+conf_files="/etc/dnsdist/dnsdist.conf"
+hostmakedepends="autoconf automake pkgconf"
+makedepends="libsodium-devel boost-devel re2-devel net-snmp-devel libcap-devel
+ lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel gnutls-devel
+ h2o-devel"
+short_desc="Dynamic DNS loadbalancer"
+maintainer="JailBird <jailbird@fdf.net>"
+license="GPL-2.0-only"
+homepage="https://dnsdist.org/"
+distfiles="https://downloads.powerdns.com/releases/${pkgname}-${version}.tar.bz2"
+checksum=a7783a04d8d4ad2b0168ffaaf85ef95d5f557057b0462280684dd799d0cdd292
+
+system_accounts="_dnsdist"
+
+if [ "$XBPS_WORDSIZE" = 32 ]; then
+ # This can be removed once MUSL is updated to 1.2.x
+ broken="no y2038 support"
+fi
+
+pre_configure() {
+ export LMDB_LIBS="-llmdb"
+}
+
+post_install() {
+ vinstall dnsdistconf.lua 644 etc/dnsdist dnsdist.conf
+ vsv dnsdist
+}
diff --git a/srcpkgs/dnsdist/update b/srcpkgs/dnsdist/update
new file mode 100644
index 000000000000..da8e072d218c
--- /dev/null
+++ b/srcpkgs/dnsdist/update
@@ -0,0 +1 @@
+ignore="*alpha[0-9]* *rc[0-9]* *beta[0-9]*"
From d2542fe0e0eefda44c23d9d58e8344403da41a5a Mon Sep 17 00:00:00 2001
From: jailbird777 <jailbird@fdf.net>
Date: Fri, 16 Jul 2021 23:27:25 -0500
Subject: [PATCH 2/2] New package: dnsdist-1.6.0
---
srcpkgs/dnsdist/files/dnsdist/run | 4 ++--
srcpkgs/dnsdist/template | 5 ++---
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/srcpkgs/dnsdist/files/dnsdist/run b/srcpkgs/dnsdist/files/dnsdist/run
index 0a0c14508312..130506a53b7d 100644
--- a/srcpkgs/dnsdist/files/dnsdist/run
+++ b/srcpkgs/dnsdist/files/dnsdist/run
@@ -1,4 +1,4 @@
#!/bin/sh
[ -f ./conf ] && . ./conf
-NOFILES=${NOFILES:-2048}
-exec chpst -o $NOFILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
+MAX_OPEN_FILES=${MAX_OPEN_FILES:-2048}
+exec chpst -o $MAX_OPEN_FILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
diff --git a/srcpkgs/dnsdist/template b/srcpkgs/dnsdist/template
index 5a3ccbbc8aa4..e8ab9cd05e29 100644
--- a/srcpkgs/dnsdist/template
+++ b/srcpkgs/dnsdist/template
@@ -6,13 +6,12 @@ build_style=gnu-configure
configure_args="--with-pic --with-gnu-ld --with-libsodium --with-re2
--with-net-snmp --with-libcap --with-libssl --enable-dnscrypt
--with-libcrypto=${XBPS_CROSS_BASE}/usr --with-service-user=_dnsdist
- --with-service-group=_dnsdist --sbindir=/usr/bin --with-gnutls --with-lmdb
+ --with-service-group=_dnsdist --sbindir=/usr/bin --without-gnutls --with-lmdb
--enable-dns-over-tls --enable-dns-over-https --sysconfdir=/etc/dnsdist"
conf_files="/etc/dnsdist/dnsdist.conf"
hostmakedepends="autoconf automake pkgconf"
makedepends="libsodium-devel boost-devel re2-devel net-snmp-devel libcap-devel
- lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel gnutls-devel
- h2o-devel"
+ lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel h2o-devel"
short_desc="Dynamic DNS loadbalancer"
maintainer="JailBird <jailbird@fdf.net>"
license="GPL-2.0-only"
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (5 preceding siblings ...)
2021-07-17 4:34 ` jailbird777
@ 2021-07-17 4:34 ` jailbird777
2021-07-17 4:36 ` jailbird777
` (8 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-17 4:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 156 bytes --]
New review comment by jailbird777 on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#discussion_r671607861
Comment:
Done!
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (6 preceding siblings ...)
2021-07-17 4:34 ` [PR REVIEW] " jailbird777
@ 2021-07-17 4:36 ` jailbird777
2021-07-18 21:09 ` ericonr
` (7 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-17 4:36 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 395 bytes --]
New review comment by jailbird777 on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#discussion_r671608027
Comment:
I never thought so, and personally I dislike GNUtls, but that's how Debian & Ubuntu built dnsdist when I first opened this PR. I just double checked and sure enough, Debian agrees with you. They dropped GNUtls support, so I just did the same.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (7 preceding siblings ...)
2021-07-17 4:36 ` jailbird777
@ 2021-07-18 21:09 ` ericonr
2021-07-27 0:12 ` [PR PATCH] [Updated] " jailbird777
` (6 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-07-18 21:09 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 282 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#issuecomment-882116927
Comment:
If you can, please rebase this PR and squash the commits into a single one so I can merge it more easily once the repos are unlocked again :)
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Updated] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (8 preceding siblings ...)
2021-07-18 21:09 ` ericonr
@ 2021-07-27 0:12 ` jailbird777
2021-07-27 0:13 ` jailbird777
` (5 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-27 0:12 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1580 bytes --]
There is an updated pull request by jailbird777 against master on the void-packages repository
https://github.com/jailbird777/void-packages dnsdist
https://github.com/void-linux/void-packages/pull/31932
New package: dnsdist-1.6.0
<!-- Mark items with [x] where applicable -->
#### General
- [X] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [X] I built this PR locally for my native architecture, (x86_64-musl)
- [X] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [X] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
This is my second attempt (first one was #28109 ). Sorry for being such a newb, this is my first attempt at xbps packaging. The original PR wanted to use the upstream .conf instead of my sample one, so I went with the upstream one this time. The reason I didn't use it originally was because it's entirely commented out, so it doesn't work out of the box. I wanted a better user experience by default.
A patch file from https://github.com/void-linux/void-packages/pull/31932.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-dnsdist-31932.patch --]
[-- Type: text/x-diff, Size: 2611 bytes --]
From 311babf5be9689499a927c010e346e1fc30d9bc6 Mon Sep 17 00:00:00 2001
From: jailbird777 <jailbird@fdf.net>
Date: Mon, 26 Jul 2021 19:11:45 -0500
Subject: [PATCH] New package dnsdist-1.6.0
---
srcpkgs/dnsdist/files/dnsdist/run | 4 ++++
srcpkgs/dnsdist/template | 36 +++++++++++++++++++++++++++++++
srcpkgs/dnsdist/update | 1 +
3 files changed, 41 insertions(+)
create mode 100644 srcpkgs/dnsdist/files/dnsdist/run
create mode 100644 srcpkgs/dnsdist/template
create mode 100644 srcpkgs/dnsdist/update
diff --git a/srcpkgs/dnsdist/files/dnsdist/run b/srcpkgs/dnsdist/files/dnsdist/run
new file mode 100644
index 000000000000..130506a53b7d
--- /dev/null
+++ b/srcpkgs/dnsdist/files/dnsdist/run
@@ -0,0 +1,4 @@
+#!/bin/sh
+[ -f ./conf ] && . ./conf
+MAX_OPEN_FILES=${MAX_OPEN_FILES:-2048}
+exec chpst -o $MAX_OPEN_FILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
diff --git a/srcpkgs/dnsdist/template b/srcpkgs/dnsdist/template
new file mode 100644
index 000000000000..e8ab9cd05e29
--- /dev/null
+++ b/srcpkgs/dnsdist/template
@@ -0,0 +1,36 @@
+# Template file for 'dnsdist'
+pkgname=dnsdist
+version=1.6.0
+revision=1
+build_style=gnu-configure
+configure_args="--with-pic --with-gnu-ld --with-libsodium --with-re2
+ --with-net-snmp --with-libcap --with-libssl --enable-dnscrypt
+ --with-libcrypto=${XBPS_CROSS_BASE}/usr --with-service-user=_dnsdist
+ --with-service-group=_dnsdist --sbindir=/usr/bin --without-gnutls --with-lmdb
+ --enable-dns-over-tls --enable-dns-over-https --sysconfdir=/etc/dnsdist"
+conf_files="/etc/dnsdist/dnsdist.conf"
+hostmakedepends="autoconf automake pkgconf"
+makedepends="libsodium-devel boost-devel re2-devel net-snmp-devel libcap-devel
+ lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel h2o-devel"
+short_desc="Dynamic DNS loadbalancer"
+maintainer="JailBird <jailbird@fdf.net>"
+license="GPL-2.0-only"
+homepage="https://dnsdist.org/"
+distfiles="https://downloads.powerdns.com/releases/${pkgname}-${version}.tar.bz2"
+checksum=a7783a04d8d4ad2b0168ffaaf85ef95d5f557057b0462280684dd799d0cdd292
+
+system_accounts="_dnsdist"
+
+if [ "$XBPS_WORDSIZE" = 32 ]; then
+ # This can be removed once MUSL is updated to 1.2.x
+ broken="no y2038 support"
+fi
+
+pre_configure() {
+ export LMDB_LIBS="-llmdb"
+}
+
+post_install() {
+ vinstall dnsdistconf.lua 644 etc/dnsdist dnsdist.conf
+ vsv dnsdist
+}
diff --git a/srcpkgs/dnsdist/update b/srcpkgs/dnsdist/update
new file mode 100644
index 000000000000..da8e072d218c
--- /dev/null
+++ b/srcpkgs/dnsdist/update
@@ -0,0 +1 @@
+ignore="*alpha[0-9]* *rc[0-9]* *beta[0-9]*"
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (9 preceding siblings ...)
2021-07-27 0:12 ` [PR PATCH] [Updated] " jailbird777
@ 2021-07-27 0:13 ` jailbird777
2021-07-27 3:44 ` [PR PATCH] [Updated] " ericonr
` (4 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-27 0:13 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 161 bytes --]
New comment by jailbird777 on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#issuecomment-887113324
Comment:
Done! Thanks! :)
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Updated] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (10 preceding siblings ...)
2021-07-27 0:13 ` jailbird777
@ 2021-07-27 3:44 ` ericonr
2021-07-27 14:45 ` [PR REVIEW] " ericonr
` (3 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-07-27 3:44 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1576 bytes --]
There is an updated pull request by ericonr against master on the void-packages repository
https://github.com/jailbird777/void-packages dnsdist
https://github.com/void-linux/void-packages/pull/31932
New package: dnsdist-1.6.0
<!-- Mark items with [x] where applicable -->
#### General
- [X] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [X] I built this PR locally for my native architecture, (x86_64-musl)
- [X] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [X] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
This is my second attempt (first one was #28109 ). Sorry for being such a newb, this is my first attempt at xbps packaging. The original PR wanted to use the upstream .conf instead of my sample one, so I went with the upstream one this time. The reason I didn't use it originally was because it's entirely commented out, so it doesn't work out of the box. I wanted a better user experience by default.
A patch file from https://github.com/void-linux/void-packages/pull/31932.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-dnsdist-31932.patch --]
[-- Type: text/x-diff, Size: 2612 bytes --]
From e7b76b060f3a8c6ebfa8de870f246695d66435ff Mon Sep 17 00:00:00 2001
From: jailbird777 <jailbird@fdf.net>
Date: Mon, 26 Jul 2021 19:11:45 -0500
Subject: [PATCH] New package: dnsdist-1.6.0
---
srcpkgs/dnsdist/files/dnsdist/run | 4 ++++
srcpkgs/dnsdist/template | 36 +++++++++++++++++++++++++++++++
srcpkgs/dnsdist/update | 1 +
3 files changed, 41 insertions(+)
create mode 100644 srcpkgs/dnsdist/files/dnsdist/run
create mode 100644 srcpkgs/dnsdist/template
create mode 100644 srcpkgs/dnsdist/update
diff --git a/srcpkgs/dnsdist/files/dnsdist/run b/srcpkgs/dnsdist/files/dnsdist/run
new file mode 100644
index 000000000000..130506a53b7d
--- /dev/null
+++ b/srcpkgs/dnsdist/files/dnsdist/run
@@ -0,0 +1,4 @@
+#!/bin/sh
+[ -f ./conf ] && . ./conf
+MAX_OPEN_FILES=${MAX_OPEN_FILES:-2048}
+exec chpst -o $MAX_OPEN_FILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
diff --git a/srcpkgs/dnsdist/template b/srcpkgs/dnsdist/template
new file mode 100644
index 000000000000..e8ab9cd05e29
--- /dev/null
+++ b/srcpkgs/dnsdist/template
@@ -0,0 +1,36 @@
+# Template file for 'dnsdist'
+pkgname=dnsdist
+version=1.6.0
+revision=1
+build_style=gnu-configure
+configure_args="--with-pic --with-gnu-ld --with-libsodium --with-re2
+ --with-net-snmp --with-libcap --with-libssl --enable-dnscrypt
+ --with-libcrypto=${XBPS_CROSS_BASE}/usr --with-service-user=_dnsdist
+ --with-service-group=_dnsdist --sbindir=/usr/bin --without-gnutls --with-lmdb
+ --enable-dns-over-tls --enable-dns-over-https --sysconfdir=/etc/dnsdist"
+conf_files="/etc/dnsdist/dnsdist.conf"
+hostmakedepends="autoconf automake pkgconf"
+makedepends="libsodium-devel boost-devel re2-devel net-snmp-devel libcap-devel
+ lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel h2o-devel"
+short_desc="Dynamic DNS loadbalancer"
+maintainer="JailBird <jailbird@fdf.net>"
+license="GPL-2.0-only"
+homepage="https://dnsdist.org/"
+distfiles="https://downloads.powerdns.com/releases/${pkgname}-${version}.tar.bz2"
+checksum=a7783a04d8d4ad2b0168ffaaf85ef95d5f557057b0462280684dd799d0cdd292
+
+system_accounts="_dnsdist"
+
+if [ "$XBPS_WORDSIZE" = 32 ]; then
+ # This can be removed once MUSL is updated to 1.2.x
+ broken="no y2038 support"
+fi
+
+pre_configure() {
+ export LMDB_LIBS="-llmdb"
+}
+
+post_install() {
+ vinstall dnsdistconf.lua 644 etc/dnsdist dnsdist.conf
+ vsv dnsdist
+}
diff --git a/srcpkgs/dnsdist/update b/srcpkgs/dnsdist/update
new file mode 100644
index 000000000000..da8e072d218c
--- /dev/null
+++ b/srcpkgs/dnsdist/update
@@ -0,0 +1 @@
+ignore="*alpha[0-9]* *rc[0-9]* *beta[0-9]*"
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (11 preceding siblings ...)
2021-07-27 3:44 ` [PR PATCH] [Updated] " ericonr
@ 2021-07-27 14:45 ` ericonr
2021-07-30 1:18 ` [PR PATCH] [Updated] " jailbird777
` (2 subsequent siblings)
15 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-07-27 14:45 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 189 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#discussion_r677522214
Comment:
Ah, this should be `XBPS_TARGET_WORDSIZE`.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Updated] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (12 preceding siblings ...)
2021-07-27 14:45 ` [PR REVIEW] " ericonr
@ 2021-07-30 1:18 ` jailbird777
2021-07-30 1:18 ` [PR REVIEW] " jailbird777
2021-07-31 2:11 ` [PR PATCH] [Merged]: " ericonr
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-30 1:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1580 bytes --]
There is an updated pull request by jailbird777 against master on the void-packages repository
https://github.com/jailbird777/void-packages dnsdist
https://github.com/void-linux/void-packages/pull/31932
New package: dnsdist-1.6.0
<!-- Mark items with [x] where applicable -->
#### General
- [X] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [X] I built this PR locally for my native architecture, (x86_64-musl)
- [X] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [X] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
This is my second attempt (first one was #28109 ). Sorry for being such a newb, this is my first attempt at xbps packaging. The original PR wanted to use the upstream .conf instead of my sample one, so I went with the upstream one this time. The reason I didn't use it originally was because it's entirely commented out, so it doesn't work out of the box. I wanted a better user experience by default.
A patch file from https://github.com/void-linux/void-packages/pull/31932.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-dnsdist-31932.patch --]
[-- Type: text/x-diff, Size: 2619 bytes --]
From 9f932ef156ef3f677fadf194e53a3a95362c92dd Mon Sep 17 00:00:00 2001
From: jailbird777 <jailbird@fdf.net>
Date: Mon, 26 Jul 2021 19:11:45 -0500
Subject: [PATCH] New package: dnsdist-1.6.0
---
srcpkgs/dnsdist/files/dnsdist/run | 4 ++++
srcpkgs/dnsdist/template | 36 +++++++++++++++++++++++++++++++
srcpkgs/dnsdist/update | 1 +
3 files changed, 41 insertions(+)
create mode 100644 srcpkgs/dnsdist/files/dnsdist/run
create mode 100644 srcpkgs/dnsdist/template
create mode 100644 srcpkgs/dnsdist/update
diff --git a/srcpkgs/dnsdist/files/dnsdist/run b/srcpkgs/dnsdist/files/dnsdist/run
new file mode 100644
index 000000000000..130506a53b7d
--- /dev/null
+++ b/srcpkgs/dnsdist/files/dnsdist/run
@@ -0,0 +1,4 @@
+#!/bin/sh
+[ -f ./conf ] && . ./conf
+MAX_OPEN_FILES=${MAX_OPEN_FILES:-2048}
+exec chpst -o $MAX_OPEN_FILES dnsdist --uid _dnsdist --gid _dnsdist --supervised --disable-syslog
diff --git a/srcpkgs/dnsdist/template b/srcpkgs/dnsdist/template
new file mode 100644
index 000000000000..a75da37f0c49
--- /dev/null
+++ b/srcpkgs/dnsdist/template
@@ -0,0 +1,36 @@
+# Template file for 'dnsdist'
+pkgname=dnsdist
+version=1.6.0
+revision=1
+build_style=gnu-configure
+configure_args="--with-pic --with-gnu-ld --with-libsodium --with-re2
+ --with-net-snmp --with-libcap --with-libssl --enable-dnscrypt
+ --with-libcrypto=${XBPS_CROSS_BASE}/usr --with-service-user=_dnsdist
+ --with-service-group=_dnsdist --sbindir=/usr/bin --without-gnutls --with-lmdb
+ --enable-dns-over-tls --enable-dns-over-https --sysconfdir=/etc/dnsdist"
+conf_files="/etc/dnsdist/dnsdist.conf"
+hostmakedepends="autoconf automake pkgconf"
+makedepends="libsodium-devel boost-devel re2-devel net-snmp-devel libcap-devel
+ lua54-devel openssl-devel lmdb-devel fstrm-devel libedit-devel h2o-devel"
+short_desc="Dynamic DNS loadbalancer"
+maintainer="JailBird <jailbird@fdf.net>"
+license="GPL-2.0-only"
+homepage="https://dnsdist.org/"
+distfiles="https://downloads.powerdns.com/releases/${pkgname}-${version}.tar.bz2"
+checksum=a7783a04d8d4ad2b0168ffaaf85ef95d5f557057b0462280684dd799d0cdd292
+
+system_accounts="_dnsdist"
+
+if [ "$XBPS_TARGET_WORDSIZE" = 32 ]; then
+ # This can be removed once MUSL is updated to 1.2.x
+ broken="no y2038 support"
+fi
+
+pre_configure() {
+ export LMDB_LIBS="-llmdb"
+}
+
+post_install() {
+ vinstall dnsdistconf.lua 644 etc/dnsdist dnsdist.conf
+ vsv dnsdist
+}
diff --git a/srcpkgs/dnsdist/update b/srcpkgs/dnsdist/update
new file mode 100644
index 000000000000..da8e072d218c
--- /dev/null
+++ b/srcpkgs/dnsdist/update
@@ -0,0 +1 @@
+ignore="*alpha[0-9]* *rc[0-9]* *beta[0-9]*"
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (13 preceding siblings ...)
2021-07-30 1:18 ` [PR PATCH] [Updated] " jailbird777
@ 2021-07-30 1:18 ` jailbird777
2021-07-31 2:11 ` [PR PATCH] [Merged]: " ericonr
15 siblings, 0 replies; 19+ messages in thread
From: jailbird777 @ 2021-07-30 1:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 185 bytes --]
New review comment by jailbird777 on void-packages repository
https://github.com/void-linux/void-packages/pull/31932#discussion_r679582984
Comment:
AH-HA. For cross-compiling. Fixed!
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Merged]: New package: dnsdist-1.6.0
2021-07-13 0:27 [PR PATCH] New package: dnsdist-1.6.0 jailbird777
` (14 preceding siblings ...)
2021-07-30 1:18 ` [PR REVIEW] " jailbird777
@ 2021-07-31 2:11 ` ericonr
15 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-07-31 2:11 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1416 bytes --]
There's a merged pull request on the void-packages repository
New package: dnsdist-1.6.0
https://github.com/void-linux/void-packages/pull/31932
Description:
<!-- Mark items with [x] where applicable -->
#### General
- [X] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [X] I built this PR locally for my native architecture, (x86_64-musl)
- [X] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [X] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
This is my second attempt (first one was #28109 ). Sorry for being such a newb, this is my first attempt at xbps packaging. The original PR wanted to use the upstream .conf instead of my sample one, so I went with the upstream one this time. The reason I didn't use it originally was because it's entirely commented out, so it doesn't work out of the box. I wanted a better user experience by default.
^ permalink raw reply [flat|nested] 19+ messages in thread