Issues trying to use nnrss

Issues trying to use nnrss

[David Z Maze]

[-- Attachment #1: Type: text/plain, Size: 323 bytes --]

I'm experimenting with trying to use nnrss to browse various RSS
feeds.  (Shock, that.)  A couple of issues:

(1) The condition-case construction in nnrss-find-rss-via-syndic8 is
    wrong, and things die horribly if I don't have an xml-rpc
    package.  The handler needs to have a condition attached to it,
    like so:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 531 bytes --]

--- nnrss.el    27 Apr 2003 04:38:20 -0000      6.43
+++ nnrss.el    29 Apr 2003 21:44:10 -0000
@@ -707,7 +707,7 @@
                                  "Multiple feeds found.  Select one: "
                                  selection nil t) urllist)))
                       (cdar urllist))))))
-    (message "XML-RPC is not available... not checking Syndic8.")))
+    (error (message "XML-RPC is not available... not checking Syndic8."))))

 (defun nnrss-rss-p (data)
   "Test if data is an RSS feed.  Simply ensures that the first

[-- Attachment #3: Type: text/plain, Size: 195 bytes --]


(2) Given this, I still can't look at things like Slashdot.  If I do
    'G m slashdot nnrss RET http://slashdot.org/' to create a group,
    Gnus will hunt around for a group and then display


[-- Attachment #4: Type: text/plain, Size: 122 bytes --]

Could not contact host: : / 80
Attempted using gateway method: native
---- Error was: ----
:/80 Name or service not known

[-- Attachment #5: Type: text/plain, Size: 554 bytes --]


    in an index.rss buffer.

(3) If I know the explicit URL of the RSS feed, say for a livejournal
    journal, I get an even more special failure.  I create the group
    by 'G m lj.debian nnrss RET
    http://www.livejournal.com/community/debian/rss', and get a
    stringp-of-list error.  It looks like this happens inside
    nnrss-check-group; if does (setq url (nnrss-discover-feed ...)),
    but what nnrss-discover-feed actually returns is an alist with the
    description, title, and href of the feed.  This patch will set url
    correctly:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #6: Type: text/x-patch, Size: 672 bytes --]

--- nnrss.el    27 Apr 2003 04:38:20 -0000      6.43
+++ nnrss.el    29 Apr 2003 21:59:17 -0000
@@ -413,9 +413,11 @@
                    (second (assoc group nnrss-group-alist))))
       (unless url
        (setq url
-             (nnrss-discover-feed
-              (read-string
-               (format "URL to search for %s: " group) "http://")))
+             (cdr
+              (assoc 'href
+                     (nnrss-discover-feed
+                      (read-string
+                       (format "URL to search for %s: " group) "http://")))))
        (let ((pair (assoc group nnrss-server-data)))
          (if pair
              (setcdr (cdr pair) (list url))

[-- Attachment #7: Type: text/plain, Size: 297 bytes --]


    There's no error recovery, though; if nnrss-discover-feed fails
    utterly, you wind up with url set to sommething bizarre.

-- 
David Maze             dmaze@mit.edu          http://www.mit.edu/~dmaze/
"Theoretical politics is interesting.  Politicking should be illegal."
	-- Abra Mitchell

Re: Issues trying to use nnrss

[Lars Magne Ingebrigtsen]

David Z Maze <dmaze@MIT.EDU> writes:

> (1) The condition-case construction in nnrss-find-rss-via-syndic8 is
>     wrong, and things die horribly if I don't have an xml-rpc
>     package.  The handler needs to have a condition attached to it,
>     like so:

Thanks for the patch; I've applied it to Oort Gnus v0.20 (i. e., CVS).

>     http://www.livejournal.com/community/debian/rss', and get a
>     stringp-of-list error.  It looks like this happens inside
>     nnrss-check-group; if does (setq url (nnrss-discover-feed ...)),
>     but what nnrss-discover-feed actually returns is an alist with the
>     description, title, and href of the feed.  This patch will set url
>     correctly:

Ditto.

-- 
(domestic pets only, the antidote for overdose, milk.)
   larsi@gnus.org * Lars Magne Ingebrigtsen