On Fri, Aug 20, 2010 at 10:02:58AM +0200, Johannes Ring wrote: > On Thu, Aug 19, 2010 at 9:31 PM, <nore...@launchpad.net> wrote: > > ------------------------------------------------------------ > > revno: 5032 > > committer: Anders Logg <l...@simula.no> > > branch nick: dolfin > > timestamp: Thu 2010-08-19 21:28:38 +0200 > > message: > > Add extra includes for Boost used by Armadillo test, not sure if this > > is the right way. > > I think it is better if we use the Boost_INCLUDE_DIR variable defined > by the FindBoost module. We should either call find_package(Boost) in > FindArmadillo or assume that Boost_INCLUDE_DIR has already been > defined. The latter requires the find_package(Boost...) statement (in > CMakeLists.txt) to be moved before the call to > find_package(Armadillo...).
That sounds good to me! Just go ahead an fix it. -- Anders
signature.asc
Description: Digital signature
_______________________________________________ Mailing list: https://launchpad.net/~dolfin Post to : dolfin@lists.launchpad.net Unsubscribe : https://launchpad.net/~dolfin More help : https://help.launchpad.net/ListHelp
