Bug#517315: w3m: Broken links in manual

2013-12-27 Thread Tatsuya Kinoshita
tags 517315 + pending
thanks

On December 26, 2013 at 2:11PM +0100, rafael (at laboissiere.net) wrote:
 0001-Avoid-compression-of-README.func-and-get-the-link-in.patch

Applied.

 0002-Fix-links-to-the-keymap-examples-in-MANUAL.html.patch

Fixed with symlinks instead.

 0003-Fix-links-to-the-local-CGI-examples-in-MANUAL.html.patch

Fixed with removing links instead.

Will be closed in the next upload.

Thanks,
--
Tatsuya Kinoshita


pgphIb8fcGMag.pgp
Description: PGP signature


Bug#517315: w3m: Broken links in manual

2013-12-26 Thread Rafael Laboissiere

Control: tags -1 patch

Please, find below three patches that fix Bug#517315.  They apply cleanly 
to the current debian branch of the Git repository at alioth.d.o.


The links to the local CGI examples cannot be really fixed, since 
there is no way to get the a href=... links to use the POST method 
instead of the GET method, what makes the CGI scripts w3mbookmark and 
w3mhelperpanel fail.  I implemented another approach, namely by invoking 
two new CGI scripts, called add-bookmark and helper-app, that use 
W3m-control: headers.


Best,

Rafael Laboissiere





From b0a22bf7ff0fef6e0a8baa1362098ede26398ef8 Mon Sep 17 00:00:00 2001
From: Rafael Laboissiere raf...@laboissiere.net
Date: Thu, 26 Dec 2013 13:10:41 +0100
Subject: [PATCH 1/3] Avoid compression of README.func and get the link in
 MANUAL.html working

---
 debian/rules | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/rules b/debian/rules
index a3d4e40..2a1c2b5 100755
--- a/debian/rules
+++ b/debian/rules
@@ -94,7 +94,7 @@ binary-arch: install
 #	dh_installman
 	dh_installchangelogs -a $(builddir)/ChangeLog
 	dh_strip -a
-	dh_compress -a
+	dh_compress -a -XREADME.func
 	dh_lintian -a
 	dh_fixperms -a -Xw3mimgdisplay
 	dh_installdeb -a
-- 
1.8.5.1

From af0d5705d0abd90135f18c820c4ab13580d7c01b Mon Sep 17 00:00:00 2001
From: Rafael Laboissiere raf...@laboissiere.net
Date: Thu, 26 Dec 2013 13:11:47 +0100
Subject: [PATCH 2/3] Fix links to the keymap examples in MANUAL.html

---
 debian/patches/300_keymap-links.patch | 18 ++
 debian/patches/series |  1 +
 2 files changed, 19 insertions(+)
 create mode 100644 debian/patches/300_keymap-links.patch

diff --git a/debian/patches/300_keymap-links.patch b/debian/patches/300_keymap-links.patch
new file mode 100644
index 000..214a1e6
--- /dev/null
+++ b/debian/patches/300_keymap-links.patch
@@ -0,0 +1,18 @@
+Description: Fix links to the keymap examples in the manual
+Author: Rafael Laboissiere raf...@laboissiere.net
+Bug-Debian: http://bugs.debian.org/517315
+Last-Update: 2013-12-26
+
+--- w3m-0.5.3.orig/doc/MANUAL.html
 w3m-0.5.3/doc/MANUAL.html
