[Bug 1758626] Review Request: octave-iso2mesh - A 3D surface and volumetric mesh generator for MATLAB/Octave

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1758626



--- Comment #28 from Robert-André Mauchin <zebob.m@xxxxxxxxx> ---
You're right, Tetgen is AGPLv3 amd JMeshLib is GPLv2, but I didn't understand
it was into the resulting binary:


AGPL (v3 or later)
------------------
iso2mesh-1.9.1/tools/tetgen/LICENSE

GPL (v2 or later)
-----------------
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/binTree.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/clusterGraph.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/dijkstraGraph.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/edge.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/graph.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/heap.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/j_mesh.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/jmesh.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/jqsort.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/list.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/matrix.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/point.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/tin.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/triangle.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/include/vertex.h
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/JMESH/jmesh.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/MESH_STRUCTURE/checkAndRepair.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/MESH_STRUCTURE/edge.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/MESH_STRUCTURE/io.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/MESH_STRUCTURE/point.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/MESH_STRUCTURE/tin.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/MESH_STRUCTURE/triangle.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/MESH_STRUCTURE/vertex.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/binTree.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/clusterGraph.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/dijkstraGraph.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/graph.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/heap.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/jqsort.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/list.cpp
iso2mesh-1.9.1/tools/meshfix/contrib/JMeshLib/src/PRIMITIVES/matrix.cpp
iso2mesh-1.9.1/tools/meshfix/include/detectIntersections.h
iso2mesh-1.9.1/tools/meshfix/include/epsilonSampling.h
iso2mesh-1.9.1/tools/meshfix/include/exttrimesh.h
iso2mesh-1.9.1/tools/meshfix/include/holeFilling.h
iso2mesh-1.9.1/tools/meshfix/include/simplification.h
iso2mesh-1.9.1/tools/meshfix/include/sparseLSystem.h
iso2mesh-1.9.1/tools/meshfix/src/detectIntersections.cpp
iso2mesh-1.9.1/tools/meshfix/src/epsilonSampling.cpp
iso2mesh-1.9.1/tools/meshfix/src/holeFilling.cpp
iso2mesh-1.9.1/tools/meshfix/src/simplification.cpp
iso2mesh-1.9.1/tools/meshfix/src/smoothing.cpp
iso2mesh-1.9.1/tools/meshfix/src/sparseLSystem.cpp
iso2mesh-1.9.1/tools/meshfix/src/uniform.cpp

They should be added to the License: field 


> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/cgalmesh
> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/cgalpoly
> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/cgalsimp2
> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/cgalsurf
> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/cork
> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/jmeshlib
> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/meshfix
> octave-iso2mesh.x86_64: E: arch-dependent-file-in-usr-share
> /usr/share/octave/packages/iso2mesh-1.9.1/bin/tetgen1.5

>All those "tools" should be in /usr/libexec/iso2mesh/ instead of 
>/usr/share/octave/packages/iso2mesh-1.9.1/bin/. The mex files should be 
>able to run the tools. If you want to be lazy, just move the tools into 
>/usr/libexec/iso2mesh/ and package symbolic links in /usr/share/octave
>/packages/iso2mesh-1.9.1/bin/.

I concir, these error did not show up in my rpmlint output as seen above. They
shouldn't be in %_datadir, most likely in bindir or libexecdir.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux