Github messages for voidlinux
 help / color / mirror / Atom feed
From: cinerea0 <cinerea0@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] dnscrypt-proxy: update to 2.1.5, move config file
Date: Thu, 24 Aug 2023 18:49:18 +0200	[thread overview]
Message-ID: <20230824164918.eC1TGZTQUTIjP18CgTjdH5qApXEJaqd5g58Gttql41s@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-45595@inbox.vuxu.org>

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

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

https://github.com/cinerea0/void-packages dnscrypt
https://github.com/void-linux/void-packages/pull/45595

dnscrypt-proxy: update to 2.1.5, move config file
#### Testing the changes
- I tested the changes in this PR: **YES**

---

I'm using this as an opportunity to pick up where #34517 left off. `dnscrypt-proxy` installs all of its supplemental files (that it periodically refreshes from upstream) into the same directory as its main config file. Currently this is `/etc`, which means we get a bunch of ambiguously-named files like "relays.md" polluting that directory. This change would group all those files together, at the expense of users having to move their config files to the new location.

Other changes:
* The files previously being installed as documentation were actually examples, so they are now installed with `vsconf`.
* The `generate-domains-blocklist` python helper script is now installed.
* The log directory is no longer needed after the mass switch to vlogger (#42026), so it has been removed.

A patch file from https://github.com/void-linux/void-packages/pull/45595.patch is attached

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

From b73b4782532b774262b6d6856352d2c55118aa21 Mon Sep 17 00:00:00 2001
From: cinerea0 <cinerea0@disroot.org>
Date: Thu, 24 Aug 2023 12:48:49 -0400
Subject: [PATCH] dnscrypt-proxy: update to 2.1.5, move config file

dnscrypt-proxy pollutes /etc with ambiguous files because it installs
everything into the same directory that contains its main config file.
This moves all those files into /etc/dnscrypt-proxy.
Files previously installed as documentation are now correctly installed
as examples.
---
 srcpkgs/dnscrypt-proxy/INSTALL.msg            |  3 +++
 .../dnscrypt-proxy/files/dnscrypt-proxy/run   |  2 +-
 srcpkgs/dnscrypt-proxy/template               | 21 ++++++++++---------
 3 files changed, 15 insertions(+), 11 deletions(-)
 create mode 100644 srcpkgs/dnscrypt-proxy/INSTALL.msg

diff --git a/srcpkgs/dnscrypt-proxy/INSTALL.msg b/srcpkgs/dnscrypt-proxy/INSTALL.msg
new file mode 100644
index 0000000000000..cce1591d19ff1
--- /dev/null
+++ b/srcpkgs/dnscrypt-proxy/INSTALL.msg
@@ -0,0 +1,3 @@
+The dnscrypt-proxy service now installs and looks for the config file at
+/etc/dnscrypt-proxy/dnscrypt-proxy.toml. Any previous configuration will
+need to be moved into this directory manually.
diff --git a/srcpkgs/dnscrypt-proxy/files/dnscrypt-proxy/run b/srcpkgs/dnscrypt-proxy/files/dnscrypt-proxy/run
index 6e240fb488e29..3f0cd61724353 100755
--- a/srcpkgs/dnscrypt-proxy/files/dnscrypt-proxy/run
+++ b/srcpkgs/dnscrypt-proxy/files/dnscrypt-proxy/run
@@ -1,3 +1,3 @@
 #!/bin/sh
 exec 2>&1
-exec dnscrypt-proxy -config /etc/dnscrypt-proxy.toml
+exec dnscrypt-proxy -config /etc/dnscrypt-proxy/dnscrypt-proxy.toml
diff --git a/srcpkgs/dnscrypt-proxy/template b/srcpkgs/dnscrypt-proxy/template
index e66cd6110ac6d..d7a2942943733 100644
--- a/srcpkgs/dnscrypt-proxy/template
+++ b/srcpkgs/dnscrypt-proxy/template
@@ -1,29 +1,30 @@
 # Template file for 'dnscrypt-proxy'
 pkgname=dnscrypt-proxy
-version=2.1.4
-revision=5
+version=2.1.5
+revision=1
 build_style=go
 go_import_path=github.com/dnscrypt/dnscrypt-proxy
 go_package="${go_import_path}/dnscrypt-proxy"
-hostmakedepends="go1.20"
+depends="python3-urllib3"
 short_desc="DNS proxy that encrypts queries"
 maintainer="cinerea0 <cinerea0@protonmail.com>"
 license="ISC"
 homepage="https://github.com/DNSCrypt/dnscrypt-proxy"
 changelog="https://raw.githubusercontent.com/DNSCrypt/dnscrypt-proxy/master/ChangeLog"
 distfiles="https://github.com/DNSCrypt/dnscrypt-proxy/archive/refs/tags/${version}.tar.gz"
-checksum=05f0a3e8c8f489caf95919e2a75a1ec4598edd3428d2b9dd357caba6adb2607d
-conf_files="/etc/dnscrypt-proxy.toml"
+checksum=044c4db9a3c7bdcf886ff8f83c4b137d2fd37a65477a92bfe86bf69587ea7355
+conf_files="/etc/dnscrypt-proxy/dnscrypt-proxy.toml"
 system_accounts="dnscrypt_proxy"
-make_dirs="/var/log/dnscrypt-proxy 0750 dnscrypt_proxy dnscrypt_proxy"
-
-export GOTOOLCHAIN=go1.20
 
 post_install() {
-	vconf dnscrypt-proxy/example-dnscrypt-proxy.toml dnscrypt-proxy.toml
+	vinstall dnscrypt-proxy/example-dnscrypt-proxy.toml 644 /etc/dnscrypt-proxy dnscrypt-proxy.toml
 	vlicense LICENSE
 	vsv dnscrypt-proxy
 	for example in dnscrypt-proxy/example*txt; do
-		vdoc $example
+		vsconf "${example}"
+	done
+	vbin utils/generate-domains-blocklist/generate-domains-blocklist.py
+	for f in utils/generate-domains-blocklist/*.{conf,txt}; do
+		vinstall "${f}" 644 /usr/share/dnscrypt-proxy
 	done
 }

  parent reply	other threads:[~2023-08-24 16:49 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-14  2:26 [PR PATCH] " cinerea0
2023-08-14  2:47 ` [PR PATCH] [Updated] " cinerea0
2023-08-14  3:02 ` cinerea0
2023-08-14 22:28 ` cinerea0
2023-08-14 22:30 ` [PR PATCH] [Updated] " cinerea0
2023-08-24 16:49 ` cinerea0 [this message]
2023-08-24 18:29 ` [PR PATCH] [Merged]: " Duncaen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230824164918.eC1TGZTQUTIjP18CgTjdH5qApXEJaqd5g58Gttql41s@z \
    --to=cinerea0@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).