← Back to team overview

fenics team mailing list archive

Re: Cleanup of repositories

 

On Thu, Mar 21, 2013 at 10:52:54PM +0100, Martin Sandve Alnæs wrote:
>   >  b) The list contains files like Changelog, CMakeLists.txt from the
>   >  build system which obviously need to be retained, as well as meshes
>   >  used by demos.
>     Did you look at the wrong file?
>   Yes...
>     And I don't see why we need to keep the meshes. They will be
>     downloaded and bundled as part of tarballs and distributions. And
>     can
>     be easily downloaded by users who clone the repository via a simple
>     script.
>   The consequence is that when downloading an earlier dolfin version the
>   meshes may not be available or may have changed, unless we version
>   the downloadable meshes as well.

If you download a tarball for a release, then you will get everything
bundled. It is only if you do 'git clone' and then revert to an
earlier version that you run into trouble.

I'm not planning to unpack all the old tarballs and remove
stuff from them... :-)

>     I see the point but have heard different opinions on this issue.
>     More
>     opinions? We have a few files currently in dolfin/ale and many more
>     generated files in the past being part of the library.
>   Not only does it make dolfin depend on the entire fenics toolchain
>   being
>   installed, but that compatible versions of ufl/ffc are installed.
>   This makes it harder to check out and build different versions of
>   dolfin
>   in a bughunt where reverting ufl/ffc would not be necessary.

Hmm... Need to think about that.

>     My suggestion to Johannes was to make it part of calling cmake. Then
>     it gets built only once, and developers will know where to look for
>     a
>     script to rebuild them when necessary (inside cmake/scripts).
>   Ok, I actually wondered once why that wasn't the case to begin with :)
>   It can't easily know when e.g. ffc has changed though, so keeping the
>   forced rebuild script will be useful.

Yes.

--
Anders


Follow ups

References