Closed Neo-Oli closed 3 years ago
I have the same issue on this podcast
http://radiofrance-podcast.net/podcast09/rss_14631.xml
and this one
I thing I have found the issue, there is a tag with
WOAF (Official audio file webpage): http://www.fipradio.fr/emissions/c-est-magnifip
and the : seems to be analyzed like an object...
I have made a custom download and retag script to handle the issue
@cbonnissent Could you share more details about this? I, too, am encountering this issue — in a different feed, and it doesn't seem to have a colon in the tags. And it breaks sync completely, so it is a major problem....
@n8willis, would you mind letting us know which feed is causing this behavior? @cbonnissent's comment is very old and they might, understandably, not be interested in this issue anymore.
Sorry I don't use anymore greg
Good luck with the issue :D
@n8willis, would you mind letting us know which feed is causing this behavior? @cbonnissent's comment is very old and they might, understandably, not be interested in this issue anymore.
In this case, I'm referring to https://badvoltage.org/feed/mp3/ (which does use the multiplication sign "×" in filenames, but also uses spaces; both potentially parsing concerns).
@cbonnissent I'm not asking you to work on the issue, but just saying what you did that solved it would go a long way towards helping other people.
In this case, I'm referring to https://badvoltage.org/feed/mp3/ (which does use the multiplication sign "×" in filenames, but also uses spaces; both potentially parsing concerns).
I have just tried adding and syncing this podcast, and there were no problems. Would you mind pasting here your error trace? Also, I made a couple of commits recently, just in case you've been using an older greg.
Did you tag it? If so, which tags? Also, which episode(s) did you try?
I did add a couple tags to the feed's section in greg.conf
, but I just noticed greg didn't actually tag them. So I'll need to look into this a bit more.
Yes; as the original reporter's logs indicated, this seems to happen when calling stagger.
It doesn't look like stagger is, shall we say, being vigorously maintained these days, which is worrying in its own right.
Yeah, I chose stagger way back when because it was one of the only python3 options around; but of course now everything is 3, so we could perhaps give eyeD3 a try. It should be a straightforward drop-in replacement.
@n8willis, if you have time, could you please check whether this PR solves your issue?
Feel free to open again if needed.
Right; will put it through a workout this weekend -- just couldn't set aside the time yet.
(In general, I'm very much in favor of eyeD3 usage. Especially because I already use it in my wrapper scripts.)
Can you make this a new release on PyPI? Please? Or at least bump the numbering?
Because apparently pip checks the internal version-numbering and doesn't just install any old git+https:// ref ... unless the user is willing to throw a bunch of workarounds at it. Which is a path I definitely don't want to go down.
Besides, it has been ages since the last official release.
Sure, I'll do it by Monday.
Done
When downloading Cortex (https://www.relay.fm/cortex/feed) I get the following error, after the download is completed.
It stoppes greg from completing the sync and it will try to download this podcast episode again during the next sync. As far as I can tell this happens only with that podcast. It used to work fine, so maybe there is a problem with the feed now. All my other podcast download normally (at least after I issue
greg edit --downloadfrom now "Cortex"
). This problem happened for the last two episodes.