From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Ted Zlatanov Newsgroups: gmane.emacs.bugs Subject: bug#17958: SHR: base handling broken (shr-parse-base, shr-expand-url) Date: Tue, 04 Nov 2014 11:44:07 -0500 Organization: =?UTF-8?Q?=D0=A2=D0=B5=D0=BE=D0=B4=D0=BE=D1=80_?= =?UTF-8?Q?=D0=97=D0=BB=D0=B0=D1=82=D0=B0=D0=BD=D0=BE=D0=B2?= @ Cienfuegos Message-ID: <8761eu7w6g.fsf@lifelogs.com> References: <8738eeconi.fsf@violet.siamics.net> <871tsirjmb.fsf@violet.siamics.net> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Trace: ger.gmane.org 1415119526 20775 80.91.229.3 (4 Nov 2014 16:45:26 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 4 Nov 2014 16:45:26 +0000 (UTC) Cc: 17958@debbugs.gnu.org To: Ivan Shmakov Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Nov 04 17:45:20 2014 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 1XlhEN-00071x-Pc for geb-bug-gnu-emacs@m.gmane.org; Tue, 04 Nov 2014 17:45:19 +0100 Original-Received: from localhost ([::1]:41687 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XlhEN-0002Vj-9T for geb-bug-gnu-emacs@m.gmane.org; Tue, 04 Nov 2014 11:45:19 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:52789) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XlhEE-0002ME-4g for bug-gnu-emacs@gnu.org; Tue, 04 Nov 2014 11:45:15 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XlhE7-0004R5-9S for bug-gnu-emacs@gnu.org; Tue, 04 Nov 2014 11:45:10 -0500 Original-Received: from debbugs.gnu.org ([140.186.70.43]:51806) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XlhE7-0004QY-56 for bug-gnu-emacs@gnu.org; Tue, 04 Nov 2014 11:45:03 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1XlhE6-0002kM-QA for bug-gnu-emacs@gnu.org; Tue, 04 Nov 2014 11:45:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Ted Zlatanov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 04 Nov 2014 16:45:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 17958 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 17958-submit@debbugs.gnu.org id=B17958.141511944310463 (code B ref 17958); Tue, 04 Nov 2014 16:45:02 +0000 Original-Received: (at 17958) by debbugs.gnu.org; 4 Nov 2014 16:44:03 +0000 Original-Received: from localhost ([127.0.0.1]:49019 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1XlhD9-0002ih-1L for submit@debbugs.gnu.org; Tue, 04 Nov 2014 11:44:03 -0500 Original-Received: from mail-qg0-f42.google.com ([209.85.192.42]:35675) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1XlhD7-0002iM-ED for 17958@debbugs.gnu.org; Tue, 04 Nov 2014 11:44:01 -0500 Original-Received: by mail-qg0-f42.google.com with SMTP id i50so9880014qgf.15 for <17958@debbugs.gnu.org>; Tue, 04 Nov 2014 08:44:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lifelogs.com; s=google; h=from:to:cc:subject:organization:references:mail-copies-to :gmane-reply-to-list:date:in-reply-to:message-id:user-agent :mime-version:content-type:content-transfer-encoding; bh=MU29flMb4zirXD8pv4s7TvGZSenOPH4L7FHz+jqVxMk=; b=oy8sB+5c6PhyjrqJchOUAt9Gcyo61Toy0ugEo5wBh/vCg+I7M4m4Wu6BHe2ghFWs6h Zk6toJ0XJaa61OAPrPOjUq4wdFx75xwCcswRCx+0aMCk8ZuGZj8oTFIV3OiYLpSDhT0Y WjT2DOVfVfrSdI1YyIvUoGqcocHqEFbUbcnsU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:organization:references :mail-copies-to:gmane-reply-to-list:date:in-reply-to:message-id :user-agent:mime-version:content-type:content-transfer-encoding; bh=MU29flMb4zirXD8pv4s7TvGZSenOPH4L7FHz+jqVxMk=; b=eE8wtlFKZEP9Dqsz28uQWzqSmqcuSWe7nnn3+jkJ6lS+H+Qrkfqj0Q11xW7DE2RrH5 oaQFoadZDSTCpFEmfifb3sIuLwRP/noc7A2zs2hNLeQWFs/LNkCfc9HWveKS6OKr5Nox v9mNP+MQWJWdtyye8UE4Fys9hH5ATymG2Gz33dTUZOVWjkHzOWKk0siIYXZZ3vznmV+o nEvZ6X4DjuiL4Og0Cc72Wz6wAHU62D/XYEYlLCwRiCnBzpn8dreyNj+/AAAmKk9kHzrZ J+XgYvnYXj5QETrBY6Ya9gNq3lbXYJ3KoMzgK3/mCSBYa9kJcFnBraVc31J84xDxmR7y S6fg== X-Gm-Message-State: ALoCoQlUKRhoDPzPhG04ZL/cbLX1E95aPp8UFnk511Sochr8tlt7IpHH4tppc16QqnLMzNZdhIqx X-Received: by 10.140.84.106 with SMTP id k97mr54161651qgd.76.1415119433420; Tue, 04 Nov 2014 08:43:53 -0800 (PST) Original-Received: from flea (c-98-229-61-72.hsd1.ma.comcast.net. [98.229.61.72]) by mx.google.com with ESMTPSA id c63sm785773qgf.14.2014.11.04.08.43.52 for (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Tue, 04 Nov 2014 08:43:52 -0800 (PST) X-Face: bd.DQ~'29fIs`T_%O%C\g%6jW)yi[zuz6; d4V0`@y-~$#3P_Ng{@m+e4o<4P'#(_GJQ%TT= D}[Ep*b!\e,fBZ'j_+#"Ps?s2!4H2-Y"sx" Mail-Copies-To: never Gmane-Reply-To-List: yes In-Reply-To: <871tsirjmb.fsf@violet.siamics.net> (Ivan Shmakov's message of "Thu, 14 Aug 2014 18:50:20 +0000") User-Agent: Gnus/5.130012 (Ma Gnus v0.12) Emacs/25.0.50 (gnu/linux) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x 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:95483 On Thu, 14 Aug 2014 18:50:20 +0000 Ivan Shmakov wrote:=20 >> Thus, I suggest changing shr-expand-url to return not the 0th element >> of the (parsed) =E2=80=98base=E2=80=99 (see below), but the 3rd. ... IS> With the patch MIMEd (which also fixes the issue described in my IS> initial bug report), it instead gives what I deem to be the IS> correct result: IS> (let ((shr-base (shr-parse-base "http://example.org/"))) IS> (shr-tag-base '((:href . "/relative"))) IS> shr-base) IS> ;; =E2=87=92 IS> ("http://example.org" "/" "http" "http://example.org/relative") This seems reasonable to me as far as usability but Lars will have to review the patch for correctness. Ted