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

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

Merge branch 'generated-cmake-dep' into 'main'

Ensure correct file-level dependency

See merge request graphviz/graphviz!2142

add CHANGELOG entry for prior commit

Ensure correct file-level dependency for generated file in cmake generated projects

When using ninja parallel compilation, it is possible that the generated ninja
file might not create the necessary files, color_lib and svgcolor_lib, before
they are needed and this results in a failed build. Providing an absolute path
to depend on in the add_custom_command rectifies the situation.

Closes #2119

Merge branch 'smattr/1264c66c-9666-43b2-b04a-877f0d54f2be' into 'main'

remove `make check` Python targets

Closes #2112

See merge request graphviz/graphviz!2131

remove `make check` Python targets

The current Graphviz test suite is orchestrated via Pytest and assumes it is
dealing with an installed Graphviz (e.g. headers and libraries available in
expected locations). The legacy Autotools check targets gave the incorrect
impression that this was the current test suite and it was expected to pass. It
has not been possible to run these legacy test scripts for a long time.

This commit removes the legacy testing pieces to avoid future confusion. In
future it may be useful to plumb the current Pytest test suite to the Autotools
post-install test targets (`make installcheck`).

Closes #2112.

Merge branch 'smattr/24AB31CD-C80E-4E3C-905F-C63B2C4382DD' into 'main'

some typo fixes

See merge request graphviz/graphviz!2133

ps_font_equiv.h: remove unused 'DARWIN' branch

The macro defined on macOS is `__APPLE__`, not `DARWIN`. The build system
conditionally defines `DARWIN`, but config.h is not included in this file so
that has no effect. The else case of this logic is the same as the `DARWIN` case
so removing this has no effect.

fix message typo in D bindings configuration

remove Fedora 26 legacy comment

Merge branch 'smattr/623e3f1a-6199-44f5-9e36-ee92e01a7630' into 'main'

fix various problems with the Javascript bindings

See merge request graphviz/graphviz!2129

default the Javascript bindings to JSC

SWIG is capable of targeting three Javascript runtimes when generating code:
JSC, V8, and NodeJS. One of these needs to be selected at generation time; there
is no default. In future it may be desirable to target the more popular of
these, NodeJS, but for now we leave it with the traditional JSC.

There are still multiple blockers to enabling this fully in the build system:

  1. The configuration setup checks for a `js` binary. I do not know what this
     is. Maybe it is intending to check for `jsc`, the JSC compiler?

  2. Even if that were true, this is the wrong thing to check for. Generation of
     bindings and their compilation requires not a Javascript compiler but the
     runtime’s C/C++ headers. For JSC, JavaScriptCore/JavaScript.h.

use a less error prone pattern for output filenames in the bindings Makefile

fix Javascript binding generation

This appears to have been copied from D bindings without updating the output
filename.

fix various typos in Javascript bindings configuration

This appears to have been copied from the D bindings in
aebb9927a3670e02a0c3e83933dfea3c7bf8dbbe, with numerous text chunks taken
verbatim.

Merge branch 'smattr/F500167B-4E3B-48CB-B3CC-D74707171300' into 'main'

yet more Pylint warning squashing

See merge request graphviz/graphviz!2136

test_shapes.py: address missing docstrings

rtest.py: remove shadowing of 'TEST'

rtest.py: remove 'OUTDIR' parameter to 'doDiff'

This parameter is available as a global. This change removes a Pylint
redefined-outer-name warning.

rtest.py: remove 'REFDIR' parameter to 'doDiff'

This parameter is available as a global. This change removes a Pylint
redefined-outer-name warning and a Pylint too-many-arguments warning.

rtest.py: remove unused TESTNAME

Every usage of this within this file is actually referring to a closer variable
that shadows this global.

rtest.py: address Pylint missing-function-docstring warnings

shapes.py: remove variable shadowing

Squashes Pylint redefined-outer-name warnings.

Merge branch 'smattr/07ffb8b9-aba1-45d9-a6b4-abf34e9278e4' into 'main'

ellipse.c clean up

See merge request graphviz/graphviz!2125

change ellipse.c #if STANDALONE to #ifdef

