Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] syncthing: install sysctl file
@ 2024-11-06 20:42 dkwo
  2024-11-06 21:18 ` [PR PATCH] [Updated] " dkwo
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: dkwo @ 2024-11-06 20:42 UTC (permalink / raw)
  To: ml

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

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

https://github.com/dkwo/void-packages st
https://github.com/void-linux/void-packages/pull/52938

syncthing: install sysctl file
Upstream recommends installing this.
I tested the changes in this PR within my dinit user service.

cc @Duncaen 

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

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

From a71211e56eb5d8aacf8899a74060a652d7a8a07e Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Wed, 6 Nov 2024 15:32:56 -0500
Subject: [PATCH] syncthing: install sysctl file

---
 srcpkgs/syncthing/template | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/syncthing/template b/srcpkgs/syncthing/template
index d559424b6a3f84..ee899ed6370bbf 100644
--- a/srcpkgs/syncthing/template
+++ b/srcpkgs/syncthing/template
@@ -1,7 +1,7 @@
 # Template file for 'syncthing'
 pkgname=syncthing
 version=1.28.0
-revision=1
+revision=2
 build_style=go
 go_import_path="github.com/syncthing/syncthing"
 go_package="
@@ -28,6 +28,7 @@ post_install() {
 	vinstall etc/firewall-ufw/syncthing 644 etc/ufw/applications.d
 	vinstall etc/linux-desktop/syncthing-start.desktop 664 usr/share/applications
 	vinstall etc/linux-desktop/syncthing-ui.desktop 644 usr/share/applications
+	vinstall etc/linux-sysctl/30-syncthing.conf 644 usr/lib/sysctl.d
 	vlicense LICENSE
 	vdoc README.md
 	for f in man/syncthing*; do vman $f; done

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

* Re: [PR PATCH] [Updated] syncthing: install sysctl file
  2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
@ 2024-11-06 21:18 ` dkwo
  2024-12-08 15:47 ` dkwo
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: dkwo @ 2024-11-06 21:18 UTC (permalink / raw)
  To: ml

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

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

https://github.com/dkwo/void-packages st
https://github.com/void-linux/void-packages/pull/52938

syncthing: install sysctl file
Upstream recommends installing this.
I tested the changes in this PR within my dinit user service.

cc @Duncaen 

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

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

From 01cbbf563070bbc5a98bd47f30c38a40aaf296e4 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Wed, 6 Nov 2024 15:32:56 -0500
Subject: [PATCH] syncthing: install sysctl file, only run meaningful tests

---
 srcpkgs/syncthing/template | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/syncthing/template b/srcpkgs/syncthing/template
index d559424b6a3f84..9a418dda544ac3 100644
--- a/srcpkgs/syncthing/template
+++ b/srcpkgs/syncthing/template
@@ -1,7 +1,7 @@
 # Template file for 'syncthing'
 pkgname=syncthing
 version=1.28.0
-revision=1
+revision=2
 build_style=go
 go_import_path="github.com/syncthing/syncthing"
 go_package="
@@ -10,6 +10,10 @@ go_package="
  ${go_import_path}/cmd/syncthing"
 go_build_tags="noupgrade"
 go_ldflags="-X github.com/syncthing/syncthing/lib/build.Version=v${version}"
+make_check_target="./lib/...
+ ./cmd/stdiscosrv
+ ./cmd/strelaysrv
+ ./cmd/syncthing"
 short_desc="Open Source Continuous File Synchronization"
 maintainer="Duncaen <duncaen@voidlinux.org>"
 license="MPL-2.0"
@@ -28,6 +32,7 @@ post_install() {
 	vinstall etc/firewall-ufw/syncthing 644 etc/ufw/applications.d
 	vinstall etc/linux-desktop/syncthing-start.desktop 664 usr/share/applications
 	vinstall etc/linux-desktop/syncthing-ui.desktop 644 usr/share/applications
+	vinstall etc/linux-sysctl/30-syncthing.conf 644 usr/lib/sysctl.d
 	vlicense LICENSE
 	vdoc README.md
 	for f in man/syncthing*; do vman $f; done

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

* Re: [PR PATCH] [Updated] syncthing: install sysctl file
  2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
  2024-11-06 21:18 ` [PR PATCH] [Updated] " dkwo
@ 2024-12-08 15:47 ` dkwo
  2025-01-15 11:21 ` dkwo
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: dkwo @ 2024-12-08 15:47 UTC (permalink / raw)
  To: ml

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

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

https://github.com/dkwo/void-packages st
https://github.com/void-linux/void-packages/pull/52938

syncthing: install sysctl file
Upstream recommends installing this.
I tested the changes in this PR within my dinit user service.

cc @Duncaen 

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

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

From c03fbe3d3f05db7b1b2b82cd69edd615ba77bb67 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Wed, 6 Nov 2024 15:32:56 -0500
Subject: [PATCH] syncthing: install sysctl file, only run meaningful tests

---
 srcpkgs/syncthing/template | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/syncthing/template b/srcpkgs/syncthing/template
