remove Fedora 32 CI testing

Fedora 32 was EOLed in May 2021.¹

¹ https://fedoraproject.org/wiki/End_of_life?rd=LifeCycle/EOL

Merge branch 'smattr/C574183C-758B-4503-AFE2-3BEE5F2BBA87' into 'main'

some Pylint warning squashing

See merge request graphviz/graphviz!2118

fix some missing Python docstrings

squash Pylint warnings about shadowing 'root'

squash Pylint warnings about shadowing 'p'

squash Pylint wrong-import-position warnings

squash a Pylint line-too-long warning

Merge branch 'smattr/8D176743-758B-4021-8E02-E18155F7503A' into 'main'

smyrna: remove expat installation when using MS Build

See merge request graphviz/graphviz!2115

smyrna: remove expat installation when using MS Build

AFAICT Smyrna does not directly depend on expat. It depends on it indirectly,
but should then pick up expat installation steps from dependent MS Build files.

Merge branch 'add-mingw-cmake-builds-to-ci' into 'main'

Add MinGW CMake build jobs to CI

See merge request graphviz/graphviz!2107

CI: add windows-mingw64-cmake-build job

CI: add windows-mingw32-cmake-build job

add ci/mingw-build.sh script

ci/build.sh: CMake: move the MinGW zip and exe build artifacts to the release directory

remove import/export declaration of lt_preloaded_sybols in gvcext.h

lt_preloaded_symbols is always defined in the executable that links to
the gvc lib, so it should only have a standard extern declaration.

Fixes this error with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: cmd/dot/CMakeFiles/dot.dir/dot.c.obj:dot.c:(.text+0x1e9): undefined reference to `__imp_lt_preloaded_symbols'

Also removes this warning with MinGw:

C:/Users/magja/graphviz/cmd/dot/no_builtins.c:13:14: warning: 'lt_preloaded_symbols' redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]
   13 | lt_symlist_t lt_preloaded_symbols[] = { { 0, 0 } };
      |              ^~~~~~~~~~~~~~~~~~~~

Also removes these three warnings in standard Windows MSVC builds:

C:\Users\magja\graphviz\cmd\dot\no_builtins.c(13,37): warning C4273: 'lt_preloaded_symbols': inconsistent dll linkage [C:\Users\magja\graphviz\build\cmd\dot\dot.vcxproj]
C:\Users\magja\graphviz\cmd\tools\gvpack.c(34,37): warning C4273: 'lt_preloaded_symbols': inconsistent dll linkage [C:\Users\magja\graphviz\build\cmd\tools\gvpack.vcxproj]
LINK : warning LNK4217: symbol 'lt_preloaded_symbols' defined in 'no_builtins.obj' is imported by 'dot.obj' in function 'main' [C:\Users\magja\graphviz\build\cmd\dot\dot.vcxproj]

label: CMake: ensure label symbols are exported from the gvc lib

The object modules from the static label lib are statically linked
into the gvc shared lib through the common object library which means
that their symbols should be exported from gvc instead of
imported. Defining GVC_EXPORTS achieves this.

This is what the autotools build already does.

Fixes this error with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../label/liblabel.a(xlabels.c.obj):xlabels.c:(.text+0x98): undefined reference to `__imp_zmalloc'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../label/liblabel.a(xlabels.c.obj):xlabels.c:(.text+0x1f40): undefined reference to `__imp_zmalloc'

ortho: CMake: ensure ortho symbols are exported from gvc

The object modules from the static ortho lib are statically linked
into the gvc shared lib which means that its symbols should be
exported from gvc instead of imported. Defining GVC_EXPORTS achieves
this.

Fixes errors like this with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../ortho/CMakeFiles/ortho_obj.dir/fPQ.c.obj:fPQ.c:(.text+0x6d): undefined reference to `__imp_gcalloc'

common: ensure correct import/export declarations of expat with MinGW

The expat library does not seem to recognize MinGW and therefore does
not apply the Microsoft storage class attributes to its function
declarations when using MinGW. Defining XML_USE_MSC_EXTENSIONS forces
it to use those.

Fixes errors like this with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../common/CMakeFiles/common_obj.dir/htmllex.c.obj:htmllex.c:(.text+0x1cec): undefined reference to `XML_ParserCreate'

CMake: fix expat runtime library not found by cpack on msys2

Fixes these errors with MinGW:

