On 2013-08-08 10:01-0600 Orion Poplawski wrote:
> Okay, next issue :)
>
> ------------------------------------------------------------------------
> r12284 | airwin | 2013-01-07 22:26:34 -0700 (Mon, 07 Jan 2013) | 13 lines
>
> Insert \$ENV{DESTDIR} appropriately (with the "$" escaped because you
> want the environment variable DESTDIR to be read at install time
> rather than cmake time) for all install(CODE ... signatures usage.
>
> This change should let "make DESTDIR=<whatever> install" work
> correctly at install time whenever the doxygen documentation and the
> docbook documentation are being installed.
>
> Only the doxygen install change has been tested, but the
> corresponding change to the docbook install should also work
> since the same CMake rules should apply.
> ------------------------------------------------------------------------
>
> This ends up adding an extra DESTDIR in the path since cmake automatically
> handles adding DESTDIR to the paths. This is with -DBUILD_DOC:BOOL=ON. I
> have not tested with PREBUILT_DOC.
>
> Reverting this commit fixes the install for me (unsurprising since 12281 was
> working fine for me before).
>From the above commit message I explicitly tested that this worked
for the doxygen case. Is that case also fine for you, i.e., is
the docbook cases the only ones where DESTDIR is applied twice?
I also notice (now) that the docbook case uses the file(INSTALL...)
signature inside the install(CODE...) command. Normally,
install(CODE...) needs explicit DESTDIR (which my above commit
supplied), but one interpretation of the documentation would imply the
case where file(INSTALL...) is used inside install(CODE...) is an
exception to that rule which would mean DESTDIR would be applied twice
but only for the docbook cases and not for the doxgyen case.
Anyhow, I will test both the doxygen install and docbook install
cases and figure out what to do.
Alan
__________________________
Alan W. Irwin
Astronomical research affiliation with Department of Physics and Astronomy,
University of Victoria (astrowww.phys.uvic.ca).
Programming affiliations with the FreeEOS equation-of-state
implementation for stellar interiors (freeeos.sf.net); the Time
Ephemerides project (timeephem.sf.net); PLplot scientific plotting
software package (plplot.sf.net); the libLASi project
(unifont.org/lasi); the Loads of Linux Links project (loll.sf.net);
and the Linux Brochure Project (lbproject.sf.net).
__________________________
Linux-powered Science
__________________________
------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
Plplot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/plplot-devel
