[m-rev.] diff: check for lynx

Fergus Henderson fjh at cs.mu.OZ.AU
Mon Oct 27 12:07:08 AEDT 2003


On 25-Oct-2003, Peter Ross <pro at missioncriticalit.com> wrote:
> 
> Not everybody has lynx installed on their systems, so test for it.
...
> +++ Mmakefile	25 Oct 2003 15:03:17 -0000
> @@ -264,7 +264,11 @@
>  INSTALL: .INSTALL.in VERSION
>  	sed 's/@VERSION@/$(VERSION)/g' .INSTALL.in > INSTALL
>  
> +ifeq ("$(LYNX)","")
> +TEXT_TO_HTML = cat
> +else
>  TEXT_TO_HTML = TERM=vt100 lynx -dump
> +endif

This is a problem with the existing code, rather than anything you've
changed... but that variable should be named "HTML_TO_TEXT", not
"TEXT_TO_HTML".

> Index: configure.in
...
> @@ -443,6 +443,9 @@
>  esac
>  AC_SUBST(TEXI2DVI)
>  #-----------------------------------------------------------------------------#
> +AC_PATH_PROG(LYNX,lynx)
> +AC_SUBST(lynx)

That should be

	AC_SUBST(LYNX)

(note capitals).

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list