for review: add mercury_tcltk library

Thomas Charles CONWAY conway at cs.mu.OZ.AU
Tue Feb 17 08:30:52 AEDT 1998


Tyson Dowd, you write:

> 	EXTRA_MLLIBS  = -ltk8.0 -ltcl8.0 -L/usr/X11R6/lib -lX11 -lXmu \
> 				-lXext -lm -ldl
> 
> 	# Specify the location of the `mercury_tcltk' package.
> 	MERCURY_TCLTK_DIR  = some/directory/mercury_tcltk
> 
> 	# Tell mmake to use the `mercury_tcltk' library.
> 	VPATH           = $(MERCURY_TCLTK_DIR):$(MMAKE_VPATH)
> 	MCFLAGS         = -I$(MERCURY_TCLTK_DIR) $(EXTRA_MCFLAGS)
> 	MLFLAGS         = -R$(MERCURY_TCLTK_DIR) $(EXTRA_MLFLAGS) \
> 			  -L$(MERCURY_TCLTK_DIR) 
> 	MLLIBS          = -lmercury_tcltk $(EXTRA_MLLIBS)
> 	C2INITFLAGS     = $(MERCURY_TCLTK_DIR)/mercury_tcltk.init 

Any particular reason that you include -lmercury_tcltk in the MLLIBS
but the rest of the *necessary* libraries in the EXTRA_MLLIBS?

> 
> You can then use EXTRA_MCFLAGS, EXTRA_MLFLAGS, etc to set any other
> options you need.
> 

This comment suggests that the other libs (eg -ltk8.0, etc) belong
in the MLLIBS flags.

> 
> New File: extras/graphics/mercury_tcltk/Mmakefile
> ===================================================================
> 
> # You may need to modify the line belw

s/belw/below/

> %
> % file: mtk.
> % main author: conway.

probably should add
% stability: very low

I've got most of typeclasses based version which is nicer.
It won't be ready for release until DJ has made a couple of
minor enhancements to typeclasses.

Thomas
-- 
ZZ:wq!
^X^C
Thomas Conway               				      conway at cs.mu.oz.au
AD DEUM ET VINUM	  			   Nail here [] for new monitor.



More information about the developers mailing list