Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1618 closed Enhancement (fixed)

tm-daemon: peerport can't be set from the commandline (overriding the settings.json)

Reported by: fkrueger Owned by: charles
Priority: Normal Milestone: 1.50
Component: Daemon Version: 1.42
Severity: Normal Keywords:
Cc:

Description

PeerPort? couldn't be set from the commandline, so here's a small patch that may or may not work to provide -P <peerport>. Could work for the other startups as well.

Attachments (3)

100_add_peerport_to_commandline.patch (2.4 KB) - added by fkrueger 12 years ago.
daemon_more_commandline_options.diff (5.3 KB) - added by er13 12 years ago.
encryption_options_mixed_up.diff (1.6 KB) - added by er13 12 years ago.

Download all attachments as: .zip

Change History (12)

Changed 12 years ago by fkrueger

comment:1 Changed 12 years ago by charles

  • Type changed from Bug to Enhancement

comment:2 Changed 12 years ago by charles

  • Milestone changed from 1.41 to 1.50

comment:3 Changed 12 years ago by er13

  • Version changed from 1.40 to 1.42+

Here is yet another patch that adds the peerport option from above and a couple of other ones. Most of them are useful on (embedded) systems running daemon and webinterface only (i.e. no transmission-remote). Would appreciate it, if the patch would make it into the upstream. The patch is for trunk r7546.

Changed 12 years ago by er13

comment:4 Changed 12 years ago by er13

don't know why, but patch-web-view misses some changed files

comment:5 Changed 12 years ago by charles

  • Status changed from new to assigned
  • Version changed from 1.42+ to 1.42

comment:6 Changed 12 years ago by charles

  • Resolution set to fixed
  • Status changed from assigned to closed

daemon_more_commandline_options.diff added to trunk in r7547. Thanks for the patch!

comment:7 Changed 12 years ago by er13

  • Resolution fixed deleted
  • Status changed from closed to reopened

shame on me, copy-&-pasted code without reading/testing it, fix attached

Changed 12 years ago by er13

comment:8 Changed 12 years ago by charles

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:9 Changed 12 years ago by er13

@Charles: just for the case you've missed it. The encryption_options_mixed_up.diff fixes the bug in the daemon as well as in the client (this is where I copy-&-pasted it from). In r7554 you've committed only the daemon part.

Note: See TracTickets for help on using tickets.