Add macos-cmake-test job to CI

Resolves https://gitlab.com/graphviz/graphviz/-/issues/1727.

Add macos-autotools-test job to CI

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

Add skip of test_1818 also for macOS

Fails because of https://gitlab.com/graphviz/graphviz/-/issues/1858.

Add skip of test_1813 also for macOS

Fails because of https://gitlab.com/graphviz/graphviz/-/issues/1857.

Add skip of all tools not built for macOS in CI using autotools in tools test

Fails because not all tools are built for macOS in CI using
autotools. See https://gitlab.com/graphviz/graphviz/-/issues/1854

Add subset of macOS rtest reference files

Generated with ./rtest.py -g. The .gv files are identical to those in
the linux.x86 subdirectory. The .ps file has slight differences, but
no visible differences.

Add brew install pango to macos build jobs

Needed for png format and cairo renderer.

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1860.

Add support for macOS to ci/install.sh

Refactor ci/install.sh to check for build system before OS

Fix tred -? to actually print usage instead of error on macOS

Final commit that fixes https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix sccmap -? to actually print usage instead of error on macOS

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

Fix nop -? to actually print usage instead of error on macOS

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

Fix mm2gv -? to actually print usage instead of error on macOS

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

Fix gvpack -? to actually print usage instead of error on macOS

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

Fix gvcolor -? to actually print usage instead of error on macOS

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

Fix gc -? to actually print usage instead of error on macOS

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

Fix dijkstra -? to actually print usage instead of error on macOS

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

Fix gvmap -? to actually print usage instead of error on macOS

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

Fix cluster -? to actually print usage instead of error on macOS

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

Fix edgepaint -? to actually print usage instead of error on macOS

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

Fix ccomps -? to actually print usage instead of error on macOS

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

Fix bcomps -? to actually print usage instead of error on macOS

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

Restore whitespace and comment after earlier bad merge conflict resolution

Merge branch 'smattr/629f9eac-8e8a-486d-9695-cc857dce075c' into 'master'

some clean up

See merge request graphviz/graphviz!1620

remove rtest.sh redirect

Nothing in the repository uses this any more.

remove unnecessary 'global' statements in rtest

These variables were either unused in the contained function or only read from.

fix some comparisons between booleans and ints in rtest

This all worked fine, but for consistency we should be treating these variables
as booleans everywhere.

remove legacy comment

The variable this is referring to, LINECNT, was removed when this script was
ported to Python in a1b9ffc0c37a435607ac8b1c123c8c542ef1cec9.

remove some unused variables in rtest

squash a -Wabsolute-value warning

In 8414e62738bf7f52596eb83049ecef758582c760 I misread the surrounding code and
thought this was a float being operated on here.

squash -Wcomment warning

remove unused onexit() prototype

remove unused _exit() prototype

remove unused prototypes for frexp() and ldexp()

remove unneeded X_OK shim

Nothing in Graphviz uses X_OK.

remove POOLMTXRETURN()

remove POOLMTXSTART() no-op

remove POOLMTXUNLOCK() no-op

remove POOLMTXLOCK() no-op

remove SFMTXUNLOCK() no-op

remove no-op SFMTXLOCK()

remove useless extern declarations

These variables already have extern linkage.

remove now unused vthread.h

remove unused sf_mutex field from Sfextern_t

remove unused mtx parameter to SFCLEAR() macro

remove unused mutex field from SFIO structures

remove unused Sfonce functionality

remove SFONCE() no-op

remove vmtx mutex no-ops

remove unused SFIO vthread prototypes

remove unused sftell()

remove unused _sfstrtod()

Merge branch 'orientation' into 'master'

Combine `orientation` attr for graphs and nodes

See merge request graphviz/graphviz!1640

Combine `orientation` attr for graphs and nodes

This simplifies the generation quite a lot, gets rid of the duplicated
anchors for differently-named.

Removes a little confusion too, in my opinion.

It's not perfect: they have different datatypes.

Merge branch 'stray-a' into 'master'

attrs.html: Close TT tag

See merge request graphviz/graphviz!1639

attrs.html: Close TT tag

Merge branch 'stray-a' into 'master'

attrs.html: Remove stray anchor close tag

See merge request graphviz/graphviz!1638

attrs.html: Remove stray anchor close tag

Merge branch 'ps' into 'master'

attrs.html: Rename 'kind' to 'types'.

See merge request graphviz/graphviz!1636

attrs.html: Rename 'kind' to 'types'.

Matches the title of the table, and colloquial use.

Merge branch 'ps' into 'master'

attrs.html: Rename attr.uses to used_by

See merge request graphviz/graphviz!1635

attrs.html: Rename attr.uses to used_by

Matches table header.

Makes more sense when you're looking at an attribute definition

Merge branch 'ps' into 'master'

Add consistent indentation to attrs.html

See merge request graphviz/graphviz!1634

Add consistent indentation to attrs.html

No-op as far as appearance goes.

But makes looking at diffs much easier!

Merge branch 'ps' into 'master'

attrs.html: don't special-case double/int/string

See merge request graphviz/graphviz!1633

attrs.html: don't special-case double/int/string

Instead, add (very basic) type definitions for them.

This simplifies the template.

Merge branch 'ps' into 'master'

attrs.html: Stop special-casing bitmap output

See merge request graphviz/graphviz!1632

attrs.html: Stop special-casing bitmap output

No-op change.

Merge branch 'ps' into 'master'

attrs: Remove special-casing postscript template

See merge request graphviz/graphviz!1631

attrs: Remove special-casing postscript template

Just spell out postscript in the source

Merge branch 'smattr/07caebcd-6088-4479-89d4-80d864a567fa' into 'master'

speed up Neato

See merge request graphviz/graphviz!1619

squash a -Wfloat-conversion warning

inline and fuse a number of loops in majorization code

This accelerates the Neato example from #1652 by ~9.5%.

inline some vector computations and fuse two loops

This slightly accelerates a current test case by ~0.1%. Related to #1652.

Merge branch 'nslimit6' into 'master'

Remove no-op bool case

See merge request graphviz/graphviz!1628

Remove no-op bool case

It behaved the same as the 'else'

Merge branch 'nslimit5' into 'master'

Replace TT with CODE element

See merge request graphviz/graphviz!1627

Replace TT with CODE element

TT is deprecated

Merge branch 'nslimit4' into 'master'

attrs.html: clean ID and NAMEs

See merge request graphviz/graphviz!1626

attrs.html: clean ID and NAMEs

- Convert NAMEs to IDs, as recommended for non-form elements.
https://stackoverflow.com/a/7470325/171898
- Quote HREFs and IDs

Merge branch 'nslimit3' into 'master'

Remove two-spacing from front of HTML continuation

See merge request graphviz/graphviz!1625

Merge branch 'nslimit2' into 'master'

Simplify parsing, now all attributes have one name

Closes graphviz.gitlab.io#64

See merge request graphviz/graphviz!1623

Remove two-spacing from front of HTML continuation

No need for this, just adds complexity to the parser.

Merge branch 'nslimie1' into 'master'

Split nslimit and nslimit1 attributes

See merge request graphviz/graphviz!1622

Simplify parsing, now all attributes have one name

Closes #64

Split nslimit and nslimit1 attributes

They're the only attributes that don't already have their own sections,
they complicate parsing and emitting, and on balance I think they're
better explained separately (but next to each other).

Closes #64

Merge branch '9ddbb9ea-f0a4-481d-9c40-b9709aa1680c' into HEAD

squash a -Wimplicit-fallthrough warning

GCC recognizes "fall through" and "fallthrough" comments to suppress this
warning but not "falling through."

remove now-unnecessary use of volatile on some locals

Related to #1801.

remove now-unused ljbuf

Related to #1801.

undo an unnecessary use of setjmp/longjmp

This code actually correctly marks the variables that are affected by the
setjmp/longjmp as volatile, but there is still no need for this complexity. A
standard error code return does the trick much simpler. Related to #1801.

Merge branch 'remove-fedora-29-and-30-jobs-from-ci' into 'master'

Remove Fedora 29 and 30 jobs from CI

See merge request graphviz/graphviz!1617

Remove EOL Fedora 30 Dockerfile

Remove EOL Fedora 29 Dockerfile

Stop building & testing for EOL Fedora 30 in CI

Stop building & testing for EOL Fedora 29 in CI

Merge branch '91908b3f-1fea-401e-a238-435c742d3589' into HEAD

fix Lefty's reliance on dup() internals

This addresses the following Coverity warnings:

  Error: RESOURCE_LEAK (CWE-772): [#def39]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:362: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:362: leaked_handle: Failing to save or close handle opened by "dup(fd[1])" leaks it.
  #  360|           panic2 (POS, "ptyopen", "cannot fork");
  #  361|       case 0:
  #  362|->         close (fd[0]), close (0), dup (fd[1]);
  #  363|           close (1), dup (fd[1]), close (fd[1]);
  #  364|           execl (shell, shbname, "-c", cmd, NULL);

  Error: RESOURCE_LEAK (CWE-772): [#def40]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:363: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:363: leaked_handle: Failing to save or close handle opened by "dup(fd[1])" leaks it.
  #  361|       case 0:
  #  362|           close (fd[0]), close (0), dup (fd[1]);
  #  363|->         close (1), dup (fd[1]), close (fd[1]);
  #  364|           execl (shell, shbname, "-c", cmd, NULL);
  #  365|           panic2 (POS, "ptyopen", "child cannot exec: %s\n", cmd);

  Error: RESOURCE_LEAK (CWE-772): [#def41]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:429: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:429: leaked_handle: Failing to save or close handle opened by "dup(p1[1])" leaks it.
  #  427|                   panic2 (POS, "pipeopen", "child cannot exec: %s\n", cmd2);
  #  428|               }
  #  429|->         close (1), dup (p1[1]), close (p1[1]);
  #  430|           close (0), dup (p2[0]), close (p2[0]);
  #  431|           execl (shell, shbname, "-c", cmd, NULL);

  Error: RESOURCE_LEAK (CWE-772): [#def42]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:430: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:430: leaked_handle: Failing to save or close handle opened by "dup(p2[0])" leaks it.
  #  428|               }
  #  429|           close (1), dup (p1[1]), close (p1[1]);
  #  430|->         close (0), dup (p2[0]), close (p2[0]);
  #  431|           execl (shell, shbname, "-c", cmd, NULL);
  #  432|           panic2 (POS, "pipeopen", "child cannot exec: %s\n", cmd);

Fixes #1823. Related to #1464.

Merge branch 'master' into 'master'

Since the string from strerror should never be modified, use const.

See merge request graphviz/graphviz!1618

Merge branch 'b2f0e4be-93c5-4d03-9a33-16c7f45988a8' into HEAD