On Mon, Mar 19, 2012 at 9:56 AM, Satish Balay <[email protected]> wrote:

On Sat, 17 Mar 2012, Matthew Knepley wrote:

...

On Sat, Mar 17, 2012 at 4:08 PM, Jed Brown <[email protected]> wrote:

...

On Sat, Mar 17, 2012 at 16:04, Matthew Knepley <[email protected]> wrote:

...

On Sat, Mar 17, 2012 at 3:49 PM, Barry Smith <[email protected]> wrote:

...

It should be done as a --download-tetgen

I don't want that --with-gnu-copyright approach reintroduced.

1) I already fixed these symbols hours ago. Pull your fucking repository.

2) Turn it on with a download option. I don't give a shit.

This has nothing to do with namespaces, it has to do with software licenses. You just included a bunch of source that has a license which is far more restrictive than the PETSc license. You can't do that (while keeping the PETSc license) without making its use optional. Barry wants that done via an external package instead of an option within the PETSc source tree.

It does not fucking matter whether its there to begin with or you download it into the tree with HTTP, and since I am maintaining it, I want it there to begin with.

I think it matters. When the user uses --download-package - he is explicitly requesting the package - and can aussme the licence is different.

I will only turn it on if they give the download argument.

On the other hand - if you've are in a position to change its license [to the part thats in petsc] to be same as petsc license - then its fine.

BTW: libfast in: /home/balay/spetsc/src/dm/impls/complex complex.c: In function ‘DMComplexGenerate_CTetgen’: complex.c:4128:65: warning: variable ‘face’ set but not used [-Wunused-but-set-variable] ctetgen.c: In function ‘TetGenMeshBTreeSort’: ctetgen.c:10050:20: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] ctetgen.c:10070:21: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]

I can't change this, since this is what the package does. I guess they get away with it now. Matt ctetgen.c: In function ‘TetGenMeshFindDirection2’:

ctetgen.c:13153:49: warning: variable ‘cop’ set but not used [-Wunused-but-set-variable] ctetgen.c: In function ‘TetGenMeshConstrainedFacets’: ctetgen.c:15042:22: warning: variable ‘delaunayflag’ set but not used [-Wunused-but-set-variable] ctetgen.c: In function ‘TetGenMeshConstrainedFacets2’: ctetgen.c:15340:22: warning: variable ‘delaunayflag’ set but not used [-Wunused-but-set-variable] ctetgen.c: In function ‘TetGenMeshRepairEncSubs’: ctetgen.c:18617:8: warning: variable ‘oldptnum’ set but not used [-Wunused-but-set-variable] ctetgen.c: In function ‘TetGenMeshOptimize’: ctetgen.c:19734:18: warning: variable ‘maxdihed’ set but not used [-Wunused-but-set-variable] ctetgen.c: At top level: ctetgen.c:2026:23: warning: ‘ListHasItem’ defined but not used [-Wunused-function]

libfast in: /home/balay/spetsc/src/dm/impls/complex/ftn-auto

libfast in: /home/balay/spetsc/src/dm/impls/shell

Satish

-- What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead. -- Norbert Wiener