cluster: autotools: link to the gvc lib instead of the static common lib

The common lib is part of gvc and should not be linked to directly.

This will allow an upcoming MR to correct the storage class attributes
in lib/common/globals.h.

Wihout this commit, many errors like this would appear when linking
cluster:

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: ../../lib/edgepaint/.libs/libedgepaint_C.a(node_distinct_coloring.o): in function `node_distinct_coloring_internal2':
/home/magja/graphviz/lib/edgepaint/node_distinct_coloring.c:137: undefined reference to `__imp_Verbose'

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

gvmap: autotools: link to the gvc lib instead of the static common lib

The common lib is part of gvc and should not be linked to directly.

This will allow an upcoming MR to correct the storage class attributes
in lib/common/globals.h.

Wihout this commit, many errors like this would appear when linking
gvmap:

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: ../../lib/edgepaint/.libs/libedgepaint_C.a(node_distinct_coloring.o): in function `node_distinct_coloring_internal2':
/home/magja/graphviz/lib/edgepaint/node_distinct_coloring.c:137: undefined reference to `__imp_Verbose'

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

plugin/pango: autotools: add linking to cgraph

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

Fixes errors like this with Cygwin:

/usr/lib/gcc/x86_64-pc-cygwin/11/../../../../x86_64-pc-cygwin/bin/ld: .libs/gvgetfontlist_pango.o: in function `agxbputc':
/cygdrive/c/GitLab-Runner/builds/magjac/graphviz/graphviz-2.49.1~dev.20210911.2106/plugin/pango/../../lib/cgraph/agxbuf.h:93: undefined reference to `agxbmore'

plugin/visio: autotools: add linking to cgraph

E.g. VisioRender.cpp uses agroot from lib/cgraph/obj.c.

plugin/core: autotools: add linking to xdot

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

Fixes these errors when building with Cygwin from the portable source
produced by the Ubuntu 18.03 runner in CI:

/cygdrive/c/GitLab-Runner/builds/magjac/graphviz/graphviz-2.49.1~dev.20210915.2014/plugin/core/gvrender_core_json.c:313: undefined reference to `parseXDot'
/usr/lib/gcc/x86_64-pc-cygwin/11/../../../../x86_64-pc-cygwin/bin/ld: /cygdrive/c/GitLab-Runner/builds/magjac/graphviz/graphviz-2.49.1~dev.20210915.2014/plugin/core/gvrender_core_json.c:331: undefined reference to `freeXDot'

For some reason, this error did not show up when building using a
build system generated on Cygwin with autogen.sh.

lefty: autotools: don't try to build lefty when it's not possible

Fixes this error with Cygwin:

In file included from garray.c:14:
../../../../cmd/lefty/g.h:20:10: fatal error: X11/Intrinsic.h: No such file or directory
   20 | #include <X11/Intrinsic.h>
      |          ^~~~~~~~~~~~~~~~~

plugin/webp: autotools: add linking to gvc

E.g. plugin/webp/gvloadimage_webp.c used gvusershape_file_access from
lib/gvc/gvusershape.c.

Fixes the following errors with Cygwin:

gvloadimage_webp.o: in function `webp_loadimage':
plugin/webp/gvloadimage_webp.c:139: undefined reference to `__imp_gvusershape_file_access'
plugin/webp/gvloadimage_webp.c:153: undefined reference to `__imp_gvusershape_file_release'
plugin/webp/gvloadimage_webp.c:153: undefined reference to `__imp_gvusershape_file_release'
gvdevice_webp.o: in function `writer':
plugin/webp/gvdevice_webp.c:43: undefined reference to `gvwrite'

plugin/neato_layout: autotools: remove reference to nonexistent .def file

The gvplugin_neato_layout.def file has never existed in the git repo.

This reference was added in e74824c819a069dca2162c57deea96a7b3e5df1c,
but the .def file was never committed.

Fixes this error with MinGW:

libtool:   error: symbol file '../../plugin/neato_layout/gvplugin_neato_layout.def' does not exist

Merge branch 'smattr/508C5931-2894-41DE-9B0F-C8B9CE32549B' into 'main'

lefty clean up

See merge request graphviz/graphviz!2170

lefty SFclearList: abbreviate some float literals

lefty: remove no-op doLayout function

lefty: remove commented out code

lefty: remove some unnecessary casts

lefty SFstatAndCheck: use 'size_t' variables when dealing with buffer sizes

Squashes two -Wconversion warnings and improves portability.

lefty SFgetDir: standardize 'qsort' usage

The `qsort` function has been ISO standardized as far back as C89. There is no
need for workaround on any toolchain that correctly implements C99.

lefty SFcompareEntries: replace case-insensitive alternative with 'strcasecmp'

This functionality is built in to libc these days.

lefty SFcompareEntries: remove some unnecessary casts

