Bug 269392 - net-p2p/transmission: configure logging in transmission-daemon service
Summary: net-p2p/transmission: configure logging in transmission-daemon service
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Robert Clausecker
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-07 18:57 UTC by dereckson
Modified: 2023-09-22 05:14 UTC (History)
4 users (show)

See Also:
mondo.debater_0q: maintainer-feedback+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description dereckson 2023-02-07 18:57:48 UTC
Transmission 3.0 allows --log-info --log=/path/to/transmission.log to the command line to specify the logging target.

The settings.json file does NOT support that option.

The upstream implementation systemd service contains those option.

Transmission 4 will change --log-<level> into --log-level=<level>, but Transmission 3 hasn't support for this syntax.

Patch: https://reviews.freebsd.org/D38424
Comment 1 Alexandre C. Guimarães freebsd_committer freebsd_triage 2023-02-08 17:59:19 UTC
Hi, Transmission 4.00 just came out. I will try to updaate the port during the weekend, and have a look on it at the same time.

Thank you.
Comment 2 dereckson 2023-02-08 18:03:40 UTC
It's still fresh on my mind because I was looking in source code for log default level.

Two changes:
  1. as indicated in description, --log-info becomes --log-level=info
  2. more levels accepted

Will update the patch directly for 4.0 so.
Comment 3 dereckson 2023-02-08 18:24:29 UTC
Patch updated for Transmission 4 compat: https://reviews.freebsd.org/D38424?vs=116720&id=116793#toc
Comment 4 Rafe 2023-09-11 19:43:11 UTC
(In reply to dereckson from comment #3)

I wish I had seen this prior to submitting my update to the port recently. I will review and incorporate this into a revision of the port.
Comment 5 Rafe 2023-09-19 19:54:10 UTC
Please see Bug 273841 for a proposed patch with a fix for this issue.
Comment 6 commit-hook freebsd_committer freebsd_triage 2023-09-22 05:08:57 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=f40cba02c2f877b8e9e5e1a591c49ff118831c19

commit f40cba02c2f877b8e9e5e1a591c49ff118831c19
Author:     Rafe <mondo.debater_0q@icloud.com>
AuthorDate: 2023-09-19 19:37:51 +0000
Commit:     Robert Clausecker <fuz@FreeBSD.org>
CommitDate: 2023-09-22 04:56:47 +0000

    net-p2p/transmission: Refine flavors, port fixes

     *Fixes flavor conflicts in Bug 273841
     *Patches to build with libfmt>=10.0. Bug 273887
     *Adds log-location and log-level options. Bug 269392
     *Fixes build errors arising from pkg-plist issues. Bug 273836

     See UPDATING for additional information on the usage of flavors.

    PR:             273841, 273887, 269392, 273836
    Reported by:    Benjamin Takacs <nimaje+fbz@bureaucracy.de>
    Reported by:    diizzy, dereckson@gmail.com
    Reported by:    Igor Pavlov <igor.arabesc.pavlov@gmail.com>

 UPDATING                                           |  13 ++
 net-p2p/transmission/Makefile                      |  87 +++++++-----
 .../files/extra-patch-CMakeLists.txt (new)         |  23 +++
 .../files/patch-gtk_Application.cc (new)           |  11 ++
 .../patch-libtransmission_file-posix.cc (new)      |  11 ++
 .../patch-libtransmission_variant-json.cc (new)    |  11 ++
 net-p2p/transmission/files/transmission.in         |  12 +-
 net-p2p/transmission/pkg-plist.cli                 |  27 ----
 net-p2p/transmission/pkg-plist.daemon              |  12 +-
 net-p2p/transmission/pkg-plist.default (new)       | 155 +++++++++++++++++++++
 net-p2p/transmission/pkg-plist.full (gone)         | 155 ---------------------
 net-p2p/transmission/pkg-plist.gtk                 |  30 +---
 net-p2p/transmission/pkg-plist.qt                  |  27 ----
 net-p2p/transmission/pkg-plist.utils               |  10 --
 14 files changed, 290 insertions(+), 294 deletions(-)