dolfin team mailing list archive

["Garth N. Wells" <gnw20@xxxxxxxxx>]

On Mon, 2010-08-16 at 14:25 +0200, Anders Logg wrote:
> On Mon, Aug 16, 2010 at 01:09:04PM +0100, Garth N. Wells wrote:
> > On Mon, 2010-08-16 at 13:58 +0200, Anders Logg wrote:
> > > On Mon, Aug 16, 2010 at 12:52:34PM +0100, Garth N. Wells wrote:
> > > > On Mon, 2010-08-16 at 13:48 +0200, Anders Logg wrote:
> > > > > On Mon, Aug 16, 2010 at 12:24:11PM +0100, Garth N. Wells wrote:
> > > > > > On Mon, 2010-08-16 at 13:10 +0200, Anders Logg wrote:
> > > > > > > On Mon, Aug 16, 2010 at 10:52:12AM +0100, Garth N. Wells wrote:
> > > > > > > > On Mon, 2010-08-16 at 11:49 +0200, Anders Logg wrote:
> > > > > > > > > On Mon, Aug 16, 2010 at 11:12:49AM +0200, Anders Logg wrote:
> > > > > > > > > > On Mon, Aug 16, 2010 at 10:11:08AM +0100, Garth N. Wells wrote:
> > > > > > > > > > > Could the two files containing a 'main' function in
> > > > > > > > > > > demo/ode/harmonic/cpp be merged, or one removed? I have written a script
> > > > > > > > > > > to generate CMakeLists.txt for all demos, but it doesn't like having two
> > > > > > > > > > > 'main' functions in .cpp files in one directory.
> > > > > > > > > >
> > > > > > > > > > Yes, that should be simple to fix. I'll take a look.
> > > > > > > > >
> > > > > > > > > Fixed.
> > > > > > > > >
> > > > > > > >
> > > > > > > > Thanks.
> > > > > > > >
> > > > > > > > Now, if you run
> > > > > > > >
> > > > > > > >   ./scripts/generate_cmake
> > > > > > > >
> > > > > > > > after installing DOLFIN, then run
> > > > > > > >
> > > > > > > >   cmake .
> > > > > > > >
> > > > > > > > in demo/
> > > > > > > >
> > > > > > > > and then
> > > > > > > >
> > > > > > > >   make
> > > > > > > >
> > > > > > > > in demo, all the demos will be built.
> > > > > > > >
> > > > > > > > Garth
> > > > > > >
> > > > > > > Doesn't work here. Is there a top-level Makefile missing in demo/?
> > > > > > >
> > > > > >
> > > > > > Yes, the CMakeFile.txt was missing. I've just added it.
> > > > >
> > > > > It looks a bit strange:
> > > > >
> > > > > # Manual list of C++ demo directories
> > > > > set(DOLFIN_DEMOS_DIRS
> > > > >   /home/garth/code/fenics/dolfin/dolfin-cmake/demo/pde/poisson/cpp
> > > > >   #/home/garth/code/fenics/dolfin/dolfin-cmake/demo/pde/elasticity/cpp
> > > > >   #/home/garth/code/fenics/dolfin/dolfin-cmake/demo/pde/simple/cpp
> > > > >   )
> > > > >
> > > >
> > > > Try again now.
> > >
> > > Much better. :-)
> > >
> >
> > It would be helpful to kick around what is the desirable approach to
> > building demos (and eventually tests and benchmarks). Do we want it as
> > part of the main build process, or a separate operation after DOLFIN has
> > been installed?
> 
> I prefer it as a separate step since there's otherwise a risk of
> integrating the build for the demos in one big non-transparent build
> system which is difficult for users to copy and use as templates.
> 

We'll add the CMakeLists.txt files to each demo dir (for now we can just
generate them using generate_cmake until we settle on the format), so
users can always compile a single demos, and they can mimic the
CMakeLists.txt file. We can also have this for the test dir (which we
don't presently have with the SCons-based system, which is a bit
annoying).

What I'm getting at is do we want to enable/disable demos from the top
level CMake configuration, and build them in the ./build  directory, or
have users navigate to ./demos and then run CMake?

Garth

> --
> Anders