index d7195fd2e279f0..736ccb2b9e4897 100644
--- a/srcpkgs/syncthing/template
+++ b/srcpkgs/syncthing/template
@@ -1,7 +1,7 @@
 # Template file for 'syncthing'
 pkgname=syncthing
 version=1.28.1
-revision=1
+revision=2
 build_style=go
 go_import_path="github.com/syncthing/syncthing"
 go_package="
@@ -10,6 +10,10 @@ go_package="
  ${go_import_path}/cmd/syncthing"
 go_build_tags="noupgrade"
 go_ldflags="-X github.com/syncthing/syncthing/lib/build.Version=v${version}"
+make_check_target="./lib/...
+ ./cmd/stdiscosrv
+ ./cmd/strelaysrv
+ ./cmd/syncthing"
 short_desc="Open Source Continuous File Synchronization"
 maintainer="Duncaen <duncaen@voidlinux.org>"
 license="MPL-2.0"
@@ -28,6 +32,7 @@ post_install() {
 	vinstall etc/firewall-ufw/syncthing 644 etc/ufw/applications.d
 	vinstall etc/linux-desktop/syncthing-start.desktop 664 usr/share/applications
 	vinstall etc/linux-desktop/syncthing-ui.desktop 644 usr/share/applications
+	vinstall etc/linux-sysctl/30-syncthing.conf 644 usr/lib/sysctl.d
 	vlicense LICENSE
 	vdoc README.md
 	for f in man/syncthing*; do vman $f; done

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

* Re: [PR PATCH] [Updated] syncthing: install sysctl file
  2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
  2024-11-06 21:18 ` [PR PATCH] [Updated] " dkwo
  2024-12-08 15:47 ` dkwo
@ 2025-01-15 11:21 ` dkwo
  2025-01-15 14:27 ` dkwo
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: dkwo @ 2025-01-15 11:21 UTC (permalink / raw)
  To: ml

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

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

https://github.com/dkwo/void-packages st
https://github.com/void-linux/void-packages/pull/52938

syncthing: install sysctl file
Upstream recommends installing this.
I tested the changes in this PR within my dinit user service.

cc @Duncaen 

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

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

From a43aa37d77b5cce60dfe4f6a47bf2cbd6f783f61 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Wed, 6 Nov 2024 15:32:56 -0500
Subject: [PATCH] syncthing: install sysctl file, only run meaningful tests

---
 srcpkgs/syncthing/template | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/syncthing/template b/srcpkgs/syncthing/template
index efea121b99ca6f..c5edab9880e80a 100644
--- a/srcpkgs/syncthing/template
+++ b/srcpkgs/syncthing/template
@@ -1,7 +1,7 @@
 # Template file for 'syncthing'
 pkgname=syncthing
 version=1.29.2
-revision=1
+revision=2
 build_style=go
 go_import_path="github.com/syncthing/syncthing"
 go_package="
@@ -10,6 +10,10 @@ go_package="
  ${go_import_path}/cmd/syncthing"
 go_build_tags="noupgrade"
 go_ldflags="-X github.com/syncthing/syncthing/lib/build.Version=v${version}"
+make_check_target="./lib/...
+ ./cmd/stdiscosrv
+ ./cmd/strelaysrv
+ ./cmd/syncthing"
 short_desc="Open Source Continuous File Synchronization"
 maintainer="Duncaen <duncaen@voidlinux.org>"
 license="MPL-2.0"
@@ -28,6 +32,7 @@ post_install() {
 	vinstall etc/firewall-ufw/syncthing 644 etc/ufw/applications.d
 	vinstall etc/linux-desktop/syncthing-start.desktop 664 usr/share/applications
 	vinstall etc/linux-desktop/syncthing-ui.desktop 644 usr/share/applications
+	vinstall etc/linux-sysctl/30-syncthing.conf 644 usr/lib/sysctl.d
 	vlicense LICENSE
 	vdoc README.md
 	for f in man/syncthing*; do vman $f; done

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

* Re: syncthing: install sysctl file
  2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
                   ` (2 preceding siblings ...)
  2025-01-15 11:21 ` dkwo
@ 2025-01-15 14:27 ` dkwo
  2025-01-17 10:19 ` [PR PATCH] [Updated] " dkwo
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: dkwo @ 2025-01-15 14:27 UTC (permalink / raw)
  To: ml

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

New comment by dkwo on void-packages repository

https://github.com/void-linux/void-packages/pull/52938#issuecomment-2593007514

Comment:
On i686, one test fails, it runs out of memory:
```
=== RUN   TestStressBufferPool
runtime: out of memory: cannot allocate 20971520-byte block (4080566272 in use)
fatal error: out of memory
FAIL	github.com/syncthing/syncthing/lib/protocol	1.969s
```
Not sure it's very relevant.

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

* Re: [PR PATCH] [Updated] syncthing: install sysctl file
  2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
                   ` (3 preceding siblings ...)
  2025-01-15 14:27 ` dkwo
@ 2025-01-17 10:19 ` dkwo
  2025-01-17 10:53 ` dkwo
  2025-01-18  3:02 ` [PR PATCH] [Merged]: " Duncaen
  6 siblings, 0 replies; 8+ messages in thread
From: dkwo @ 2025-01-17 10:19 UTC (permalink / raw)
  To: ml

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

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

https://github.com/dkwo/void-packages st
https://github.com/void-linux/void-packages/pull/52938

syncthing: install sysctl file
Upstream recommends installing this.
I tested the changes in this PR within my dinit user service.

cc @Duncaen 

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

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

From 300fcbe6a68921d763d3da18e5abda68d49750a9 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Wed, 6 Nov 2024 15:32:56 -0500
Subject: [PATCH] syncthing: install sysctl file, only test and generate
 meaningful stuff

---
 srcpkgs/syncthing/template | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/syncthing/template b/srcpkgs/syncthing/template
index efea121b99ca6f..fe4f31c502f50e 100644
--- a/srcpkgs/syncthing/template
+++ b/srcpkgs/syncthing/template
@@ -1,15 +1,19 @@
 # Template file for 'syncthing'
 pkgname=syncthing
 version=1.29.2
-revision=1
+revision=2
 build_style=go
 go_import_path="github.com/syncthing/syncthing"
 go_package="
  ${go_import_path}/cmd/stdiscosrv
  ${go_import_path}/cmd/strelaysrv
  ${go_import_path}/cmd/syncthing"
-go_build_tags="noupgrade"
+go_build_tags="-no-upgrade"
 go_ldflags="-X github.com/syncthing/syncthing/lib/build.Version=v${version}"
+make_check_target="./lib/...
+ ./cmd/stdiscosrv
+ ./cmd/strelaysrv
+ ./cmd/syncthing"
 short_desc="Open Source Continuous File Synchronization"
 maintainer="Duncaen <duncaen@voidlinux.org>"
 license="MPL-2.0"
@@ -19,15 +23,14 @@ distfiles="https://github.com/syncthing/syncthing/archive/v${version}.tar.gz"
 checksum=6446e52cb5ca6584710c3abd9250e0d9708486d2dc2264f7c869ade169876a57
 
 pre_build() {
-	GOARCH= go generate \
-		github.com/syncthing/syncthing/lib/api/auto \
-		github.com/syncthing/syncthing/cmd/infra/strelaypoolsrv/auto
+	GOARCH= go generate github.com/syncthing/syncthing/lib/api/auto
 }
 
 post_install() {
 	vinstall etc/firewall-ufw/syncthing 644 etc/ufw/applications.d
 	vinstall etc/linux-desktop/syncthing-start.desktop 664 usr/share/applications
 	vinstall etc/linux-desktop/syncthing-ui.desktop 644 usr/share/applications
+	vinstall etc/linux-sysctl/30-syncthing.conf 644 usr/lib/sysctl.d
 	vlicense LICENSE
 	vdoc README.md
 	for f in man/syncthing*; do vman $f; done

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

* Re: syncthing: install sysctl file
  2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
                   ` (4 preceding siblings ...)
  2025-01-17 10:19 ` [PR PATCH] [Updated] " dkwo
@ 2025-01-17 10:53 ` dkwo
  2025-01-18  3:02 ` [PR PATCH] [Merged]: " Duncaen
  6 siblings, 0 replies; 8+ messages in thread
From: dkwo @ 2025-01-17 10:53 UTC (permalink / raw)
  To: ml

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

New comment by dkwo on void-packages repository

https://github.com/void-linux/void-packages/pull/52938#issuecomment-2593007514

Comment:
On i686, one test fails, it runs out of memory:
```
=== RUN   TestStressBufferPool
runtime: out of memory: cannot allocate 20971520-byte block (4080566272 in use)
fatal error: out of memory
FAIL	github.com/syncthing/syncthing/lib/protocol	1.969s
```
Not sure it's very relevant.

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

* Re: [PR PATCH] [Merged]: syncthing: install sysctl file
  2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
                   ` (5 preceding siblings ...)
  2025-01-17 10:53 ` dkwo
@ 2025-01-18  3:02 ` Duncaen
  6 siblings, 0 replies; 8+ messages in thread
From: Duncaen @ 2025-01-18  3:02 UTC (permalink / raw)
  To: ml

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

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

syncthing: install sysctl file
https://github.com/void-linux/void-packages/pull/52938

Description:
Upstream recommends installing this.
I tested the changes in this PR within my dinit user service.

cc @Duncaen 

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

end of thread, other threads:[~2025-01-18  3:02 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-11-06 20:42 [PR PATCH] syncthing: install sysctl file dkwo
2024-11-06 21:18 ` [PR PATCH] [Updated] " dkwo
2024-12-08 15:47 ` dkwo
2025-01-15 11:21 ` dkwo
2025-01-15 14:27 ` dkwo
2025-01-17 10:19 ` [PR PATCH] [Updated] " dkwo
2025-01-17 10:53 ` dkwo
2025-01-18  3:02 ` [PR PATCH] [Merged]: " Duncaen

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