[Plplot-devel] top-level Makefile.am changes

[Alan W. I. <ir...@be...>]

Rafael, I am concerned about your recent change to the top-level Makefile.am
since I don't see how static drivers could possibly work with it starting
from a clean checkout with no old drivers convenience library (possibly
configured differently from what you want in the new build) left over from
the last build.

All my experiments in December indicated it was absolutely essential to
build the static drivers before libplplot (since in that case the drivers
convenience library built in drivers becomes part of libplplot). Therefore,
for static drivers the directory order in SUBDIRS *must* have drivers before
src. Your changes have broken that rule for the static drivers case (in fact
the drivers directory is not mentioned at all) so I don't see how the
drivers convenience library is going to be available to add into libplplot
when you are building libplplot.

I haven't done this myself so I may be missing something, but my prediction
is if you start from a clean checkout or run make clean before your test
(which should get rid of any old convenience library that will cover up this
error), I think you will quickly see why I am concerned.

I also don't understand your commit comment about drivers must come after
bindings since that already occurred for the previous Makefile.am for the
dynamic drivers case.  That cannot be the source of the bug you found if you
were installing dynamic drivers at the time.

Could you be more explicit about the exact error you have been trying to fix
with this change?  I may be able to find a fix that doesn't completely break
static drivers.

Alan
__________________________
Alan W. Irwin
email: ir...@be...
phone: 250-727-2902

Astronomical research affiliation with Department of Physics and Astronomy,
University of Victoria (astrowww.phys.uvic.ca).

Programming affiliations with the Canadian Centre for Climate Modelling and
Analysis (www.cccma.bc.ec.gc.ca) and the PLplot scientific plotting software
package (plplot.org).

__________________________

Linux-powered Science
__________________________

---------- Forwarded message ----------
Date: Wed, 12 Feb 2003 04:12:42 -0800
From: Rafael Laboissiere <rla...@us...>
To: plp...@li...
Subject: [Plplot-cvs] CVS update notice (plplot/doc)

Update of /cvsroot/plplot/plplot/doc
In directory sc8-pr-cvs1:/tmp/cvs-serv30113/doc

Added Files:
	Makefile.am
Log Message:
[...]
2) Changed the order of directories in SUBDIRS, by putting "drivers" after
   "bindings".  This is _very_ important, because the old oreder triggers an
   error in the very specific situation: (a) the option --prefix=/usr is
   given to configure and (b) an old version of the PLplot package is
   already installed under /usr (this happened when I tried to build the
   Debian packages).

There is also another change in Makefile.am: instead of having the list of
directories duplicated in the enable_dyndrivers conditional, only the
DRIVERSDIR variable is set and then used later when setting the value of the
SUBDIRS variable. This is a cosmetic change, I know, but let us follow
William of Occam's maxim: "Pluralitas non est ponenda sine neccesitate".



-------------------------------------------------------
This SF.NET email is sponsored by:
SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See!
http://www.vasoftware.com
_______________________________________________
Plplot-cvs mailing list
Plp...@li...
https://lists.sourceforge.net/lists/listinfo/plplot-cvs