From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: upgradingdave Newsgroups: gmane.emacs.bugs Subject: bug#12407: url-retrieve-synchronously fails when passed a vector as URL argument Date: Wed, 12 Sep 2012 08:24:27 -0400 Message-ID: References: NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: multipart/alternative; boundary=047d7b66f57d930ef604c9804532 X-Trace: ger.gmane.org 1347452737 17533 80.91.229.3 (12 Sep 2012 12:25:37 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Wed, 12 Sep 2012 12:25:37 +0000 (UTC) Cc: 12407@debbugs.gnu.org To: Glenn Morris Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Wed Sep 12 14:25:38 2012 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1TBm0g-0003Km-4t for geb-bug-gnu-emacs@m.gmane.org; Wed, 12 Sep 2012 14:25:38 +0200 Original-Received: from localhost ([::1]:46428 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TBm0b-0001JN-RI for geb-bug-gnu-emacs@m.gmane.org; Wed, 12 Sep 2012 08:25:33 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:56937) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TBm0Z-0001JB-3H for bug-gnu-emacs@gnu.org; Wed, 12 Sep 2012 08:25:32 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TBm0L-00083a-Uv for bug-gnu-emacs@gnu.org; Wed, 12 Sep 2012 08:25:31 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:46589) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TBm0L-00083A-Qn for bug-gnu-emacs@gnu.org; Wed, 12 Sep 2012 08:25:17 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.72) (envelope-from ) id 1TBm14-0007az-DT for bug-gnu-emacs@gnu.org; Wed, 12 Sep 2012 08:26:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: upgradingdave Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 12 Sep 2012 12:26:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 12407 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 12407-submit@debbugs.gnu.org id=B12407.134745275129180 (code B ref 12407); Wed, 12 Sep 2012 12:26:02 +0000 Original-Received: (at 12407) by debbugs.gnu.org; 12 Sep 2012 12:25:51 +0000 Original-Received: from localhost ([127.0.0.1]:56135 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1TBm0M-0007Zy-QI for submit@debbugs.gnu.org; Wed, 12 Sep 2012 08:25:23 -0400 Original-Received: from mail-ey0-f172.google.com ([209.85.215.172]:39293) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1TBm0H-0007Zn-Sw for 12407@debbugs.gnu.org; Wed, 12 Sep 2012 08:25:16 -0400 Original-Received: by eaai11 with SMTP id i11so773284eaa.3 for <12407@debbugs.gnu.org>; Wed, 12 Sep 2012 05:24:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; bh=L4YsgrfE7e3bAQI7Y++jqWZABW7dPAxl6YKW8xHQNIg=; b=Wxcz7fo5o3x+zs0M5/P0KLItMnz+P8gdKv8OZ2vOpnU21awdDsT61CZmhuc66v2sM/ Sv+falP2P7VNoPZTBJwVa3I6biY4swwGLHdPJiyylyCdwYasQKipIGn79+HxdVGvUzbQ mKuuP92SmmRkT8TDKq7UBDmYdkog/K7LHY75wX+nZrwF8z2WD2bd6uyBcSAQvqsg6jas B5U8WJjcIgiRDw0UyH//cBzKK2x1Ep/xGSb48IrHFD5NBJ5PQrUMqlwU1rr3boOmuuvK Or+nOjWq5IeUka4Hf849VDPSn9NfO3x2sF5fQHoQb7prTYaDkkLq1QhHAetr7tmzomMp qJJA== Original-Received: by 10.14.221.197 with SMTP id r45mr30771541eep.41.1347452667104; Wed, 12 Sep 2012 05:24:27 -0700 (PDT) Original-Received: by 10.14.179.196 with HTTP; Wed, 12 Sep 2012 05:24:27 -0700 (PDT) In-Reply-To: X-Google-Sender-Auth: nKHqbNHRkq6aPczR7fZxlpzd8EA X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.13 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:64181 Archived-At: --047d7b66f57d930ef604c9804532 Content-Type: text/plain; charset=ISO-8859-1 Thanks for checking into this so quickly. The computer where I saw the problem was using the latest emacs-snapshot Ubuntu RPM package and so I bet that is slightly behind the current trunk. I'll check current trunk before sending a bug report next time. Thanks also for the tips for submitting bug reports, I'll include a stack trace as well as more context next time! Dave On Wed, Sep 12, 2012 at 3:47 AM, Glenn Morris wrote: > upgradingdave wrote: > > > `url-retrieve-synchronously` docs claims to accept a parsed URL vector as > > an argument, but in latest version of emacs 24.1.50 it seems to break > when > > passed a URL vector. For example, this throws an error: > > > > (url-retrieve-synchronously > > (url-generic-parse-url > > (concat "http://search.maven.org" "/solrsearch/select?q=\"" > > "log4j" "\"&rows=20&wt=json"))) > > Thanks for the report, but this works fine in the current trunk. > > > I think the problem is the following (line 185 of url.el.gz) should > > first check to make sure the URL is not a vector: > > > > (setq url (url-encode-url url)) > > Such a check was added back in June: > > http://lists.gnu.org/archive/html/emacs-diffs/2012-06/msg00249.html > > It seems your sources must be rather out-of-date (they claim to date > from the end of July). > > > This is my first time submitting a bug report, so if you need any more > > info, please let me know. > > For your next bug report; "throws an error" is vague. Please include the > exact error message (ideally set debug-on-error t and post the > backtrace). Also, saying "line X of file Y" is vague, because we don't > know exactly what your copy of file Y looks like. It is better to quote > a few lines of context. Or even better, send a patch! :) > > > In GNU Emacs 24.1.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.2.0) > > of 2012-07-28 on meitnerium, modified by Debian > -- Preferred Version LLC m. 703.989.3562 --047d7b66f57d930ef604c9804532 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Thanks for checking into this so quickly. The computer where I saw the prob= lem was using the latest emacs-snapshot Ubuntu RPM package and so I bet tha= t is slightly behind the current trunk.=A0

