Opened 14 years ago

Closed 14 years ago

#732 closed Enhancement (invalid)

auto-add directory feature for transmission-daemon

Reported by: bsteinb Owned by: bsteinb
Priority: Normal Milestone: None Set
Component: Daemon Version:
Severity: Normal Keywords:
Cc:

Description

As I intend to replace an instance of btlaunchmany I'm running on a remote server by transmission-daemon, I have started to implement an auto-add directory feature like in btlaunchmany or the Mac OS X version of Transmission for transmission-daemon. Is there any interest for such a feature? If so, I would be glad to contribute the code, although I might need some help/advice.

As of now, my code succeeds in adding the .torrent files it finds on any level below a given point (the global download directory at this time) in the filesystem hierarchy (yay!). It schedules a timer using the libevent facilities and periodically rescans the directory layout for .torrent files, which means, it does not check for actual changes, like the Mac OS X GUI version, but instead just adds all the .torrent files it finds periodically, which leads to a number of error messages from torrents.c, line 547.

Attachments (3)

daemon_c.diff (546 bytes) - added by bsteinb 14 years ago.
autoload.h (158 bytes) - added by bsteinb 14 years ago.
autoload.c (2.5 KB) - added by bsteinb 14 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 14 years ago by bsteinb

  • Type changed from Bug to Enhancement

comment:2 Changed 14 years ago by bsteinb

  • Owner changed from SoftwareElves to bsteinb

Changed 14 years ago by bsteinb

Changed 14 years ago by bsteinb

Changed 14 years ago by bsteinb

comment:3 Changed 14 years ago by mamruoc

I would love to see this happend

comment:5 Changed 14 years ago by charles

  • Resolution set to invalid
  • Status changed from new to closed

On the whole the patch looks good; thanks for submitting it.

However, IMO this patch is incomplete. I'd be more inclined to accept a revised patch which (1) lets the user turn the feature on & off via the client, (2) lets the user specify the watch directory via the client, and (3) is off by default.

Please attach an updated patch and reopen this ticket. Thanks!

Note: See TracTickets for help on using tickets.