graphml2gv: remove unused struct name

Merge branch 'smattr/bb628679-594c-4502-a50b-9e32e5ed7316' into 'main'

neatogen warning squashing

See merge request graphviz/graphviz!2146

neatogen multispline freeIpair: squash a -Wunused-parameter compiler warning

neatogen multispline newIpair: squash a -Wunused-parameter compiler warning

neatogen multispline freeItem: squash -Wunused-parameter compiler warnings

neatogen multipline newItem: squash a -Wunused-parameter compiler warning

neatogen multispline swap_ends_p: squash a -Wunused-parameter compiler warning

neatogen multispline spline_merge: squash a -Wunused-parameter compiler warning

neatogen multispline: squash a -Wmissing-field-initializer warning

makeObstacle: squash a spurious -Wswitch-default compiler warning

The surrounding context ensures that the cases in this switch cover every
possibility.

neatogen cmpitems: squash -Wunused-parameter compiler warnings

neatogen cmpitems: rephrase to avoid double→int conversion

This had a number of problems with respect to possible overflow and loss of
precision. This commit rephrases the comparison into something that does not
rely on any coercion or conversion.

neatogen freeitem: squash -Wunused-parameter compiler warnings

neatogen newitem: squash a -Wunused-parameter compiler warning

neatogen swap_ends_p: squash a -Wunused-parameter warning

neatogen spline_merge: squash a -Wunused-parameter compiler warning

find_closest_pairs: squash a -Wmissing-field-initializer compiler warning

neatogen splineInfo: squash a -Wmissing-field-initializer compiler warning

dotgen splineInfo: squash a -Wmissing-field-initializer compiler warning

Merge branch 'smattr/659C49B6-0A1A-4490-B1DD-DE326B38C850' into 'main'

clean up in tcldot

See merge request graphviz/graphviz!2145

tcldot-io.c: remove unnecessary bracketing

tcldot myiodisc_memiofread: remove unnecessary cast

This pointer implicitly coerces.

tcldot-io.c: remove commented out code

tcldot myiddisc_idregister: squash a spurious -Wswitch-default compiler warning

tcldot-id.c: remove unnecessary casts of state pointers

These pointers implicitly coerce.

tcldot: myiddisc_free: squash a -Wunused-parameter compiler warning

tcldot myiddisc_map: squash a -Wunused-parameter compiler warning

tcldot myagxset: use a 'size_t' for string length instead of 'int'

Squashes a -Wconversion compiler warning.

tcldot deleteNode: squash a -Wunused-parameter compiler warning

tcldot deleteEdge: squash a -Wunused-parameter compiler warning

tcldot obj2cmd: squash a spurious -Wswitch-default compiler warning

Tcldot_string_writer: remove unnecessary cast

This pointer implicitly coerces.

Merge branch 'smattr/DD0814CF-B2A0-4318-A2E6-A7F84616387A' into 'main'

fix remaining Pylint warnings; enabling gating Pylint CI check

See merge request graphviz/graphviz!2140

make final Pylint step fail for any non-FIXME warning/error

The gating Pylint check now fails if any warning or error is detected, excluding
FIXME comments. This should hopefully lead to improved code quality in Python
introduced into the code base in future.

disable Pylint duplicate-code warning

This triggers for a lot of Graphviz Python code because, by its nature of being
used to write test cases in Graphviz, there is inherently always going to be a
lot of similar looking code.

vuln.py: address a Pylint missing-function-docstring warning

shapes.py: address a Pylint missing-function-docstring warning

silence Pylint import-errors about regression_test_helpers

I do not know why, but Pylint does not understand this import while Python
itself does. I have tried different variations, but Pylint complains about all
of them.

regression_test_helpers.py: address a Pylint missing-function-docstring warning

vuln.py: remove variable shadowing

address Pylint missing-module-docstring warnings

test_regression.py: remove unused 'stdout' variable

chkPos: remove strcmp micro-optimization

Merge branch 'smattr/7890fb90-8b73-4677-a4f3-0f3a17dcf0ca' into 'main'

more SWIG bindings clean up

See merge request graphviz/graphviz!2147

OCaml bindings: fix: only pass -fpermissive when compiling C++

The `-fpermissive` flag is only valid for Objective-C and C++, but was being set
for C and C++. When compiling the C part of these bindings, this was generating
the warning:

  cc1: warning: command line option ‘-fpermissive’ is valid for C++/ObjC++ but
    not for C

This flag was introduced in 8f4667edb410a6d11b53746849304fb953b5c6ae to work
around non-conformant code generation by SWIG. In future, it might be worth
investigating whether the underlying issue has been fixed in SWIG and this work
around can be removed.

Java bindings gv_channel_writer: suppress -Wunused-parameter compiler warnings

Go bindings: suppress -Wunused-function compiler warnings

SWIG generates some functions that go unused in the Graphviz bindings. There is
no value to warning about these.

Merge branch 'smattr/ab033750-c4e7-4e83-930b-4c85fa29cdb3' into 'main'

BinaryHeap clean up

See merge request graphviz/graphviz!2148

BinaryHeap: switch 'mask' in sanity check to 0 vs 1

Using this instead of -1 vs 1 allows slightly more efficient code.

BinaryHeap: use 'size_t' to track heap sizes/positions instead of 'int'

BinaryHeap: use 'calloc' instead of 'malloc' when allocating arrays

There is not much we can do about the `realloc` calls, but we can at least do
initial allocation in a less error prone way.

BinaryHeap: rephrase 'sizeof(type)' code to 'sizeof(arr[0])'

This has no effect on functionality, but it makes it clearer what this code is
doing and makes it more robust against changing the underlying types.

BinaryHeap_realloc: rephrase division by 5 code

This makes the intent of this code clearer as well as squashing a
-Wfloat-conversion compiler warning.

Merge branch 'add-ubuntu21-04-static-build-job-to-CI' into 'main'

Add static build for Ubuntu 21.04 to CI

See merge request graphviz/graphviz!2134

CI: add ubuntu21-04-static-build job

CI: remove incorrect path to .gz file in deb build template

No .gz file is produced by the builds that use this template anyway.

ci/build.sh: add support for static builds using configure options

tclpkg: install .so files only when linking dynamically

Fixes errors like this when using static linking:

cp: cannot stat '/home/magjac/graphviz/build/lib/graphviz/ruby/libgv_ruby.so': No such file or directory

gvpack_static: force the C++ compiler to be used also for static linking

The trick to force the C++ compiler to be used for linking was applied
only for gvpack, not for gvpack_static.

Fixes many errors like this:

/usr/bin/ld: ../../lib/vpsc/.libs/libvpsc_C.a(csolve_VPSC.o): in function `newVariable':
csolve_VPSC.cpp:(.text+0x20): undefined reference to `operator new(unsigned long)'

Merge branch 'add-cygwin-cmake-build-to-ci' into 'main'

Add Cygwin CMake build to CI

See merge request graphviz/graphviz!2137

CI: add windows-cygwin-cmake-build job

CPack: silence CPACK_CYGWIN_PATCH_NUMBER not set warning

Silences this warning:

CPack Warning: CPACK_CYGWIN_PATCH_NUMBER not specified using 1

CPack: generate Cygwin binary installer on Cygwin

tools: add workaround for cpack problem with Cygwin

Copy the gml2gv executable to to the gv2gml alias instead of
symlinking it. Creating a symlink to gml2gv works fine in itself, but
results in an error like this when running cpack:

CPack Error: Problem while adding file </home/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml> to archive </home/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN.zip>, ERROR = Unable to read from file 'Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml': Couldn't list extended attributes

gv2gml is a symlink. Trying to read the extended attributes manually gives:

attr -l /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml
attr_list: Permission denied
Could not list /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml

dot: add workaround for cpack problem with Cygwin

Copy the dot executable to its alias commands instead of symlinking
it. Creating symlinks to dot works fine in itself, but results in an
error like this when running cpack:

CPack Error: Problem while adding file </cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo> to archive </cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN.zip>, ERROR = Unable to read from file 'Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo': Couldn't list extended attributes

circo is a symlink. Trying to read the extended attributes manually gives:

attr -l /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo
attr_list: Permission denied
Could not list /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo

add ci/cygwin-build.sh script

ci/build.sh: remove trailing text after actual version ID

Fixes a problem on Cygwin where 'uname -r' gives:

3.2.0(0.340/5/3)

which, when used as the name of a directory, actually represents a
directory named '3.2.0(0.340', containing a subdirectory named '5'
containing a subdirectory named '3)'.

ci/build.sh: add support for Cygwin

The /etc/os-release file does not exist on Cygwin so we use the more
reliable built-in bash variable OSTYPE instead.

plugin/core: make awk crlf agnostic when generating ps.h

Fixes errors like this with Cygwin in a crlf environment with
SHELLOPTS=igncr:

/cygdrive/c/Users/magja/graphviz/build/plugin/core/ps.h:2:1: error: missing terminating " character
    2 | "%%BeginProlog ",
      | ^~~~~~~~~~~~~~

Although not shown in the error message, this line actually contained
a carriage return after %%BeginProlog:

"%%BeginProlog\r",

common: make awk crlf agnostic when generating svgcolor_lib

Fixes errors like this with Cygwin in a crlf environment with
SHELLOPTS=igncr:

/cygdrive/c/Users/magja/graphviz/build/lib/common/common/colortbl.h:1524:2: error: missing terminating " character
 1524 | {"/svg/ ",0,255,255,255,0,0,0},
      |  ^~~~~~

Although not shown in the error message, this line actually contained
a carriage return after /svg/:

{"/svg/\r",0,255,255,255,0,0,0},

Merge branch 'smattr/C617D6FE-BA39-4D70-BF6F-7FAAFAEA2A69' into 'main'

