Skip to content
This repository has been archived by the owner on Oct 10, 2019. It is now read-only.

An image instead of audio is selected as the Podcast Download URL #604

Open
wzhd opened this issue Oct 28, 2017 · 3 comments
Open

An image instead of audio is selected as the Podcast Download URL #604

wzhd opened this issue Oct 28, 2017 · 3 comments

Comments

@wzhd
Copy link

wzhd commented Oct 28, 2017

Newsbeuter version (copy from newsbeuter -v): newsbeuter 2.9

Steps to reproduce the issue:

  1. Add a podcast that has both image and audio files, for instance, nstaaf. If you open this in Firefox, you can see that each episode contains too files.

  2. Press Enter to view details, notice the Podcast Download URL links to a jpg file

Other info you think is relevant:

I think there should be an option to download all attachments.

@Minoru
Copy link
Collaborator

Minoru commented Oct 28, 2017

Newsbeuter is no longer maintained; I'm leading a fork named Newsboat.

Regarding your suggestion, do you need all attachments, or will the first audio suffice? (The latter is much easier to implement.)

@wzhd
Copy link
Author

wzhd commented Oct 29, 2017

@Minoru

(Thanks for keeping an eye on issues posted here)

Yes, the first audio is all that I need.

Minoru added a commit to newsboat/newsboat that referenced this issue Oct 29, 2017
@Minoru
Copy link
Collaborator

Minoru commented Oct 29, 2017

Newsboat now uses the last audio enclosure as its podcast URL. "Last" is easier to implement than "first"; works fine for "audio and jpeg" situation, and if it ever becomes a problem for someone, we'll need much bigger changes anyway.

Please build from my repo and test if you can.

0-wiz-0 pushed a commit to NetBSD/pkgsrc-wip that referenced this issue Dec 26, 2017
Lists below only mention user-visible changes, but I would also like to
acknowledge contributions from the following people: Alok Singh, Carno, Jonas
Karlsson, Kamil Wsół, Mike Crute, Niels Kobschätzki, and maiki.

- HTML anchors for all config commands in docs. You can now link to each command
    separately (#10) (Lysander Trischler)
- Support for Inoreader (Bart Libert)
- Slovak translation (František Hájik)

- Enqueue *last* audio enclosure
    (akrennmair/newsbeuter#604)
- `text-width` doesn't apply if it's bigger than terminal width
    (akrennmair/newsbeuter#602)
- Translations: German (Lysander Trischler), Russian, Ukrainian
    (Alexander Batischev)

- Build dependency on Perl (#6)
- Test dependency on bc (Nikos Tsipinakis)

- Do not create XDG data dir if not using XDG (#8)
- When used with Newsblur, check on startup if cookie-cache exists or can be
    created, because integration doesn't work without cookies (#13)
- Builds on AARCH64 and ARMHF (#43)
- Only show an error message once when unknown option is supplied
    (Lysander Trischler)
- License header used to say it's MIT/X Consortium License, whereas in reality
    it's a MIT License (discovered by Nikos Tsipinakis)
- Cross-compilation made possible by conditionally assigning to RANLIB and AR in
    Makefile (Fredrik Fornwall)
- Cookies actually get persisted (Simon Schuster, reported and tested by Håkan
    Jerning)
- CJK text is wrapped at correct code-point boundaries (#71) (nmtake)
- Don't segfault if `error-log` points to non-existent file (Simon Schuster)
- Spanish translation (José Manuel García-Patos)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants