Gnus development mailing list
 help / color / mirror / Atom feed
* shr.el - browse-url-url-encode-chars problem
@ 2010-10-04 17:19 CHENG Gao
  2010-10-04 17:25 ` Lars Magne Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: CHENG Gao @ 2010-10-04 17:19 UTC (permalink / raw)
  To: ding

While visiting gwene group, I got:
,----
| shr-tag-img: Symbol's function definition is void: browse-url-url-encode-chars
`----

Guess shr.el need require browse-url.el or make
browse-url-url-encode-chars autoload.




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: shr.el - browse-url-url-encode-chars problem
  2010-10-04 17:19 shr.el - browse-url-url-encode-chars problem CHENG Gao
@ 2010-10-04 17:25 ` Lars Magne Ingebrigtsen
  2010-10-05  0:39   ` Katsumi Yamaoka
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Magne Ingebrigtsen @ 2010-10-04 17:25 UTC (permalink / raw)
  To: ding

CHENG Gao <chenggao@cyberhut.org> writes:

> Guess shr.el need require browse-url.el or make
> browse-url-url-encode-chars autoload.

Yes, this has been fixed recently.

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




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: shr.el - browse-url-url-encode-chars problem
  2010-10-04 17:25 ` Lars Magne Ingebrigtsen
@ 2010-10-05  0:39   ` Katsumi Yamaoka
  2010-10-05  1:16     ` Katsumi Yamaoka
  0 siblings, 1 reply; 6+ messages in thread
From: Katsumi Yamaoka @ 2010-10-05  0:39 UTC (permalink / raw)
  To: ding

Lars Magne Ingebrigtsen wrote:
> CHENG Gao <chenggao@cyberhut.org> writes:

>> Guess shr.el need require browse-url.el or make
>> browse-url-url-encode-chars autoload.

> Yes, this has been fixed recently.

Neither Emacs 22 nor XEmacs provides browse-url-url-encode-chars.
I've introduced `shr-encode-url-chars' that's a copy of it.



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: shr.el - browse-url-url-encode-chars problem
  2010-10-05  0:39   ` Katsumi Yamaoka
@ 2010-10-05  1:16     ` Katsumi Yamaoka
  2010-10-05  1:29       ` Katsumi Yamaoka
  0 siblings, 1 reply; 6+ messages in thread
From: Katsumi Yamaoka @ 2010-10-05  1:16 UTC (permalink / raw)
  To: ding

Katsumi Yamaoka wrote:
> Lars Magne Ingebrigtsen wrote:
>> CHENG Gao <chenggao@cyberhut.org> writes:

>>> Guess shr.el need require browse-url.el or make
>>> browse-url-url-encode-chars autoload.

>> Yes, this has been fixed recently.

> Neither Emacs 22 nor XEmacs provides browse-url-url-encode-chars.
> I've introduced `shr-encode-url-chars' that's a copy of it.

Oops.  I'll revert it later, since shr.el is for only Emacsen
having `libxml-parse-html-region' support.

How about excluding shr.el from the byte-compilation in `make'?



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: shr.el - browse-url-url-encode-chars problem
  2010-10-05  1:16     ` Katsumi Yamaoka
@ 2010-10-05  1:29       ` Katsumi Yamaoka
  2010-10-05 16:41         ` Lars Magne Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Katsumi Yamaoka @ 2010-10-05  1:29 UTC (permalink / raw)
  To: ding

Katsumi Yamaoka wrote:
> How about excluding shr.el from the byte-compilation in `make'?

That is:

--- dgnushack.el~	2010-10-04 22:06:51 +0000
+++ dgnushack.el	2010-10-05 01:28:25 +0000
@@ -283,6 +283,8 @@
 		 '("md5.el")
 	       '("gnus-xmas.el" "messagexmas.el" "nnheaderxm.el")))
       (setq files (delete file files)))
+    (unless (fboundp 'libxml-parse-html-region)
+      (setq files (delete "shr.el" files)))
 
     (dolist (file files)
       (setq file (expand-file-name file srcdir))



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: shr.el - browse-url-url-encode-chars problem
  2010-10-05  1:29       ` Katsumi Yamaoka
@ 2010-10-05 16:41         ` Lars Magne Ingebrigtsen
  0 siblings, 0 replies; 6+ messages in thread
From: Lars Magne Ingebrigtsen @ 2010-10-05 16:41 UTC (permalink / raw)
  To: ding

Katsumi Yamaoka <yamaoka@jpl.org> writes:

>> How about excluding shr.el from the byte-compilation in `make'?
>
> That is:

Yes, please apply.

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




^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-10-05 16:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-04 17:19 shr.el - browse-url-url-encode-chars problem CHENG Gao
2010-10-04 17:25 ` Lars Magne Ingebrigtsen
2010-10-05  0:39   ` Katsumi Yamaoka
2010-10-05  1:16     ` Katsumi Yamaoka
2010-10-05  1:29       ` Katsumi Yamaoka
2010-10-05 16:41         ` Lars Magne Ingebrigtsen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).