fix: write Graphviz build version to GRAPHVIZ_VERSION file instead of VERSION

Closes #2122

See merge request graphviz/graphviz!2149

fix: write Graphviz build version to GRAPHVIZ_VERSION file instead of VERSION

C++20 standardizes a new `version` header.¹ A side effect of this is that
Autotools-driven Graphviz compilation fails when using both (1) a C++20-capable
compiler and (2) a case-insensitive file system:

  Making all in vpsc
  CXX      block.lo
  In file included from block.cpp:20:
  In file included from …/include/c++/v1/memory:667:
  In file included from …/include/c++/v1/type_traits:417:
  In file included from …/include/c++/v1/cstddef:37:
  ../../version:1:1: error: expected unqualified-id
  2.49.1~dev.20210907.1517
  ^

The core problem is that the root of the repository checkout (or unpacked
tarball) is in the compiler’s include path in order to allow including
`config.h`. But the generated `VERSION` file is also in this same directory.
Nothing in Graphviz attempts to include the C++20 `version` header, but some of
the other stdlib headers transitively include `version`.

To avoid this confusion, we rename Graphviz’ generated file.

Fixes #2122.

¹ https://en.cppreference.com/w/cpp/header/version

autogen.sh: remove computation of GRAPHVIZ_VERSION

As of commit 46fadbefa1c7810513129051c8fa01a506316e6e, this is not longer used.

Merge branch 'smattr/960bee28-a488-4e56-90bf-10b4b3408ad8' into 'main'

clean up of SWIG bindings code

See merge request graphviz/graphviz!2135

clang-format tclpkg/gv/gv.cpp

use C++ way of #including C headers in SWIG bindings

rephrase some integer to boolean conversions in the SWIG bindings code

fix comment typo in SWIG bindings code

move declarations closer to their usage in SWIG bindings code

rephrase some nullptr tests in the SWIG bindings code

remove unnecessary conditional guards in the SWIG bindings code

It is fine to reinterpret_cast nullptr.

remove an unnecessary use of C string functions in SWIG bindings code

use more modern reinterpret_cast instead of C-style casts in SWIG bindings code

remove unnecessary realloc in SWIG bindings code

This is attempting to shrink a buffer that was allocated by gvRenderData to its
tight bounds. This kind of thing is no longer a necessary optimization and, in
some cases, can be a de-optimization.

remove commented out code in SWIG bindings code

replace NULL with more modern nullptr in SWIG bindings code

Merge branch 'clean-up-build.sh' into 'main'

Clean up ci/build.sh and fix MinGW artifact archiving

See merge request graphviz/graphviz!2143

CI: archive build artifacts in windows-mingw64-cmake-build job

CI: archive build artifacts in windows-mingw32-cmake-build job

ci/build.sh: fail if trying to use an unset variable

ci/build.sh: fix missing VERSION_ID on MSYS2/MinGW

This change will allow treating unset variables as errors in an
upcoming commit.

This problem also caused the build artifacts to be moved to the wrong
directory, but that did not have any consequences since the build
artifacts are currently not archived in the CI jobs. An upcoming
commit in this series will however change that.

ci/build.sh: CMake: use fallback for unset CMAKE_OPTIONS

This is no functional change but will allow treating unset variables
as errors in an upcoming commit.

ci/build.sh: explicitly set build_system to autotools by default

This is no functional change but will allow treating unset variables
as errors in an upcoming commit.

ci/build.sh: remove obsolete workaround for missing ID_LIKE in /etc/os-release

This workaround was introduced in
029acf11d1d9a9eb20eddae84cadb61447da95f3, but was actually never
brought into use when 9ca0ccd4e368d0e9bd545e5f1698b8eef931dbbc
introduced an ASan build to CI, which used a Ubunutu docker container
instead of the native GitLab Debian Linux runner for which the
workaround was intended.

Also Fedora doesn't have ID_LIKE, but the workaround was really never
necessary for it either.

ci/build.sh: use /etc/os-release if and only if it exists

An upcoming MR will introduce Cygwin builds that also do not have
/etc/os-release.

ci/build.sh: autotools: fail if OSTYPE is unknown

ci/build.sh: autotools: use OSTYPE to determine if on GNU/Linux

OSTYPE is a built-in variable in bash and exists on all operating
systems.

As a second discriminator, use ID_LIKE to determine if on a Debian
like OS, e.g. Ubuntu.

ci/build.sh: CMake: fail if OSTYPE is unknown

ci/build.sh: CMake: use OSTYPE to determine if on GNU/Linux

OSTYPE is a built-in variable in bash and exists on all operating
systems.

As a second discriminator, use ID_LIKE to determine if on a Debian
like OS, e.g. Ubuntu.

ci/build.sh: CMake: use OSTYPE to determine if on MSYS2/MinGW

OSTYPE is a built-in variable in bash and exists on all operating
systems.