[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