+@@ -457,8 +457,8 @@ by describing ~/.w3m/keymap. For example
+ binds `NEXT_PAGE' function (normally bound to SPC and C-v)
+ to control-o. See a href=README.funcREADME.func/a for
+ list of available functions. Original and Lynx-like keymap
+-definitions are provided (a href=keymap.defaultkeymap.default/a
+-and a href=keymap.lynxkeymap.lynx/a) as examples.
++definitions are provided (a href=examples/keymap.defaultkeymap.default/a
++and a href=examples/keymap.lynxkeymap.lynx/a) as examples.
+ 
+ hr
+ a name=LocalCGI/a
diff --git a/debian/patches/series b/debian/patches/series
index 38de3ff..191ac7b 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -28,4 +28,5 @@
 270_refresh-url.patch
 280_search-next.patch
 290_closedir.patch
+300_keymap-links.patch
 900_ChangeLog.patch
-- 
1.8.5.1

From e4232cbfbe6d18f3180a09f393c9fd62aff6ffd8 Mon Sep 17 00:00:00 2001
From: Rafael Laboissiere raf...@laboissiere.net
Date: Thu, 26 Dec 2013 13:13:45 +0100
Subject: [PATCH 3/3] Fix links to the local CGI examples in MANUAL.html

---
 debian/add-bookmark  |  5 +
 debian/helper-app|  5 +
 debian/patches/310_local-cgi-links.patch | 27 +++
 debian/patches/series|  1 +
 debian/rules |  2 ++
 5 files changed, 40 insertions(+)
 create mode 100755 debian/add-bookmark
 create mode 100755 debian/helper-app
 create mode 100644 debian/patches/310_local-cgi-links.patch

diff --git a/debian/add-bookmark b/debian/add-bookmark
new file mode 100755
index 000..77cf6ea
--- /dev/null
+++ b/debian/add-bookmark
@@ -0,0 +1,5 @@
+#!/bin/sh
+
+echo Content-Type: text/plain
+echo W3m-control: BACK
+echo W3m-control: ADD_BOOKMARK
diff --git a/debian/helper-app b/debian/helper-app
new file mode 100755
index 000..478d7ec
--- /dev/null
+++ b/debian/helper-app
@@ -0,0 +1,5 @@
+#!/bin/sh
+
+echo Content-Type: text/plain
+echo W3m-control: BACK
+echo W3m-control: OPTIONS
diff --git a/debian/patches/310_local-cgi-links.patch b/debian/patches/310_local-cgi-links.patch
new file mode 100644
index 000..f2adf7e
--- /dev/null
+++ b/debian/patches/310_local-cgi-links.patch
@@ -0,0 +1,27 @@
+Description: Fix links to the local CGI examples in the manual
+Author: Rafael Laboissiere raf...@laboissiere.net
+Bug-Debian: http://bugs.debian.org/517315
+Last-Update: 2013-12-26
+
+--- w3m-0.5.3.orig/doc/MANUAL.html
 w3m-0.5.3/doc/MANUAL.html
+@@ -466,8 +466,8 @@ and a href=examples/keymap.lynxkeyma
+ You can run CGI scripts using w3m, without any HTTP server.
+ It means that w3m behaves like an HTTP server and activates CGI script,
+ then w3m reads the output of the script and display it. The 
+-a href=file:///$LIB/w3mbookmark?mode=panelbmark=~/.w3m/bookmark.htmlurl=MANUAL.htmltitle=w3m+manualbookmark registration/a

Bug#517315: w3m: Broken links in manual

2009-02-26 Thread Rafael Laboissiere
Package: w3m
Version: 0.5.2-2+b1
Severity: minor


The w3m manual in /usr/share/doc/w3m/MANUAL.html contains some broken links:

a href=README.funcREADME.func/a

This one is easy to fix: just pass -XREADME.func to dh_compress in
debian/rules.

a href=keymap.defaultkeymap.default/a
a href=keymap.lynxkeymap.lynx/a

This ones are also easy, just place keymap.default and keymap.lynx in
/usr/share/doc/w3m/, not in /usr/share/doc/w3m/examples/.  Note that you
will have to change the text in README.Debian accordingly.

a 
href=file:///$LIB/w3mbookmark?mode=panelbmark=~/.w3m/bookmark.htmlurl=MANUAL.htmltitle=w3m+manualbookmark
 registration/a
a href=file:///$LIB/w3mhelperpanel?mode=panelhelper-app editor/a

These are tricky to fix, because the cookie= argument of the CGI query is
lacking.  No idea how it can be done.

Rafael Laboissiere

-- System Information:
Debian Release: 5.0
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 2.6.26-1-686 (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages w3m depends on:
ii  libc6  2.7-18GNU C Library: Shared libraries
ii  libgc1c2   1:6.8-1.1 conservative garbage collector for
ii  libgpm21.20.4-3.1General Purpose Mouse - shared lib
ii  libncurses55.7+20081213-1shared libraries for terminal hand
ii  libssl0.9.80.9.8g-15 SSL shared libraries
ii  zlib1g 1:1.2.3.3.dfsg-12 compression library - runtime

Versions of packages w3m recommends:
ii  ca-certificates   20080809   Common CA certificates

Versions of packages w3m suggests:
ii  man-db2.5.2-4on-line manual pager
ii  menu  2.1.41 generates programs menu for all me
pn  migemonone (no description available)
ii  mime-support  3.44-1 MIME files 'mime.types'  'mailcap
pn  w3m-elnone (no description available)
pn  w3m-img   none (no description available)

-- no debconf information




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org