lib/edgepaint: MSBuild: define GVDLL in edgepaintlib project file

An upcoming commit will change lab_gamut.h to use the GVDLL symbol to
enable storage-class attributes.

This change ensures that the lab_gamut symbols will continute to be
declared with the dllimport attribute.

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

lib/edgepaint: MSBuild: define GVDLL in lab_gamut project file

An upcoming commit will change lab_gamut.h to use the GVDLL symbol to
enable storage-class attributes.

This change ensures that the lab_gamut symbols will be defined with
the dllexport attribute, which will eliminate the need for a .def
file.

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

Merge MR !2331

CMake: propagate include flag for harfbuzz on Windows

Pango >=1.44.1 changed the way harfbuzz is included
xref: https://github.com/GNOME/pango/commit/a7b17aa2

This is not required on *nix because the PkgConfig CMake module
already takes care of this. For example, on Ubuntu it adds:
-I/usr/include/harbuzz to PangoCairo_CFLAGS

Merge branch 'smattr/A6C297DD-D456-46E0-99BD-E1043C7F5AB1' into 'main'

ci: standardize shell scripts on the same e,u,x,o options

See merge request graphviz/graphviz!2336

ci: standardize shell scripts on the same e,u,x,o options

All these scripts have the same expectation, of failing on anything unexpected.
So setting the same options the same way everywhere leads to more consistency
and predictability.

ci: apply '${ID_LIKE:-}' pattern from build script to install

This allows in future enabling `set -u` in this script.

Merge branch 'smattr/4DE85C4D-850E-457B-9C7E-A5F7767BFFAE' into 'main'

disable Windows Defender in all CI jobs

See merge request graphviz/graphviz!2334

disable Windows Defender in all CI jobs

Windows Defender significantly degrades I/O and slows CI tasks to the point that
it is difficult to get a passing result on many of the Windows jobs. It is
unclear what value Windows Defender is adding in CI when the entire environment
is transient and malware has no obvious way of gaining persistence. So far, the
main effect of Windows Defender has been to spuriously block MinGW tools,
preventing CI running altogether.

Merge branch 'smattr/2756CD40-47ED-43DD-A9D7-825137573F7C' into 'main'

drop Fedora 33 builds from CI

See merge request graphviz/graphviz!2332

drop Fedora 33 builds from CI

Fedora 33 was EOLed on 2021-11-30.¹

¹ https://docs.fedoraproject.org/en-US/releases/eol/

Merge branch 'smattr/B6B64825-8DDD-4591-98B8-B063F8DC4D69' into 'main'

some Python clean up and simplification

See merge request graphviz/graphviz!2328

tests: avoid a manual open of /dev/null

tests: rewrite 'Popen;communicate' code to use 'check_output'

When this testing code was first written, it looks like it was overlooked that
`check_output` supports the `input` parameter from Python 3.4 onwards. With
this, the relevant code can be written more concisely.

tests: abstract running 'dot' and checking its return code into library function

This makes the common pattern of “run `dot`, checking its exit status, and give
me the resulting output” more concise and readable in the test suite.

tests: rename 'dot' variables in test_regression.py

An upcoming change will introduce a `dot` function, which these variables will
then shadow, causing warnings. This is a pre-emptive change to avoid this.

Merge branch 'SVG-viewport-precision' into 'main'

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1855 where the SVG view...

Closes #1855

See merge request graphviz/graphviz!2327

add a changelog entry for the prior fix

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1855 where the SVG view setup needs more than two digits of precision, otherwise clipping may result.

add a test case for scale precision issue

Merge branch 'smattr/7740732D-AFA2-4734-9DFE-051A94B5EE68' into 'main'

remove commented out code

See merge request graphviz/graphviz!2326

tclpkg/gdtclft/gdtclft.c: [nfc] remove commented out code

plugin.demo/xgtk/src/callbacks.c: [nfc] remove commented out code

plugin/gd/gvrender_gd.c: [nfc] remove commented out code

plugin/gd/gvloadimage_gd.c: [nfc] remove commented out code

plugin/gdk/gvloadimage_gdk.c: [nfc] remove commented out code

cmd/lefty/ws/mswin32/gtext.c: [nfc] remove commented out code

Merge branch 'smattr/b524e557-b4bb-48fb-a6b9-4b082a913219' into 'main'

more boolean → bool modernization

See merge request graphviz/graphviz!2325

poly_init: [nfc] use a C99 bool for 'isPlain' instead of boolean

lib/common/labels.c: [nfc] remove unnecessary parens

strdup_and_subst_obj0: [nfc] use a C99 bool for 'has_tp' instead of boolean

strdup_and_subst_obj0: [nfc] use a C99 bool for 'has_hp' instead of boolean

dot: fix declaration of 'MemTest'

This variable, declared in globals.h is an `int`, not a `boolean` (a.k.a.
`unsigned char`).

lib/neatogen/conjgrad.c: [nfc] remove a commented out #include

conjugate_gradient_f: [nfc] take a C99 bool parameter instead of a boolean

expand_t.doAdd: [nfc] use a C99 bool instead of a boolean

samedir: fix return type

Commit e1181697185289594304cfb22d5979d1fc5cbf0f altered this function to use C99
bools but did not update its return type.

htmlEntityUTF8: [nfc] use a C99 bool for 'warned' local instead of boolean

Merge branch 'smattr/D5394DF4-047C-45E9-B4B4-400B6140BE96' into 'main'

ci: [nfc] fuse tarball;mv steps into one

See merge request graphviz/graphviz!2324

ci: [nfc] fuse tarball;mv steps into one

When we are immediately going to the move a created tarball to a new location,
we may as well just save time and create it in the eventual location to begin
with.

Merge branch 'smattr/8EC56F87-69CE-47A9-8317-CBB835E23A02' into 'main'

remove Ming plugin

Closes #2160

See merge request graphviz/graphviz!2323

remove Ming plugin

The format this plugin produces, Adobe Shockwave, was officially EOLed by Adobe
in April 2019.¹ Legacy support for SWF files was available through Adobe Flash
Player. Flash Player was subsequently EOLed in December 2020 with Adobe
considering it such a security risk they flipped a kill switch in January 2021
making it no longer possible to run Flash Player anywhere.²

Closes #2160

¹ https://helpx.adobe.com/shockwave/shockwave-end-of-life-faq.html
² https://www.adobe.com/au/products/flashplayer/end-of-life.html

Merge branch 'smattr/35B3A0E0-A5D9-43EB-9837-79A1DD8788C8' into 'main'

[nfc] clean up legacy lines in Windows config.h

See merge request graphviz/graphviz!2322

[nfc] remove legacy libc header line in Windows config.h

The last use of this macro was removed in
70a4ec52014697210369c6c66c9af7d72579fdc9. Graphviz has been compiled with C99
since fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc, under which standard C headers
are required to exist.

[nfc] remove legacy 'STACK_DIRECTION' lines in Windows config.h

This constant was introduced in the initial addition of the precreated Windows
config.h, 76ead5c74029a5ceea61dd6c7600dc1931c275cf, but it is unclear where it
originates from. None of the other build systems appear to define this and no
prior or current state of the Graphviz tree has ever contained use of this
macro. Graphviz does not use `alloca`.

[nfc] remove legacy sys/dir.h line from Windows config.h

The last use of sys/dir.h was removed in
0330a4d918dcbf6edb1ce46956a20bde9a2444f9.

[nfc] remove legacy memory.h line in Windows config.h

The last use of this macro was removed in
5cbd4ebcd8254e51569b7209cb964f7d260319fb.

[nfc] remove legacy NDIR lines in Windows config.h

The only use of these defines and ndir.h headers was removed in
0330a4d918dcbf6edb1ce46956a20bde9a2444f9.

Merge branch 'smattr/8B6E8ECF-9BB8-4D71-8F5E-EDA5F3EC7695' into 'main'

fix: correct Windows release file recognition during deployment

Closes #1979

See merge request graphviz/graphviz!2321

fix: correct Windows release file recognition during deployment

6117abe680037824d134149b0de42f589fb24466 updated the deployment script to add
Windows artifacts that previously needed to be appended locally to the JSON
manifest. However it failed to account for changes made to the packaging
hierarchy that removed release collection and a redundant architecture
component.

(Hopefully) closes #1979.

Merge branch 'msys2-ci-fix' into 'main'

CI: Add C:\tools\msys64 to MS Defender exclusion list

See merge request graphviz/graphviz!2333

CI: Add C:\tools\msys64 to MS Defender exclusion list

This is a workaround for:

    cc.exe: fatal error: cannot execute
    'C:/tools/msys64/path/to/cc1.exe':
    CreateProcess: No such file or directory

Merge branch 'smattr/EF4D0E71-8197-4826-862C-DD92EB96C820' into 'main'

install and use Ccache in Cygwin CI tasks

See merge request graphviz/graphviz!2303

README: fix unescaped '&'

enable persistent Ccache cache in Cygwin CI tasks

This allows cached compilation results to be preserved and reused across CI
runs. It is hoped this will accelerate these currently long running tasks.

install and use Ccache in Cygwin CI tasks

The Cygwin CI tasks come very close to the CI timeout ceiling, usually running
for close to an hour. It is hoped that this change (and upcoming caching) will
reduce this time, thus shortening the total CI time as these are the longest
running tasks at present.

Merge branch 'add-mingw-autotools-builds-to-ci-take3' into 'main'

Add MinGW autotools build jobs to CI

See merge request graphviz/graphviz!2291

CI: add windows-mingw64-build-step2-make job

CI: add windows-mingw64-build-step1-configure job

CI: add windows-mingw32-build-step2-make job

CI: add windows-mingw32-build-step1-configure job

add ci/mingw-make.sh

This will be used in an upcoming commit to create 2-step autotools
build jobs for MinGW.

add ci/out-of-source-make.sh

This will be used in an upcoming commit to create 2-step autotools
build jobs for MinGW.

add ci/mingw-configure.sh

This will be used in an upcoming commit to create 2-step autotools
build jobs for MinGW.

add ci/out-of-source-configure.sh

This will be used in an upcoming commit to create 2-step autotools
build jobs for MinGW.

ci/mingw-build.sh: set CMAKE_OPTIONS only when using CMake

ci/mingw-build.sh: factor out installation to new mingw-install.sh script

ci/mingw-build.sh: add installation of autoconf, automake & make

ci/build.sh: autotools: excplicitly specify the tclsh for MinGW

Fixes these errors when using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/gdtclft.o: in function `tclGdWriteCmd':
C:\Users\magja\graphviz\tclpkg\gdtclft/gdtclft.c:560: undefined reference to `Tcl_GetOpenFile'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/gdtclft.o: in function `tclGdCreateCmd':
C:\Users\magja\graphviz\tclpkg\gdtclft/gdtclft.c:450: undefined reference to `Tcl_GetOpenFile'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/gdtclft.o: in function `Gdtclft_Init':
C:\Users\magja\graphviz\tclpkg\gdtclft/gdtclft.c:1278: undefined reference to `Tcl_InitStubs'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/gdtclft.o:gdtclft.c:(.rdata$.refptr.tclStubsPtr[.refptr.tclStubsPtr]+0x0): undefined reference to `tclStubsPtr'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../tclpkg/tclstubs/.libs/libtclstubs_C.a(tclStubLib.o): in function `tclstubs_dummy':
C:\Users\magja\graphviz\tclpkg\tclstubs/tclStubLib.c:17: undefined reference to `Tcl_InitStubs'

ci/build.sh: autotools add support for CONFIGURE_OPTIONS for MinGW and Cygwin

ci/build.sh: autotools: add support for MinGW builds

neato_layout: use GVDLL instead _WIN32 to control storage-class attributes

Storage-class attributes should only be used for dynamic-link
libraries, but not for statically linked libraries.

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

plugin/neato_layout: MSBuild: define GVDLL

Without this change, the following error would occur in an upcoming
commit that changes the Windows specific storage-class attributes to
be controlled by GVDLL instead of by _WIN32:

LINK : fatal error LNK1181: cannot open input file 'gvplugin_neato_layout.lib' [C:\GitLab-Runner\builds\magjac\graphviz\cmd\tools\gvpack.vcxproj]

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

tclpathplan: autotools: add linking to tcl stubs

Fixes errors like this when using autotools with MinGW:

tclpkg\tclpathplan/tclpathplan.c:396: undefined reference to `tclStubsPtr'

tcldot: autotools: add --no-undefined also for libtcldot_builtin

Fixes this error using autotools with MinGW:

libtool:   error: can't build x86_64-w64-mingw32 shared library unless -no-undefined is specified

tcldot: autotools: add linking to tcl stubs

Fixes errors like this using autotools with MinGW:

C:\Users\magja\graphviz\tclpkg\tcldot/tcldot.c:176: undefined reference to `Tcl_InitStubs'

tclpkg/gdtclft: correct storage-class attribute declarations

Fixes these errors using autotools with MinGW:

gdtclft.c:1271:13: error: expected declaration specifiers or '...' before 'Gdtclft_Init'
 1271 | EXPORT(int, Gdtclft_Init) (interp)
      |             ^~~~~~~~~~~~
gdtclft.c:1297:13: error: expected declaration specifiers or '...' before 'Gdtclft_SafeInit'
 1297 | EXPORT(int, Gdtclft_SafeInit) (interp)
      |             ^~~~~~~~~~~~~~~~

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

common: correct storage-class attributes in memory.h

Fixes many errors like this using autotools with MinGW:

  CCLD     mm2gv.exe
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: matrix_market.o: in function `SparseMatrix_import_matrix_market':
C:\Users\magja\graphviz\cmd\tools/matrix_market.c:99: undefined reference to `__imp_gmalloc'

neatogen: add storage-class attribute declarations in neatoprocs.h

Fixes this error using autotools with MinGW:
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: gvpack-gvpack.o: in function `init_node_edge':
C:\Users\magja\graphviz\cmd\tools/gvpack.c:271: undefined reference to `neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: C:\Users\magja\graphviz\cmd\tools/gvpack.c:272: undefined reference to `user_pos'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: gvpack-gvpack.o: in function `init_graph':
C:\Users\magja\graphviz\cmd\tools/gvpack.c:305: undefined reference to `init_nop'

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

sfdpgen: CMake: define NEATOGEN_EXPORTS

sfdpgen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: lib/sfdpgen/libsfdpgen.a(sfdpinit.c.obj):sfdpinit.c:(.text+0xac): undefined reference to `__imp_neato_init_node'

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

twopigen: CMake: define NEATOGEN_EXPORTS

twopigen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: lib/twopigen/libtwopigen.a(twopiinit.c.obj):twopiinit.c:(.text+0x142): undefined reference to `__imp_neato_init_node'

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

osage: CMake define NEATOGEN_EXPORTS

osage is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: lib/osage/libosage.a(osageinit.c.obj):osageinit.c:(.text+0x2ec): undefined reference to `__imp_neato_init_node'

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

neatogen: CMake: define NEATOGEN_EXPORTS

It is not automatically set for static libraries, only for shared
libraries.

Fixes errors like this using CMake with MinGW:

neatogen.lib(neatoinit.obj) : error LNK2019: unresolved external symbol __imp_circuit_model referenced in function kkNeato [C:\Users\magja\graphviz\build\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

fdpgen: CMake: define NEATOGEN_EXPORTS

fdpgen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

fdpgen.lib(clusteredges.obj) : error LNK2019: unresolved external symbol __imp_Plegal_arrangement referenced in function compoundEdges [C:\Users\magja\graphviz\build\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

circogen: CMake: define NEATOGEN_EXPORTS

circogen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

circogen.lib(circularinit.obj) : error LNK2019: unresolved external symbol __imp_spline_edges referenced in function circo_layout [C:\Users\magja\graphviz\build\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

sfdpgen: autotools: define NEATOGEN_EXPORTS

sfdpgen is part of the neato_layout plugin.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/sfdpgen/.libs/libsfdpgen_C.a(sfdpinit.o): in function `sfdp_init_node_edge':
C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:37: undefined reference to `__imp_neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/sfdpgen/.libs/libsfdpgen_C.a(sfdpinit.o): in function `tuneControl':
C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:248: undefined reference to `__imp_setSeed'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/sfdpgen/.libs/libsfdpgen_C.a(sfdpinit.o): in function `sfdp_layout':
C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:331: undefined reference to `__imp_spline_edges'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:319: undefined reference to `__imp_spline_edges'

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

circogen: autotools: define NEATOGEN_EXPORTS

circogen is part of the neato_layout plugin.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/circogen/.libs/libcircogen_C.a(circularinit.o): in function `circular_init_node_edge':
C:\Users\magja\graphviz\lib\circogen/circularinit.c:49: undefined reference to `__imp_neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/circogen/.libs/libcircogen_C.a(circularinit.o): in function `circo_layout':
C:\Users\magja\graphviz\lib\circogen/circularinit.c:249: undefined reference to `__imp_spline_edges'

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

fdpgen: autotools: define NEATOGEN_EXPORTS

fdpgen is part of the neato_layout plugin.

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/fdpgen/.libs/libfdpgen_C.a(layout.o): in function `fdp_layout':
C:\Users\magja\graphviz\lib\fdpgen/layout.c:1101: undefined reference to `__imp_neato_set_aspect'

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

osage: autotools: define NEATOGEN_EXPORTS

osage is part of the neato_layout plugin.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/osage/.libs/libosage_C.a(osageinit.o): in function `cluster_init_graph':
C:\Users\magja\graphviz\lib\osage/osageinit.c:69: undefined reference to `__imp_neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/osage/.libs/libosage_C.a(osageinit.o): in function `osage_layout':
C:\Users\magja\graphviz\lib\osage/osageinit.c:367: undefined reference to `__imp_spline_edges1'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: C:\Users\magja\graphviz\lib\osage/osageinit.c:363: undefined reference to `__imp_spline_edges0'

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

twopigen: autotools: define NEATOGEN_EXPORTS

twopigen is part of the neato_layout plugin.

Fixes errors like this when using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/twopigen/.libs/libtwopigen_C.a(twopiinit.o): in function `twopi_init_node_edge':
C:\Users\magja\graphviz\lib\twopigen/twopiinit.c:39: undefined reference to `__imp_neato_init_node'

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

neatogen: MSBuild: define NEATOGEN_EXPORTS

neatogen: autotools: define NEATOGEN_EXPORTS

lib/gvpr: autotools: fix incorrect MinGW path translation in DFLT_GVPRPATH string

Add an extra forward slash in front of $(pkgdatadir) to avoid the path
being translated to use backslashes under MSYS2/MinGW. This trick is
needed since DFLT_GVPRPATH is a string which is compiled and
backslashes in strings are interpreted as escape sequences. See
https://stackoverflow.com/a/14189687

Fixes errors like this when using autotools with MinGW:

gvpr.c:228:41: error: incomplete universal character name \U
  228 |      pathp = path = concat(DFLT_GVPRPATH, path);
      |                                         ^
gvpr.c:228:41: warning: unknown escape sequence: '\m'
gvpr.c:228:41: warning: unknown escape sequence: '\g'
gvpr.c:228:41: warning: unknown escape sequence: '\s'
gvpr.c:228:41: warning: unknown escape sequence: '\g'
gvpr.c:228:41: warning: unknown escape sequence: '\g'

autotools: add WITH_MINGW to AM_CONDITIONAL

This will allow an upcoming commit in this series to introduce
MinGW-specific functionality in the autotools build system.

pathplan: autotools: add define of PATHPLAN_EXPORTS

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/cvt.o: in function `Pobsopen':
C:\Users\magja\graphviz\lib\pathplan/cvt.c:80: undefined reference to `__imp_visibility'

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

common: autotools: define also GVC_EXPORTS

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/common/.libs/libcommon_C.a(arrows.o): in function `arrow_type_curve':
C:\Users\magja\graphviz\lib\common/arrows.c:696: undefined reference to `__imp_Bezier'

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

label: autotools: define GVC_EXPORTS

Fixes this error using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/label/.libs/liblabel_C.a(xlabels.o): in function `xlnew':
C:\Users\magja\graphviz\lib\label/xlabels.c:41: undefined reference to `__imp_zmalloc'

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

pack: autotools: define GVC_EXPORTS

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/pack/.libs/libpack_C.a(pack.o): in function `fillLine':
C:\Users\magja\graphviz\lib\pack/pack.c:127: undefined reference to `__imp_addPS'

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

gvc: autotools: define GVC_EXPORTS

Fixes errors like this using autotools with MinGW:

C:\Users\magja\graphviz\lib\gvc/gvevent.c:201: undefined reference to `__imp_overlap_edge'

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

gvc: autotools: remove now obsolete linking to xdot

The core plugin that previously relied on this explicitly links to
xdot itself since commit 6d646c82d94efdfb9933328f08301f6e1600943e.

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

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/xdot/.libs/libxdot.dll.a(d000007.o):(.text+0x0): multiple definition of `parseXDotF'; ../../lib/xdot/.libs/libxdot_C.a(xdot.o):C:\Users\magja\graphviz\lib\xdot/xdot.c:475: first defined here
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/xdot/.libs/libxdot.dll.a(d000002.o):(.text+0x0): multiple definition of `freeXDot'; ../../lib/xdot/.libs/libxdot_C.a(xdot.o):C:\Users\magja\graphviz\lib\xdot/xdot.c:927: first defined here