Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Running Scripts on each episode retrieved: #163

Closed
wants to merge 1 commit into from
Closed

Support Running Scripts on each episode retrieved: #163

wants to merge 1 commit into from

Conversation

balthisar
Copy link

  • Updated to current Xcode.

  • Updated to current Swift.

  • Added GUI for running a script after each torrent is downloaded. This includes adding descriptive
    text fields for all of the options for clarity.

  • The Recents window will now run the same script, if enabled, instead of opening the file with'
    the browser. If no script is defined, then previous behavior remains unaffected.

  • Added a test_script which can be used for Transmission, for example.

  • Catch will now execute a script after downloading episodes. To enable this, select
    "Run Script" in Preferences, and select a suitable script from the popup menu. This
    script will be executed independently of the "Open Automatically" Preference.

    • If the downloaded episode is a torrent file, then the script will be called with the path of
      the file as the argument. Since you're probably getting magnet links, this won't be useful.

    • If the downloaded episode is a magnet link, Catch will continue to save the stupid webloc files
      to your selected folder, but the script will be called with the magnet URL. You can then use
      this in your script as an RPC to Transmission, or any other flexible purpose.

- Updated to current Xcode.

- Updated to current Swift.

- Added GUI for running a script after each torrent is downloaded. This includes adding descriptive
  text fields for all of the options for clarity.

- The Recents window will now run the same script, if enabled, instead of opening the file with'
  the browser. If no script is defined, then previous behavior remains unaffected.

- Added a test_script which can be used for Transmission, for example.

- Catch will now execute a script after downloading episodes. To enable this, select
  "Run Script" in Preferences, and select a suitable script from the popup menu. This
  script will be executed independently of the "Open Automatically" Preference.

    - If the downloaded episode is a torrent file, then the script will be called with the path of
      the file as the argument. Since you're probably getting magnet links, this won't be useful.

    - If the downloaded episode is a magnet link, Catch will continue to save the stupid webloc files
      to your selected folder, but the script will be called with the magnet URL. You can then use
      this in your script as an RPC to Transmission, or any other flexible purpose.

commit 8b038b2
Author: Jim Derry <[email protected]>
Date:   Mon Dec 30 15:04:58 2019 -0500
@mipstian
Copy link
Owner

Not sure how to proceed here. I was working on the app as well, implementing multi-feed support. I've also added the ability to run a script to download non-torrent feed content, and was planning to add transmission_rpc as an option for torrents.

I guess the question is, does it make sense to have a script to handle torrent downloads? If we added support for transmission_rpc (and possibly other similar remote stuff), would that cover every use case?

@balthisar
Copy link
Author

balthisar commented Feb 29, 2020

Handling transmision_rpc would handle every use case for Transmission users, assuming you make no assumptions about localhost; my particular use case is the torrent client on another machine on the network. Running on a NAS is probably not a niche situation.

@mipstian
Copy link
Owner

mipstian commented Mar 1, 2020

Right, so it's either a torrent client running on this machine, or a torrent client being invoked via some kind of network call on another machine. Do we agree there's no third thing?

@balthisar
Copy link
Author

Seems correct.

@mipstian
Copy link
Owner

mipstian commented Mar 2, 2020

Ok then I’ll close this. Too bad, the code was pretty well written! Would you like to add the rpc stuff once I push my branch?

@mipstian mipstian closed this Mar 2, 2020
@balthisar
Copy link
Author

balthisar commented Mar 13, 2020 via email

@mipstian
Copy link
Owner

@balthisar I've pushed my stuff to the multifeed branch, check it out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants