[petsc-dev] Reminder: *every* symbol with extern linkage *must* be namespaced

[Matthew Knepley]

On Mon, Mar 19, 2012 at 9:56 AM, Satish Balay <balay at mcs.anl.gov> wrote:

> On Sat, 17 Mar 2012, Matthew Knepley wrote:
>
> > On Sat, Mar 17, 2012 at 4:08 PM, Jed Brown <jedbrown at mcs.anl.gov> wrote:
> >
> > > On Sat, Mar 17, 2012 at 16:04, Matthew Knepley <knepley at gmail.com>
> wrote:
> > >
> > >> On Sat, Mar 17, 2012 at 3:49 PM, Barry Smith <bsmith at mcs.anl.gov>
> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/petsc-dev/attachments/20120319/358e55ec/attachment.html>