[bug#65357] [PATCH] gnu: scilab: Fix compilation error.

[Andreas Enge]

Am Thu, Aug 17, 2023 at 08:42:12PM +0200 schrieb Andreas Enge:
> the fix looks right to me. But our scilab version is 8 years old, while there
> are newer versions around. Would it make sense to rather update the package
> to a newer version? (I had a look at 5.2.2, 6.0.0 and 6.1.1, but all of them
> would still require your suggested fix.)

I have pushed the fix and updated scilab to 5.2.2.

I have also tried 6.1.1, and attached is how far I got.
The lines for including strings.h instead of string.h are a failed attempt
to have strcasecmp defined; with and without it the error message is
In file included from ../../modules/ast/includes/types/arrayof.hxx:33,
                 from ../../modules/ast/includes/types/double.hxx:28,
                 from ../../modules/ast/includes/symbol/variables.hxx:24,
                 from ../../modules/ast/includes/symbol/context.hxx:25,
                 from ../../modules/ast/includes/ast/runvisitor.hxx:19,
                 from ../../modules/ast/includes/ast/execvisitor.hxx:19,
                 from src/cpp/funcmanager.cpp:31:
src/cpp/funcmanager.cpp: In member function ‘bool FuncManager::AppendModules()’:
../../modules/string/includes/os_string.h:107:25: error: ‘strcasecmp’ was not 
declared in this scope; did you mean ‘wcscasecmp’?
  107 | #define stricmp         strcasecmp
      |                         ^~~~~~~~~~
src/cpp/funcmanager.cpp:134:9: note: in expansion of macro ‘stricmp’
  134 |     if (stricmp("utf-8", encoding) == 0)
      |         ^~~~~~~

The file looks unchanged in the latest git commit of scilab, and I have not
seen a corresponding patch in the Debian package, so I have given up.

Andreas

scilab.diff
Description: Text document