dolfin team mailing list archive

[Anders Logg <logg@xxxxxxxxx>]

On Mon, Aug 16, 2010 at 01:37:19PM +0100, Garth N. Wells wrote:
> 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).

Should we add both the CMakeLists.txt and Makefiles files? Or do we
require CMake since that is used to build DOLFIN anyway?

There are a few scripts now that need to be run occasionally:

  scripts/compileforms
  scripts/generate_cmake
  dolfin/swig/generate.py

It would be good to agree on when these should be run and clean it up
so that they can all be run using a single script.

> 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?

I think it should be separate. I think this becomes clear if we
consider a previous discussion we had about generating the demos from
the documentation.

--
Anders

Attachment: signature.asc
Description: Digital signature