Index | Thread | Search

From:
Josh Grosse <josh@jggimi.net>
Subject:
Re: add README-main to net/transmission
To:
Rafael Sadowski <rafael@sizeofvoid.org>
Cc:
Stuart Henderson <stu@spacehopper.org>, ports@openbsd.org, Colton Lewis <colton.w.lewis@protonmail.com>
Date:
Mon, 30 Mar 2026 11:01:14 -0400

Download raw body.

Thread
On Mon, Mar 30, 2026 at 03:55:19PM +0200, Rafael Sadowski wrote:
> On Mon Mar 30, 2026 at 02:52:47PM +0100, Stuart Henderson wrote:
> > I suggest adding transmission_daemon.login instead (see e.g. squid for an
> > example) with values set to whatever's needed for typically l typical use
> > 
> 
> +1
> 
> > -- 
> >  Sent from a phone, apologies for poor formatting.
> > 
> > On 30 March 2026 14:16:13 Josh Grosse <josh@jggimi.net> wrote:
> > 
> > > Attached is a diff to add pkg/README-main to net/transmission,
> > > and includes a revision bump and PLIST-main revision.
> > > 
> > > The default "daemon" class does not always have sufficient resources
> > > for transmission_daemon.  This documents how to add a login class
> > > when needed.
> > > 
> > > OKs?  Alternative recommendations?
> > 

Thank you!  Attached is a "v2" diff, which adds the .login file
and adds a revised README-main, with verbiage adopted from 
www/squid/pkg/README-main.
diff --git net/transmission/Makefile net/transmission/Makefile
index 0a76f8c469a..d26bc6ee2c5 100644
--- net/transmission/Makefile
+++ net/transmission/Makefile
@@ -3,6 +3,7 @@ COMMENT-gtk=	BitTorrent client with GTK+ interface
 COMMENT-qt=	BitTorrent client with Qt interface
 
 VER=		4.1.1
+REVISION=	0
 DISTNAME=	transmission-${VER}
 PKGNAME-main=	transmission-${VER}
 PKGNAME-gtk=	transmission-gtk-${VER}
diff --git net/transmission/pkg/PLIST-main net/transmission/pkg/PLIST-main
index 7710a51aec0..96ffefa8e00 100644
--- net/transmission/pkg/PLIST-main
+++ net/transmission/pkg/PLIST-main
@@ -22,6 +22,7 @@
 @man man/man1/transmission-remote.1
 @man man/man1/transmission-show.1
 @comment share/applications/
+share/doc/pkg-readmes/${PKGSTEM}
 share/doc/transmission/
 share/doc/transmission/AUTHORS
 share/doc/transmission/COPYING
@@ -72,6 +73,8 @@ share/doc/transmission/news/news-4.1.1.md
 share/doc/transmission/news/news-5.0.0-draft.md
 share/doc/transmission/rpc-spec.md
 share/doc/transmission/send-email-when-torrent-done.sh
+share/examples/login.conf.d/transmission_daemon
+@sample ${SYSCONFDIR}/login.conf.d/transmission_daemon
 @comment share/icons/
 @comment share/icons/hicolor/
 @comment share/icons/hicolor/scalable/
diff --git net/transmission/pkg/README-main net/transmission/pkg/README-main
new file mode 100644
index 00000000000..3e3911f5084
--- /dev/null
+++ net/transmission/pkg/README-main
@@ -0,0 +1,15 @@
++-------------------------------------------------------------------------------
+| Running ${PKGSTEM} on OpenBSD
++-------------------------------------------------------------------------------
+
+System resource limits
+======================
+When started from rc.d(8) (i.e. via pkg_scripts in rc.conf.local(5) or
+from "${RCDIR}/transmission_daemon start"), the transmission_daemon process
+runs in the login(1) class of "transmission_daemon".  You may need to adjust
+the limits in /etc/login.conf.d/transmission_daemon to match your needs.
+
+There is also a kernel limit on the number of open file descriptors,
+sysctl kern.maxfiles. In some circumstances you may also need to raise
+this; if this is required, pay particular attention to testing and
+monitoring your system.
diff --git net/transmission/pkg/transmission_daemon.login net/transmission/pkg/transmission_daemon.login
new file mode 100644
index 00000000000..17865891ef6
--- /dev/null
+++ net/transmission/pkg/transmission_daemon.login
@@ -0,0 +1,4 @@
+transmission_daemon:\
+        :openfiles-cur=512:\
+        :openfiles-max=1024:\
+        :tc=daemon: