GNU bug report logs - #5459
Should url-util require url-vars?

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 5459 in the body.
You can then email your comments to 5459 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Emacs Bugs <bug-gnu-emacs <at> gnu.org>
Subject: Should url-util require url-vars?
Date: Sat, 23 Jan 2010 21:36:31 +0100

I got these error messages when calling url-retrieve (which is autloaded):

error in process filter:
url-http-content-length-after-change-function: Symbol's value as
variable is void: url-show-status
error in process filter: Symbol's value as variable is void: url-show-status
error in process filter:
url-http-content-length-after-change-function: Symbol's value as
variable is void: url-show-status
error in process filter: Symbol's value as variable is void: url-show-status
...

url-show-status is defined in url-vars.el.
url-http-content-length-after-change-function calls
url-display-percentage which needs url-show-status.

url-display-percentage is defined in url-util.el. Maybe this library
should require url-vars?

Message #8 received at 5459 <at> debbugs.gnu.org (full text, mbox):

From: Chong Yidong <cyd <at> stupidchicken.com>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 5459 <at> debbugs.gnu.org
Subject: Re: Should url-util require url-vars?
Date: Sat, 23 Jan 2010 17:50:53 -0500

> url-show-status is defined in url-vars.el.
> url-http-content-length-after-change-function calls
> url-display-percentage which needs url-show-status.
>
> url-display-percentage is defined in url-util.el. Maybe this library
> should require url-vars?

Yes.  Fixed now, thanks.

Message #13 received at 5459 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Chong Yidong <cyd <at> stupidchicken.com>
Cc: 5459 <at> debbugs.gnu.org
Subject: Re: Should url-util require url-vars?
Date: Sun, 24 Jan 2010 23:40:46 +0100

On Sat, Jan 23, 2010 at 11:50 PM, Chong Yidong <cyd <at> stupidchicken.com> wrote:
>> url-show-status is defined in url-vars.el.
>> url-http-content-length-after-change-function calls
>> url-display-percentage which needs url-show-status.
>>
>> url-display-percentage is defined in url-util.el. Maybe this library
>> should require url-vars?
>
> Yes.  Fixed now, thanks.


Thanks. Unfortunately there are more trouble. And I suspect serious
trouble. Please reopen the bug.


This is what I get in my *Messages* buffer on w32 after explicitly
loading url-vars.el (I have not rebuilt Emacs yet with your changes):

  (featurep 'url-vars)=t, url-show-status=nil
  error in process filter:
url-http-content-length-after-change-function: Symbol's value as
variable is void: url-show-status

The first line is just from

  (message "(featurep 'url-vars)=%s, url-show-status=%s" (featurep
'url-vars) url-show-status)

The second line looks very strange to me since url-show-status was
defined on the first line and it is defined in url-vars.el.

Message #16 received at 5459 <at> debbugs.gnu.org (full text, mbox):

From: Chong Yidong <cyd <at> stupidchicken.com>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 5459 <at> debbugs.gnu.org
Subject: Re: Should url-util require url-vars?
Date: Sun, 24 Jan 2010 17:48:53 -0500

Lennart Borgman <lennart.borgman <at> gmail.com> writes:

> This is what I get in my *Messages* buffer on w32 after explicitly
> loading url-vars.el (I have not rebuilt Emacs yet with your changes):
>
>   (featurep 'url-vars)=t, url-show-status=nil
>   error in process filter:
> url-http-content-length-after-change-function: Symbol's value as
> variable is void: url-show-status
>
> The first line is just from
>
>   (message "(featurep 'url-vars)=%s, url-show-status=%s" (featurep
> 'url-vars) url-show-status)
>
> The second line looks very strange to me since url-show-status was
> defined on the first line and it is defined in url-vars.el.

Try rebuilding, and then provide an exact, step-by-step recipe to
reproduce the problem, starting from `emacs -Q'.

Message #19 received at 5459 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Chong Yidong <cyd <at> stupidchicken.com>
Cc: 5459 <at> debbugs.gnu.org
Subject: Re: Should url-util require url-vars?
Date: Sun, 24 Jan 2010 23:49:23 +0100

On Sun, Jan 24, 2010 at 11:40 PM, Lennart Borgman
<lennart.borgman <at> gmail.com> wrote:
> On Sat, Jan 23, 2010 at 11:50 PM, Chong Yidong <cyd <at> stupidchicken.com> wrote:
>>> url-show-status is defined in url-vars.el.
>>> url-http-content-length-after-change-function calls
>>> url-display-percentage which needs url-show-status.
>>>
>>> url-display-percentage is defined in url-util.el. Maybe this library
>>> should require url-vars?
>>
>> Yes.  Fixed now, thanks.
>
>
> Thanks. Unfortunately there are more trouble. And I suspect serious
> trouble. Please reopen the bug.
>
>
> This is what I get in my *Messages* buffer on w32 after explicitly
> loading url-vars.el (I have not rebuilt Emacs yet with your changes):
>
>  (featurep 'url-vars)=t, url-show-status=nil
>  error in process filter:
> url-http-content-length-after-change-function: Symbol's value as
> variable is void: url-show-status
>
> The first line is just from
>
>  (message "(featurep 'url-vars)=%s, url-show-status=%s" (featurep
> 'url-vars) url-show-status)
>
> The second line looks very strange to me since url-show-status was
> defined on the first line and it is defined in url-vars.el.


Hm, my bad. I did something like this:

  (let ((url-show-status nil)) ;; do not show download messages
    (require 'url-vars)
    (message "(featurep 'url-vars)=%s, url-show-status=%s" (featurep
'url-vars) url-show-status)
    (url-retrieve ...))

This prevented url-show-status to be defined when loading url-vars.
Placing (require 'url-vars) above the let statement fixed this.

Sorry for the trouble.

Message #22 received at 5459 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: Chong Yidong <cyd <at> stupidchicken.com>, 5459 <at> debbugs.gnu.org
Subject: Re: bug#5459: Should url-util require url-vars?
Date: Sun, 24 Jan 2010 22:04:05 -0500

>   (let ((url-show-status nil)) ;; do not show download messages
>     (require 'url-vars)
>     (message "(featurep 'url-vars)=%s, url-show-status=%s" (featurep
> 'url-vars) url-show-status)
>     (url-retrieve ...))

> This prevented url-show-status to be defined when loading url-vars.
> Placing (require 'url-vars) above the let statement fixed this.

This should have given you a warning.


        Stefan

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.