Hi Konstantin,

On 06.03.2012 23:03, Konstantin Kolinko wrote:
2012/3/6<mt...@apache.org>:
Author: mturk
Date: Tue Mar  6 16:54:34 2012
New Revision: 1297571

URL: http://svn.apache.org/viewvc?rev=1297571&view=rev
Log:
Add support for lynx html to txt converter

Modified:
    tomcat/jk/trunk/tools/jkrelease.sh
    tomcat/jk/trunk/xdocs/style.xsl

Modified: tomcat/jk/trunk/tools/jkrelease.sh
URL: 
http://svn.apache.org/viewvc/tomcat/jk/trunk/tools/jkrelease.sh?rev=1297571&r1=1297570&r2=1297571&view=diff
==============================================================================
--- tomcat/jk/trunk/tools/jkrelease.sh (original)
+++ tomcat/jk/trunk/tools/jkrelease.sh Tue Mar  6 16:54:34 2012
@@ -244,8 +244,9 @@ cd ${JK_DIST}/native
  W3MOPTS="-dump -cols 80 -t 4 -S -O iso-8859-1 -T text/html"
  ELNKOPTS="-dump -dump-width 80 -dump-charset iso-8859-1 -no-numbering 
-no-home"
  LNKOPTS="-dump -width 80 -codepage iso-8859-1 -no-g -html-numbered-links 0"
+LYXOPTS="-dump -width=80 -nolist -nostatus -noprint 
-assume_local_charset=iso-8859-1"

Is typo in the variable name above a deliberate one?
(LYX vs LYNX ?)

I didn't change this line, but ...

Though I see that it follows the link->LNK, elink ->ELNK pattern.


  failed=true
-for tool in `echo "w3m elinks links"`
+for tool in `echo "w3m elinks links lynx"`
  do
   found=false
   for dir in `echo ${PATH} | sed 's!^:!.:!;s!:$!:.!;s!::!:.:!g;s!:! !g'`
@@ -270,6 +271,9 @@ do
       elinks)
         TOOL="elinks $ELNKOPTS"
         ;;
+      lynx)
+        TOOL="lynx $LYXKOPTS"

I changed this line to use LYXOPTS and not the undefined LYXKOPTS.

Note that Mladen started a discuttion about removing the whole HTML to text conversion, so all of this will probably be gone before the next release.

Thanks for your always careful review!

Regards,

Rainer



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to