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 optional use of remote filename for output #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bkuhn
Copy link

@bkuhn bkuhn commented Jun 25, 2016

As an optional feature, if useremotefilename is set to 1, then the local
filename will use the filename found in the enclosure.

This fixes some problems with bashpodder I experienced when trying to
download a back catalog of a show. With this setting, one can retrieve
many old episodes into a given date's directory.

As an optional feature, if useremotefilename is set to 1, then the local
filename will use the filename found in the enclosure.

This fixes some problems with bashpodder I experienced when trying to
download a back catalog of a show.  With this setting, one can retrieve
many old episodes into a given date's directory.
@bkuhn
Copy link
Author

bkuhn commented Jun 25, 2016

I just realized that the awk code you use attempts to do this, but does it poorly. For example, this feed:

http://www.npr.org/templates/rss/podcast.php?id=510289

does not work properly because of the URL having options after it. The awk could probably be improved instead of this change.

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