CMake Error at C:/Users/magja/graphviz/build/lib/gvc/cmake_install.cmake:75 (file):
  file INSTALL cannot find
  "C:/Users/magja/graphviz/lib/gvc/EXPAT_RUNTIME_LIBRARIES-NOTFOUND": No
  error.
Call Stack (most recent call first):
  C:/Users/magja/graphviz/build/lib/cmake_install.cmake:70 (include)
  C:/Users/magja/graphviz/build/cmake_install.cmake:58 (include)

CMake Error at C:/Users/magja/graphviz/build/cmd/tools/cmake_install.cmake:269 (file):
  file INSTALL cannot find
  "C:/Users/magja/graphviz/cmd/tools/EXPAT_RUNTIME_LIBRARIES-NOTFOUND": No
  error.
Call Stack (most recent call first):
  C:/Users/magja/graphviz/build/cmd/cmake_install.cmake:44 (include)
  C:/Users/magja/graphviz/build/cmake_install.cmake:60 (include)

CPack Error: Error when generating package: Graphviz

gvc: WIN32: install expat runtime library only if we have it

gvc: CMake: link to ltdl even on Windows with MinGW

Fixes these errors with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x4f7): undefined reference to `lt_dlinit'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x529): undefined reference to `lt_dlopen'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x578): undefined reference to `lt_dlerror'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x69e): undefined reference to `lt_dlsym'
collect2.exe: error: ld returned 1 exit status

CMake: FindLTDL: ensure ltdl is found with MinGW

CMake defines both WIN32 and MINGW when the compiler is some version
of MinGW. With MinGW we do want to use the ltdl library athough on
native Windows we only use the ltdl compatibility header introduced in
d70cdba7937724c19393d97bd280320cdf8c3c8f, not the library itself.

Fixes the following errors with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x4f7): undefined reference to `lt_dlinit'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x529): undefined reference to `lt_dlopen'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x578): undefined reference to `lt_dlerror'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x69e): undefined reference to `lt_dlsym'

Towards https://gitlab.com/graphviz/graphviz/-/issues/2102.

CMake: add use_win_pre_inst_libs option

This option will be used in an upcoming commit that adds MinGW jobs to
CI.

Merge branch 'more-linking-clean-up' into 'main'

More linking cleanup

See merge request graphviz/graphviz!2127

common: link to expat privately instead of publicly

Downstream libraries that explicitly use expat now all link to expat
themselves and no longer rely on a transitive dependency to expat
through gvc via common.

plugin/neato_layout: remove linking to cdt

The neato_layout plugin source files do not refer to cdt.

plugin/gd: remove linking to cdt

The gd plugin source files do not refer to cdt.

plugin/dot_layout: remove linking to cdt

The dot_layout plugin source files do not refer to cdt.

plugin/core: remove linking to cdt

The core plugin source files do not refer to cdt.

gvc: link to cdt privately instead of publicly

Downstream libraries that explicitly use cdt now all link to cdt
themselves and no longer rely on a transitive dependency to cdt
through gvc.

gxl2gv: remove linking to cdt

The gxl2gv source files do not refer to cdt.

gvpack: remove linking to cdt

The gvpack source files do not refer to cdt.

graphml2gv: remove linking to cdt

The graphml2gv source files do not refer to cdt.

dijkstra: remove linking to cdt

The dijkstra source files do not refer to cdt.

neatogen: add linking to sparse

E.g. lib/neatogen/adjust.c uses SparseMatrix_from_coordinate_arrays
from lib/sparse/SparseMatrix.c.

neatogen: add linking to gvc

E.g. lib/neatogen/stress.c uses gcalloc from lib/common/memory.c
through the N_NEW macro in lib/common/memory.h.

neatogen: add linking to pathplan

E.g. lib/neatogen/multispline.c uses Proutespline from
lib/pathplan/route.c.

sfdpgen: add linking to neatogen

E.g. lib/sfdpgen/post_process.c uses call_tri from
lib/neatogen/call_tri.c.

sfdpgen: add linking to sparse

E.g. lib/sfdpgen/Multilevel.c uses random_permutation from
lib/sparse/general.c.

sfdpgen: add linking to gvc

E.g. lib/sfdpgen/spring_electrical.c uses gcalloc from
lib/common/memory.c and the common object files are available through
the gvc library.

graphml2gv: remove useless linking to ingraphs

The graphml2gv source files do not refer to ingraphs.

gml2gv: remove useless linking to ingraphs

The gml2gv source files do not refer to ingraphs.

ingraphs: add linking to cgraph

E.g. ingraphs.c uses agsetfile from lib/cgraph/scan.l.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

label: add linking to cdt

E.g. lib/label/xlabels.c use Dtobag from lib/cdt/dttree.c.

This removes the need to link to cdt for downstream libraries that
do not explicitly use cdt.

gvc: CMake: link to pack privately instead of publicly

The pack object modules are part of the gvc library and downstream
libraries should link to the gvc library, not to the pack
library. Hence no need for a transitive dependency to pack in gvc.

Merge branch 'smattr/550FBFBC-60E7-4C91-85B7-795331C1A531' into 'main'

remove SWIG #include malloc.h work arounds

See merge request graphviz/graphviz!2119

remove SWIG #include malloc.h work arounds

These lines were working around an issue where SWIG-generated code includes the
non-standard header malloc.h This was fixed in SWIG commit
571a84c9e788a8e97517a5d730d0e01edf0085e2 which made it into SWIG v1.3.32
released in 2007. Thus this work around is no longer necessary.

Related to #2115.

Merge branch 'clean-up-expat-xdot-and-label-linking' into 'main'

Clean up linking of expat, xdot and label

See merge request graphviz/graphviz!2123

gvc: CMake: remove now obsolete linking to label

The gvc source files do not refer to label explicitly and any implicit
references are now resolved through common.

common: add linking to label

E.g. lib/common/postproc.c uses placeLabels from lib/label/xlabels.c.

This removes the need to link to label for downstream libraries that
do not explicitly use label.

gvc: CMake: remove now obsolete linking to xdot

The core plugin that previously relied on this now explicitly links to
xdot itself.

The gvc source files do not refer to xdot explicitly and any implicit
references are now resolved by the transitive dependency through
common.

plugin/core: add linking to xdot

E.g. plugin/core/gvrender_core_dot.c uses parseXDot from
lib/xdot/xdot.c.

This will allow removal of unecessary linking to xdot in gvc in an
upcoming commit.

common: add linking to xdot

E.g. lib/common/emit.c. uses parseXDotF from lib/xdot/xdot.c.

This removes the need to link to xdot for downstream libraries that do
not explicitly use xdot.

gvc: CMake: replace now obsolete linking to expat

The gvc source files do not refer to expat explicitly and any implicit
references are now resolved through common.

common: add linking to expat if we have it

E.g. lib/common/htmllex.c uses XML_ParserCreate from
/usr/include/expat.h when we have expat.

This removes the need to link to expat for downstream libraries that
do not explicitly use expat.

Merge branch 'clean-up-cgraph-linking' into 'main'

Clean up cgraph linking

See merge request graphviz/graphviz!2124

gvc: change to link to cgraph privately

Downstream libraries that explicitly use cgraph now link to cgraph
themselves and no longer rely on a transitive dependency to cgraph
through gvc.

plugin/neato_layout: remove now obsolete linking to cgraph

The neato_layout source files do not refer to cgraph explicitly and
dependencies that need cgraph now all link to cgraph themselves.

osage: add linking to cgraph

E.g. osageinit.c uses agfstnode from lib/cgraph/node.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

patchwork: add linking to cgraph

E.g. patchwork.c uses agfstnode from lib/cgraph/node.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

twopigen: add linking to cgraph

E.g. twopiinit.c uses agnode from lib/cgraph/node.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

neatogen: add linking to cgraph

E.g. constraint.c uses agopen from lib/cgraph/graph.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

fdpgen: add linking to cgraph

E.g. layout.c uses agopen from lib/cgraph/graph.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

circogen: add linking to cgraph

E.g. circularinit.c uses agopen from lib/cgraph/graph.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

plugin/dot_layout: CMake: remove now obsolete linking to cgraph

The dot_layout source files do not refer to cgraph explicitly and
dependencies that need cgraph now all link to cgraph themselves.

dotgen: add linking to cgraph

E.g. cluster.c uses agfstnode from lib/cgraph/node.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

cmd/gvpr: explicitly include cgraph.h in gvprmain.c

gvprmain.c uses e.g. agread from cgraph.h and should not rely on an
implicit include through other header files.

dot: explicitly include cgraph.h in dot.c

