Bug#613568: gmsh exits successfully even when it detects meshing errors

[Jö Fahlke]

Package: gmsh
Version: 2.4.2.dfsg-5
Severity: normal

Consider the attached geometry file test.geo of a unit cube.  In this file I
forced the use of the meshing algorithm from tetgen by including the lines

  // ensure tetgen is used
  Mesh.Algorithm3D=1;

Of course, Debian's gmsh does not support tetgen, so this can not work:
======================================================================
joe at paranoia:/tmp$ gmsh test.geo -3
Info    : Running 'gmsh test.geo -3'
Info    : Started on Tue Feb 15 19:01:15 2011
Info    : Reading 'test.geo'
Info    : Read 'test.geo'
Info    : Meshing 1D...
[...]
Info    : Meshing 3D...
Error   : Tetgen is not compiled in this version of Gmsh
Info    : Mesh 3D complete (0 s)
Info    : 182 vertices 428 elements
Error   : ------------------------------
Error   : Mesh generation error summary
Error   :     0 warnings
Error   :     1 error
Error   : Check the full log for details
Error   : ------------------------------
Info    : Writing 'test.msh'
Info    : Wrote 'test.msh'
Info    : Stopped on Tue Feb 15 19:01:15 2011
joe at paranoia:/tmp$ echo $?
0
joe at paranoia:/tmp$ 
======================================================================
As you can see, gmsh does notice the error, but still goes on and saves the
mesh file and even returns a successfull exit code to the shell.

For scripting purposes it would be great if gmsh would return an non-zero
exit code in such situations.

Thanks,
Jö.

-- System Information:
Debian Release: 6.0
Architecture: i386 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/2 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages gmsh depends on:
ii  dpkg                    1.15.8.10        Debian package management system
ii  install-info            4.13a.dfsg.1-6   Manage installed documentation in 
ii  libatlas3gf-base [libla 3.8.3-27         Automatically Tuned Linear Algebra
ii  libblas3gf [libblas.so. 1.2-8            Basic Linear Algebra Reference imp
ii  libc6                   2.11.2-10        Embedded GNU C Library: Shared lib
ii  libcgns2                2.5.4-3          CFD General Notation System librar
ii  libfltk1.1              1.1.10-2+b1      Fast Light Toolkit - shared librar
ii  libgcc1                 1:4.4.5-8        GCC support library
ii  libgfortran3            4.4.5-8          Runtime library for GNU Fortran ap
ii  libgl1-mesa-glx [libgl1 7.7.1-4          A free implementation of the OpenG
ii  libglu1-mesa [libglu1]  7.7.1-4          The OpenGL utility library (GLU)
ii  libjpeg62               6b1-1            The Independent JPEG Group's JPEG 
ii  liblapack3gf [liblapack 3.2.1-8          library of linear algebra routines
ii  libopencascade-foundati 6.3.0.dfsg.1-6   OpenCASCADE CAE platform shared li
ii  libopencascade-modeling 6.3.0.dfsg.1-6   OpenCASCADE CAE platform shared li
ii  libpng12-0              1.2.44-1         PNG library - runtime
ii  libstdc++6              4.4.5-8          The GNU Standard C++ Library v3
ii  libx11-6                2:1.3.3-4        X11 client-side library
ii  zlib1g                  1:1.2.3.4.dfsg-3 compression library - runtime

Versions of packages gmsh recommends:
ii  mpi-default-bin               0.6        Standard MPI runtime programs

gmsh suggests no packages.

-- no debconf information

-- 
A mathematician is a device to turn coffee into theorems.
-- Paul Erdös
-------------- next part --------------
// ensure tetgen is used
Mesh.Algorithm3D=1;

// cube [0..1]^3
Point(1) = {0, 0, 0};
Point(2) = {1, 0, 0};
Point(3) = {0, 1, 0};
Point(4) = {1, 1, 0};
Point(5) = {0, 0, 1};
Point(6) = {1, 0, 1};
Point(7) = {0, 1, 1};
Point(8) = {1, 1, 1};
Line(1) = {1, 2};
Line(2) = {3, 4};
Line(3) = {1, 3};
Line(4) = {2, 4};
Line(5) = {5, 6};
Line(6) = {7, 8};
Line(7) = {5, 7};
Line(8) = {6, 8};
Line(9) = {1, 5};
Line(10) = {2, 6};
Line(11) = {3, 7};
Line(12) = {4, 8};
Line Loop(13) = {1, 4, -2, -3};
Plane Surface(14) = {13};
Line Loop(15) = {5, 8, -6, -7};
Plane Surface(16) = {15};
Line Loop(17) = {1, 10, -5, -9};
Plane Surface(18) = {17};
Line Loop(19) = {2, 12, -6, -11};
Plane Surface(20) = {19};
Line Loop(21) = {3, 11, -7, -9};
Plane Surface(22) = {21};
Line Loop(23) = {4, 12, -8, -10};
Plane Surface(24) = {23};
Surface Loop(25) = {14, 18, 24, 20, 16, 22};
Volume(26) = {25};
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 828 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/debian-science-maintainers/attachments/20110215/048a0948/attachment.pgp>