← Back to team overview

dolfin team mailing list archive

Re: [HG DOLFIN] merge

 

On Mon, May 11, 2009 at 07:44:29AM +0200, Johan Hake wrote:
> On Sunday 10 May 2009 23:47:47 Anders Logg wrote:
> > On Sun, May 10, 2009 at 11:42:50PM +0200, DOLFIN wrote:
> > > changeset:   6120:76bf86cf514eb0dbc88861c3ae3e6f113a80141a
> > > parent:      6117:84edb07b28286a6ca33c399ad30d32b634b52ea9
> > > user:        Anders Logg <logg@xxxxxxxxx>
> > > date:        Sun May 10 23:40:32 2009 +0200
> > > files:       dolfin/parameter/NewParameter.cpp
> > > dolfin/parameter/NewParameter.h dolfin/parameter/NewParameters.cpp
> > > dolfin/parameter/NewParameters.h sandbox/misc/cpp/main.cpp description:
> > > Implement parsing of command-line options using boost::program_options
> >
> > This seems to work nicely and didn't require that much
> > code. Parameters can now be read from the command line with
> 
> Works fine. How do we set the nested parameters? 

Don't know yet. There is some support for it in po but I haven't
figured it out yet.

> Does po comes with some help functionality, so 
> 
>   ./test --help 
> 
> can print the available options?

Yes, I've just added this.

But I'm not sure what to do when this option is given. Right now, the
help text is printed but the program does not exit. Suggestions?

> >   parameters.read(argc, argv);
> >
> > Should it be parameters.parse(argc, argv)?
> 
> I vote for this.

I've changed it.

-- 
Anders


> Johan
> 
> > This also makes DOLFIN depend on Program Options from Boost.
> > For Debian/Ubuntu, the following package is required:
> >
> >   libboost-program-options-dev
> >
> > The following must also be added when linking:
> >
> >   -lboost_program_options
> >
> > Could someone add this to the build system?
> 
> 

Attachment: signature.asc
Description: Digital signature


Follow ups

References