← Back to team overview

dolfin team mailing list archive

Re: [Branch ~dolfin-core/dolfin/main] Rev 5057: Collect optional packages in OPTIONAL_PACKAGES and use to set options.

 

On Mon, 2010-08-23 at 14:26 +0200, Anders Logg wrote:
> On Mon, Aug 23, 2010 at 01:10:56PM +0100, Garth N. Wells wrote:
> > On Mon, 2010-08-23 at 10:34 +0000, noreply@xxxxxxxxxxxxx wrote:
> > > ------------------------------------------------------------
> > > revno: 5057
> > > committer: Anders Logg <logg@xxxxxxxxx>
> > > branch nick: dolfin-dev
> > > timestamp: Mon 2010-08-23 12:31:46 +0200
> > > message:
> > >   Collect optional packages in OPTIONAL_PACKAGES and use to set options.
> >
> > I think that this should changed this back. It's nice to be able to
> > enable/disable packages from the GUI, and it's nice to have a
> > descriptive string.
> 
> It's still possible to change from the GUI. 

OK.


> It should be possible to
> add the strings back but they weren't very informative. They pretty
> much said the same thing all of them.

We can make them more informative.

> 
> The OPTIONAL_PACKAGES variable makes it possible to write a nice
> summary at the end. 


This isn't working properly - it converts everything to upper case, so
when it should be checking for Trilinos_FOUND and not TRILINOS_FOUND.

Garth 


> That's otherwise lost among all the various CMake
> messages.
> 
> --
> Anders





Follow ups

References