Merge branch 'smattr/0232394D-4EE9-4B97-959C-70C6A25DB850' into 'main'

more -Wmissing-field-initializer warning squashing

See merge request graphviz/graphviz!2173

p_insulator: squash a -Wmissing-field-initializer warning

p_utr: squash a -Wmissing-field-initializer warning

p_terminator: squash a -Wmissing-field-initializer warning

p_cds: squash a -Wmissing-field-initializer warning

p_promoter: squash a -Wmissing-field-initializer warning

p_star: squash a -Wmissing-field-initializer warning

p_Mcircle: squash a -Wmissing-field-initializer warning

p_Msquare: squash a -Wmissing-field-initializer warning

p_diamond: squash a -Wmissing-field-initializer warning

p_tripleoctagon: squash a -Wmissing-field-initializer warning

Merge branch 'smattr/51F91849-AB76-4338-A2A6-A23F67BF0727' into 'main'

more lefty cleanup

See merge request graphviz/graphviz!2172

lefty: remove unused 'PTR2INT' and 'INT2PTR' macros

lefty SFcompareLogins: remove some unnecessary casts

lefty SFstrncmp: replace with strncasecmp

This functionality is built in to libc these days.

lefty SFgetHomeDirs: standardize 'qsort' usage

The `qsort` function has been ISO standardized as far back as C89. There is no
need for workaround on any toolchain that correctly implements C99.

lefty: remove unnecessary getwd prototype

The only use of this function was removed in
a4468a2a875dadaee313fb2a7333881cee85fb91.

Merge branch 'smattr/0824fd6b-4a2f-49f4-99cc-ed8100eecba5' into 'main'

yet more warning squashing

See merge request graphviz/graphviz!2166

sparse_stress_subspace_majorization_kD: replace a boolean with a C99 bool

And reflow some text.

stress_majorization_kD_mkernel: replace a boolean with a C99 bool

And reflow some text.

dist_data: replace a boolean with a C99 bool

get_mq: abbreviate a malloc;memzero

The calloc function can do this more concisely and efficiently.

exisAssign: remove unnecessary bracketing

expr lexname: return a 'const char*'

The callers of this do not modify the returned value. This squashes a
-Wcast-qual warning.

expr global: squash -Wunused-parameter warning

expr global: remove unnecessary casts

expr print: use a 'size_t' for iteration

Squashes a -Wsign-compare warning.

expr scan: use a 'size_t' for iteration

Squashes a -Wsign-compare warning.

Merge branch 'smattr/f7c14ca2-789a-46b0-b5f1-2cacd08dfbc5' into 'main'

more warning squashing

See merge request graphviz/graphviz!2158

lu_decompose: remove unnecessary bracketing

lu_decompose: rephrase comparisons against 'biggest' to avoid equals

This squashes some -Wfloat-equal warnings, but has the same effect due to the
surrounding code ensuring `biggest` can only ever increase from 0.

lu_decompose: rephrase an open coded fmax

lu_decompose: remove micro-optimization of avoiding multiplication by 0

This is not on a hot path and 0 multiplications are not costly on modern
processors. This squashes a -Wfloat-equal warning.

find_ints: squash a -Wswitch-default warning

This switch on the result of a call to `gt` only ever results in -1, 1, or 0.
The 0 case was pretty clearly intended to be a no-op here.

neatogen gt: rephrase into something more obvious

This is intended to preserve the same semantics. It not only avoids two
-Wfloat-equal warnings, but makes this code so close to the leading comment that
the comment can simply be removed now.

neatogen gt: rephrase to avoid having to cast pointers to this function

neatogen between: rephrase to avoid -Wfloat-equal warnings

This is intended to have the same semantics.

find_ints: remove unused 'polygon_list' parameter

use a header with prototypes to squash bindings -Wmissing-prototypes warnings

This removes 30 build warnings and is generally a cleaner way of doing this.

Merge branch 'smattr/25F2CD31-80C6-48D6-A6EE-9D7EEEB2FA14' into 'main'

smyrna: remove unused materials.h

See merge request graphviz/graphviz!2160

smyrna: remove unused materials.h

Merge branch 'smattr/FDB29D45-8A65-4119-B80E-24C4F73985C5' into 'main'

more smyrna clean up

See merge request graphviz/graphviz!2161

Merge branch 'windows' into 'main'

Update schema of generated deploy.py JSON

See merge request graphviz/graphviz!2174

Update schema of generated deploy.py JSON

I've updated the schema in the docs dir, to make room for Windows releases in here.

I haven't filled in the windows array yet.

smyrna: remove unused hotkey macros

smyrna: remove unused 'get_key_action'

smyrna: remove unused LOCATION_X macros

smyrna: remove unused tvnode types

smyrna: remove 'attach_camera_widget' prototype

This function is not implemented.

smyrna: remove unused 'countof' macro

smyrna apply_actions: remove useless break statement

smyrna: remove unused 'setGdkColor'

smyrna realize: remove useless trailing return statement

smyrna: remove some commented out prototypes

Merge branch 'smattr/d8544aa0-66a2-485d-bc01-4017f399a188' into 'main'

progress on removing sfio

See merge request graphviz/graphviz!2154

gvpr mkStmts: use internal temporary buffers instead of sfio

This simplifies code, progresses the ongoing sfio removal goal, and also makes
it more obvious to the compiler how to optimize this code.

Related to #1873, #1998.

gvpr mkStmts: make 'lbl' parameter const

The pointed to data is not modified within this function.

gvpr mkBlock: remove an unnecessary use of an sfio buffer

Related to #1873, #1998.

gvpr mkBlock: more precise bounds on internal buffers

On most platforms, `BUFSIZ` is likely large enough to contain the data printed
to it. But since we know the exact data that is placed in these buffers, we can
write this code more portably.

expose itos buffer size as a constant

Elsewhere in Graphviz we sometimes need to print ints into fixed sized buffers,
but not using itos. So it is useful to have a value for the number of bytes this
requires that is also regularly validated by the Graphviz test suite. That is,
uses of this constant should be robust to future environment/platform changes,
because the itos tests will fail, prompting an update of this constant, that
will then correctly affect all usages of this.

gvpr mkBlock: scope a buffer closer to its uses

gvpr compile: make 'lbl' parameter const

The pointed to data is not modified within this function.

gvpr compileProg: remove now unnecessary 'strdup'

Now that the backing memory for `endgsfx` is no longer hosted in a temporary
sfio buffer that may be overwritten, there is no need to duplicate this further.

gvpr doFlags: rewrite to avoid dynamic memory allocation

There is no need for this function to go through the overhead of sfio and
dynamic string construction just for this simple logic.

Related to #1873, #1998.

gvpr compileProg: remove some dead code

The `doFlags` function, at a minimum, prints "\n" into its buffer so there is
never a case when the result is an empty string.

remove unused 'ptchk' function

gvpr compile: make 'sfx' parameter const

The pointed to data is not modified within this function.

Merge branch 'smattr/0d085d9e-4d11-43b3-9d0c-d940e49e10d5' into 'main'

rewrite `INT2PTR` and `PTR2INT` as functions

See merge request graphviz/graphviz!2153

 lib/gvpr/compile.c: rewrite 'PTR2INT' macro as a function

This is more straightforward, type safe, and shorter. This also squashes 13
-Wbad-function-cast compiler warnings.

lib/gvpr/compile.c: rewrite 'INT2PTR' macro as a function

This is more straightforward, type safe, and shorter.

Merge branch 'smattr/1c348b36-65e2-4302-b77e-2c0e358a9066' into 'main'

squash some -Wmissing-field-initializer warnings

See merge request graphviz/graphviz!2152

p_doubleoctagon: squash a -Wmissing-field-initializer warning

p_invhouse: squash a -Wmissing-field-initializer warning

p_invtrapezium: squash a -Wmissing-field-initializer warning

p_invtriangle: squash a -Wmissing-field-initializer warning

p_doublecircle: squash a -Wmissing-field-initializer warning

p_cylinder: squash -Wmissing-field-initializer warning

p_underline: squash a -Wmissing-field-initializer warning

p_component: squash -Wmissing-field-initializer warning

p_box3d: squash a -Wmissing-field-initializer warning

p_folder: squash a -Wmissing-field-initializer warning

Merge branch 'smattr/f144d6f4-ad9a-4600-815b-d8a29575145d' into 'main'

teach the Autotools build system that PHP headers are not part of Graphviz

See merge request graphviz/graphviz!2155

teach the Autotools build system that PHP headers are not part of Graphviz

tl;dr: this removes 166 spurious compiler warnings.

When building the Graphviz PHP bindings, PHP headers are included and the build
system was treating these as local includes; things to be warned about. With PHP
7, this was generating a *lot* of warnings. Many of these issues are already
fixed upstream in PHP but, moreover, these files are not part of Graphviz and
warnings about them should not be part of the Graphviz build.

This commit teaches the compiler that the PHP headers are system headers, not
local headers, and warnings about them should be suppressed.

Some open questions:

  1. I explored finding an alternative `--includes` option to `php-config` that
     would give `-isystem` prefixed paths instead of `-I` prefixed paths. It
     would be cleaner than this commit to do such a thing in configure.ac, but
     such an option does not appear to exist.

  2. Vim highlights the use of `$(patsubst …)` as if it is an error. Is it
     illegal to use GNU Make functions in Makefile.am files? I do not think so,
     but given how long both Autotools and Vim have co-existed for, I am
     wondering if this highlighting is deliberate and telling me something I am
     missing.