dot.c uses e.g. agopen from cgraph.h and should not rely on an
implicit include through other header files.

plugin/pango: add linking to cgraph

E.g. plugin/pango/gvgetfontlist_pango.c uses agxbinit from
lib/cgraph/agxbuf.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

common: add linking to cgraph

E.g. lib/common/args.c uses agerr from lib/cgraph/agerror.c.

This removes the need to link to cgraph for downstream libraries that
do not explicitly use cgraph.

Merge branch 'smattr/54AF45E5-AEC5-41BC-A85B-896646F2E566' into 'main'

clean up Python configuration in Autotools

See merge request graphviz/graphviz!2122

inline config_python.py into configure.ac

This allows removing parts of this script that were unused. configure.ac was
already calling snippets of Python inline, so this is slightly more consistent.
Supersedes !1801.

remove $PYTHON_VERSION_SHORT in Autotools

This string is already available in $PYTHON_VERSION.

remove a commented out Autotools line

remove Autotools probing of Python ≤ 2

Final Python 2 support was removed in 61ae0419198465b457df875881a2dae0aadc8bfd.
But this configure logic was still attempting to discriminate between minor
Python 2 and Python 1 versions. After this change, we just simply reject
anything < Python 3.

Related to #1992.

Merge branch 'upgrade-windows-libraries-to-rx-spencer-removal' into 'main'

Update windows libraries submodule to get removal of rxSpencer

See merge request graphviz/graphviz!2121

Merge branch 'pathplan-cmake-linking-cleanup' into 'main'

Clean up pathplan linking in CMake

See merge request graphviz/graphviz!2116

Update windows libraries submodule to get removal of rxSpencer

plugin/neato_layout: CMake: remove now obsolete linking to pathplan

The neato_layout source files do not refer to pathplan explicitly and
any implicit references are now resolved by its dependencies
themselves.

fdpgen: CMake: add linking to pathplan

E.g. lib/fdpgen/clusteredges.c uses Pobsopen from
lib/pathplan/vispath.h.

This removes the need to link to pathplan for downstream libraries
that do not explicitly use pathplan.

plugin/dot_layout: CMake: remove now obsolete linking to pathplan

The dot_layout source files do not refer to pathplan explicitly and
any implicit references are now resolved by its dependcies themselves.

mm2gv: CMake: remove now obsolete linking to pathplan

The mm2gv source files do not refer to pathplan explicitly and any
implicit references are now resolved by its dependencies themselves.

gvc: CMake: replace now obsolete linking to pathplan with linking to common

The gvc source files do not refer to pathplan explicitly and any
implicit references are now resolved through common.

common: CMake: add linking to pathplan

E.g. lib/common/emit.c uses freePath from lib/pathplan/util.c.

This removes the need to link to pathplan for downstream libraries
that do not explicitly use pathplan.

Merge branch 'smattr/7E3E9771-0DB2-434D-8F36-9A9A7FE35055' into 'main'

Start 2.49 development series

See merge request graphviz/graphviz!2120

Start 2.49 development series

Merge branch 'smattr/455AF9C3-D61B-4F88-8BB9-50A67C6EF77D' into 'main'

Stable Release 2.49.0

See merge request graphviz/graphviz!2109

Stable Release 2.49.0

Merge branch 'smattr/494AC848-5CEA-4C42-9C10-C1F7AC93BDDC' into 'main'

fix dangerous-default-value issues in gvtest.py test suite support

See merge request graphviz/graphviz!2117

fix dangerous-default-value issues in gvtest.py test suite support

Pylint warns that using `[]` as a default is dangerous due to modifications to
the parameter unintentionally persisting across calls. I do not believe that
happens in this case, but it is better to rephrase this code to pacify Pylint
and make it clearer to readers that this is not happening.

Merge branch 'cleanup-useless-expat-referencing' into 'main'

Cleanup useless expat referencing

See merge request graphviz/graphviz!2114

mm2gv: remove unused expat include dir

gvpack: remove unused expat include dir

gvgen: remove unused expat include dir

gvcolor: remove unused expat include dir

gml2gv: remove unused expat include dir

dijkstra: remove unused expat include dir

simple tools: remove unused expat include dir

sccmap: remove unused expat as a dependency

Merge branch 'smattr/47F92637-628D-4E62-846C-499A66503C2F' into 'main'

a few trivial clean up pieces

See merge request graphviz/graphviz!2113