dolfin team mailing list archive

["Johannes Ring" <johannr@xxxxxxxxx>]

On Wed, July 8, 2009 10:55, Johan Hake wrote:
> On Tuesday 07 July 2009 23:39:49 DOLFIN wrote:
>> One or more new changesets pushed to the primary dolfin repository.
>> A short summary of the last three changesets is included below.
>>
>> changeset:   6444:27b8370cf61c31e66511de5331328a4dcdfadee3
>> tag:         tip
>> user:        "Garth N. Wells <gnw20@xxxxxxxxx>"
>> date:        Tue Jul 07 22:39:36 2009 +0100
>> files:       SConstruct
>> description:
>> SConstruct fixes. More may be needed.
>
> Using PathVariable.PathIsDirCreate for all installation directories is a
> more
> natural choice as SCons will issue a warning if prefix is not a directory
> and
> will create it if is not there. However SCons make this check during
> configure (or the old scons step). If a user does not have write access to
> the handed directory, and it is not excisting, SCons configure will then
> fail.
>
> I think it is therefore we have chosen not to use
> PathVariable.PathIsDirCreate, but PathVariable.PathAccept instead. We have
> then manually added a makedir during install if the directory is not
> excisting. But I am not sure this is done for all installation
> directories.
>
> Any comments Johannes?

We use PathAccept everywhere else so I think we should use that instead of
PathIsDirCreate.

Johannes