This commit is contained in:
Joey Hess 2016-12-19 16:28:12 -04:00
parent a13b789d04
commit df5a0059ca
No known key found for this signature in database
GPG key ID: C910D9222512E3C7
2 changed files with 22 additions and 4 deletions

View file

@ -138,10 +138,12 @@ findDownloads u = go =<< downloadFeed u
Just $ ToDownload f u i $ Enclosure enclosureurl
Nothing -> mkquvi f i
mkquvi f i = case getItemLink i of
Just link -> ifM (quviSupported link)
( return $ Just $ ToDownload f u i $ QuviLink link
, return Nothing
)
Just link -> do
liftIO $ print ("link", link)
ifM (quviSupported link)
( return $ Just $ ToDownload f u i $ QuviLink link
, return Nothing
)
Nothing -> return Nothing
{- Feeds change, so a feed download cannot be resumed. -}

View file

@ -0,0 +1,16 @@
[[!comment format=mdwn
username="joey"
subject="""comment 1"""
date="2016-12-19T19:55:23Z"
content="""
It's somewhat misleading that it complains there are no enclosures in the
feed. While importfeed mostly downloads only enclosures in podcast feeds,
it also checks link tags, which this feed contains, to see if quvi supports
downloading content from them. Quvi does support the links in this feed,
so it should work despite there being no enclosures.
I've reproduced it not working, and it seems that the problem is this is
not quite a valid Atom feed, and the feed parsing library is failing to
parse it. Perhaps that can be improved; I filed a bug here
<https://github.com/bergmark/feed/issues/18>
"""]]