dolfin team mailing list archive

["Åsmund Ødegård" <aasmund@xxxxxxxxx>]

On Tue, Apr 1, 2008 at 9:54 AM, Johan Hake <hake@xxxxxxxxx> wrote:
>
> On Tuesday 01 April 2008 09:32:31 Åsmund Ødegård wrote:
>  > On Tue, Apr 1, 2008 at 9:23 AM, Johannes Ring <johannr@xxxxxxxxx> wrote:
>  > > On Tue, April 1, 2008 Ola Skavhaug wrote:
>  > >  > Joachim B Haga skrev den 31/03-2008 følgende:
>  > >  >> I'll report this for reference, but it seems to be a problem in
>  > >  >> scons.
>  > >  >>
>  > >  >> $ sudo checkinstall scons install
>  > >  >> [...]
>  > >  >> Install file: "dolfin/dolfin.h" as "/usr/include/dolfin.h"
>  > >  >> scons: *** DirNodeInfo instance has no attribute 'csig'
>  > >  >> scons: building terminated because of errors.
>  > >  >>
>  > >  >> I get the same error when not using checkinstall, but not with
>  > >  >> scons.local.
>  > >  >>
>  > >  >> This is probably related to
>  > >  >> http://scons.tigris.org/issues/show_bug.cgi?id=1953
>  > >  >> but I get it all the time and even when not using -jN.
>  > >  >
>  > >  > I get it as well. Started when upgrading my Ubuntu to hardy.
>  > >
>  > >  SCons 0.98 was released today. Perhaps this issue is fixed there?
>  >
>  > I am evaluating the new SCons release - the support for swig is
>  > completely reworked in this release. Does anybody have a concrete
>  > simple example of changes that should trigger a regeneration of
>  > dolfin_wrap.cc but fails to do so with the swig-support we have today?
>
>  Our swigscanner only scan files with .i sufix. So any changes that were made
>  in any header files that should imply a rebuild of the swig wrapper, were
>  actually filtered out by our swig scanner.

Aha - I thought that was the change Arve did - searching .h files as
well. Anyway, the new version in SCons is supposed to follow every
include, import and extern, both .i and .h (if I got it right).

>
>  Ola recently renamed dolfin_headers.h to dolfin_headers.i. Has anyone
>  encoutered the rebuild problem after that?

If that was done before yesterday, yes.

Åsmund