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

changes to Addic7ed plugin #39

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

birt
Copy link
Contributor

@birt birt commented Dec 2, 2013

I've made some changes to the Addic7ed plugin:

  • more debugging output
  • should be working in most cases but needs more testing; adding it to upstream might increase exposure and provide more test cases

Brian Damage and others added 7 commits December 2, 2013 15:56
- added some more debugging output for easier troubleshooting later on
- made some changes to the page parsing, tested it on 4 releases with good results for me but needs more testing
- added some more debugging output for easier troubleshooting later on
- made some changes to the page parsing, tested it on 4 releases with good results for me but needs more testing
…e subtitles folder.

( --subtitles-folder)

This will still first download the subtitles file in the directory where the media file is located. So it updates the time on that file/directory which is
something we want to avoid if we have scripts processing our media files based on the age of the files.

We have to assume the underlying filesystem doesn't support "time of birth" after stat() call.

See http://unix.stackexchange.com/questions/24441/get-file-created-creation-time
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.

1 participant