I'll chec= k current trunk before sending a bug report next time.

Thanks also for the tips for submitting bug reports, I&= #39;ll include a stack trace as well as more context next time!
<= br>
Dave

On Wed, Sep 12, 2012 a= t 3:47 AM, Glenn Morris <rgm@gnu.org> wrote:
upgradingdave wrote:

> `url-retrieve-synchronously` docs claims to accept a parsed URL vector= as
> an argument, but in latest version of emacs 24.1.50 it seems to break = when
> passed a URL vector. For example, this throws an error:
>
> (url-retrieve-synchronously
> =A0 =A0 =A0 =A0 (url-generic-parse-url
> =A0 =A0 =A0 =A0 =A0(concat "http://search.maven.org" "/solrsearch/select= ?q=3D\""
> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0"log4j" "\"&= ;rows=3D20&wt=3Djson")))

Thanks for the report, but this works fine in the current trunk.

> I think the problem is the following (line 185 of url.el.gz) should > first check to make sure the URL is not a vector:
>
> =A0 (setq url (url-encode-url url))

Such a check was added back in June:

=A0 =A0 http://lists.gnu.org/archive/html/emacs-diff= s/2012-06/msg00249.html

It seems your sources must be rather out-of-date (they claim to date
from the end of July).

> This is my first time submitting a bug report, so if you need any more=
> info, please let me know.

For your next bug report; "throws an error" is vague. Please incl= ude the
exact error message (ideally set debug-on-error t and post the
backtrace). Also, saying "line X of file Y" is vague, because we = don't
know exactly what your copy of file Y looks like. It is better to quote
a few lines of context. Or even better, send a patch! :)

> In GNU Emacs 24.1.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.2.0)
> =A0of 2012-07-28 on meitnerium, modified by Debian



--
Preferred Ve= rsion LLC
m. 703.989.3562
--047d7b66f57d930ef604c9804532--