From later usage of this macro in the same file, it looks like this is what was
intended. Squashes a -Wundef build warning.

genEllipticPath: remove shadowing of 'etaB' variable

genEllipticPath: remove shadowing of 'dEta' variable

endPath: remove shadowing of global 'path' variable

curveTo: remove shadowing of global 'path' variable

lineTo: remove shadowing of global 'path' variable

moveTo: remove shadowing of global 'path' variable

genEllipticPath: remove shadowing of global 'path' variable

remove some unused struct names

genEllipticPath: remove 'degree' parameter

This function is only ever called with the argument 3.

genEllipticPath: remove 'threshold' parameter

This is always called with the same threshold.

remove now-unused hand rolled boolean type

endPath: remove 'close' parameter

This function is only ever called with the argument true.

genEllipticPath: remove 'isSlice' parameter

This function is only ever called with the argument true.

genEllipticPath: remove unnecessary bracketing

genEllipticPath: use a C99 bool instead of boolean type

Merge branch 'smattr/CC730FC7-8FD1-477A-B0A9-3F40CD40836A' into 'main'

clean up in lib/common

See merge request graphviz/graphviz!2126

findStopColor: remove unnecessary bracketing

gvRenderJobs: remove unnecessary bracketing

style_token: remove unnecessary bracketing

emit_clusters: remove unnecessary bracketing

init_job_viewport: remove unnecessary bracketing

init_job_dpi: remove unnecessary bracketing

init_gvc: remove unnecessary bracketing

xdotBB: remove unnecessary bracketing

expandBB: rephrase open coding to fmax, fmin

emit_edge_label: remove unnecessary bracketing

remove unnecessary void* casts in calls to strdup_and_subst_obj

These pointers implicitly coerce.

emit_begin_edge: remove unnecessary bracketing

emit_edge_graphics: remove unnecessary bracketing

taperfun: remove unnecessary bracketing

bothfunc: remove unnecessary bracketing

nonefunc: remove unnecessary bracketing

revfunc: remove unnecessary bracketing

forfunc: remove unnecessary bracketing

multicolor: remove unnecessary bracketing

emit_attachment: use a more appropriate type for 's' local

emit_node: remove unnecessary bracketing

emit_begin_node: remove unnecessary bracketing

setup_page: remove unnecessary bracketing

emit_xdot: remove unnecessary bracketing

init_job_pagination: use fmax instead of open coding

init_job_pagination: remove unnecessary bracketing

firstlayer: remove unnecessary bracketing

init_layering: remove unnecessary bracketing

map_output_bspline: remove unnecessary bracketing

bisect: remove unnecessary bracketing

isFilled: return a C99 bool instead of int

rename 'ifFilled' to 'isFilled'

From the leading comment, this is what it was intended to be called.

stripedBox: remove unnecessary bracketing

wedgedEllipse: remove unnecessary bracketing

checkClusterStyle: remove unnecessary bracketing

getObjId: remove unnecessary bracketing

layerPagePrefix: remove unnecessary bracketing

colorseg_t: use a C99 bool for 'hasFraction' field

isRect: remove unnecessary bracketing

isRect: use a C99 bool return type instead of boolean

interpretCRNL: use a C99 bool for 'backslash_seen' local

remove some open coded NULLs

parse_style: use a C99 bool for 'in_parens' local

parse_style: use a C99 bool for 'is_first' local

is_style_delim: use a C99 bool return type instead of boolean

nodeIntersect: use a C99 bool for 'explicit' local

emit_edge_graphics: use a C99 bool for 'tapered' local

node_in_box: use a C99 bool return type instead of boolean

clust_in_layer: use a C99 bool return type instead of boolean

edge_in_layer: remove unnecessary bracketing

edge_in_layer: use a C99 bool return type instead of boolean

node_in_layer: remove unnecessary bracketing

node_in_layer: use a C99 bool return type instead of boolean

write_node_test: use a C99 bool return type instead of boolean

write_edge_test: use a C99 bool return type instead of boolean

validpage: remove unnecessary bracketing

validpage: use a C99 bool return type instead of boolean

validlayer: remove unnecessary bracketing

validlayer: use a C99 bool return type instead of boolean