Merge branch '74a4b672-00f6-4510-be6a-1c3f57a1935b' into 'master'

remove one use of longjmp

See merge request graphviz/graphviz!1593

remove no longer necessary setjmp setup in libpack

Related to #1801.

return failure from push() in libpack instead of longjmping

Related to #1801.

check for failure of push() in dfs() in libpack

Related to #1801.

add a return value to push() in libpack

Related to #1801.

anticipate return of SIZE_MAX indicating failure from dfs()

Related to #1801.

remove some unused macros

Merge branch 'add-tools-test' into 'master'

Add tools test

See merge request graphviz/graphviz!1592

Merge branch '7f9609f1-0ed6-4616-a6b4-bf59bac9919f' into 'master'

fix build system support for pstopdf

See merge request graphviz/graphviz!1591

fix: pstopdf support in the Autotools build system

pstopdf can now be used as a substitute for ps2pdf when using the Autotools
build system. This commit adapts a downstream packaging tweak from Homebrew [0]
commit f231dcfd69a290c8a289bccd8cfc9aa9b8dc5472, that now works correctly as of
the previous commit. Related to #1763.

  [0]: https://github.com/Homebrew/homebrew-core

remove explicit output filename in PDF Makefile rules

For all of these rules, the output filename ps2pdf infers is the same as the
explicit one being removed, so this change is a no-op. The intent here is to
make these invocations compatible with either ps2pdf or pstopdf filling the role
of @PS2PDF@. The usage for these two differs slightly:

  ps2pdf infile [outfile]
  pstopdf infile [-o outfile]

pstopdf is not currently supported -- there is an attempt at this in
configure.ac, but it is faulty -- but we plan to support it in an upcoming
commit. Related to #1763.

Update CHANGELOG with tools usage and other fixes

Skip execution of dot_builtins in tools test for Centos and Fedora builds

Fails because dot_builtins is not installed for Ubuntu or Centos. See
https://gitlab.com/graphviz/graphviz/-/issues/1839.

Skip execution of all tools not built with MSBuild in tools test for MSBuild builds

Fails because not all tools are built with MSBuild. See
https://gitlab.com/graphviz/graphviz/-/issues/1837.

Add all tools built with autotools on Ubuntu 18.04 to tools test

Fix gvmap.sh -? option to exit with zero exit status

Correct shebang of gvmap.sh to use ksh

Add option -? for usage to vimdot

Skip execution of gvpack in tools test for non-Windows CMake builds

Fails becasue gvpack does not find libgvplugin_neato_layout.so.6 when
built with CMake. See
https://gitlab.com/graphviz/graphviz/-/issues/1838.

Add check of usage to tools test

Fix edgepaint -? to actually print usage instead of error

Fix edgepaint -o option

Fix apparently incorrect indentation in mingle

This change corrects hard-to-read code by changing a few instances of
four spaces to tabs which is the prevalent indentation method in the
surrounding code. There are more incorrect indentation in this file,
but it is left untouched since it is less distracting.

Correct misspelled 'smyrna' in usage

Remove '"' from usage text in non-Windows version of dotty

Removed both from dotty.bsh and dotty.sh which is the one that is
released as dotty.

Skip execution of all tools not built with CMake in tools test for CMake builds

Fails because not all tools are built with CMake. See
https://gitlab.com/graphviz/graphviz/-/issues/1753 and
https://gitlab.com/graphviz/graphviz/-/issues/1836.

Skip execution of mingle in tools test for Ubuntu and Centos builds

Fails because mingle is not built for Ubuntu and Centos. See
https://gitlab.com/graphviz/graphviz/-/issues/1835.

Skip execution of smyrna in tools test for Centos builds

Fails because smyrna is not built for Centos. See
https://gitlab.com/graphviz/graphviz/-/issues/1834.

Skip execution of smyrna in tools test for Windows MSBuilds

Fails because of missing DLLs. See
https://gitlab.com/graphviz/graphviz/-/issues/1829.

Add new tools test that tests that all tools can be started

Propagate OS ID to pytest

Fix smyrna -? to actually print usage instead of error

Add option -? for usage to non-Windows version of lneato

Added the option to lneato.ksh, lneato.bsh and lneato.sh which is the
one that is released as lneato.

Add option -? for usage to non-Windows version of dotty

Add option -? for usage to Windows version of lneato

Add option -? for usage to Windows version of dotty

Add option -? for usage to diffimg

Add archiving of test report also from tests on Windows

Merge branch '0943AE25-50C6-419B-8583-E855B1BA66AA' into 'master'

fix name of rebuild_vlists in an error message

See merge request graphviz/graphviz!1596

Merge branch 'master' into HEAD

add changelog note about E2K support

Related to !1595.

Merge branch '9B333B00-0CA0-4019-8DF1-DC96CA0EFF97' into 'master'

switch to ustar POSIX format for tarballs produced by make dist

Closes #1830

See merge request graphviz/graphviz!1590

fix name of rebuild_vlists in an error message

switch to ustar POSIX format for tarballs produced by make dist

Without setting this option, Automake was defaulting to the “V7” format that
only supports filenames up to 99 characters. This was causing the following
(unnoticed) error during portable source generation:

  tardir=graphviz-2.44.2~dev.20200921.1021 && ${TAR-tar} chof - "$tardir" | eval GZIP= gzip --best -c >graphviz-2.44.2~dev.20200921.1021.tar.gz
  tar: graphviz-2.44.2~dev.20200921.1021/macosx/graphviz.xcodeproj/project.xcworkspace/contents.xcworkspacedata: file name is too long (max 99); not dumped
  tar: graphviz-2.44.2~dev.20200921.1021/macosx/graphviz.xcodeproj/project.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist: file name is too long (max 99); not dumped
  tar: Exiting with failure status due to previous errors

In this commit, we switch to the newer “ustar” format that supports filenames
up to 256 characters. This format is still widely supported. From the Automake
documentation [0]:

  tar-ustar selects the ustar format defined by POSIX 1003.1-1988. This format
  is old enough to be portable: As of 2018, it is supported by the native tar
  command on GNU, FreeBSD, NetBSD, OpenBSD, AIX, HP-UX, and Solaris, at least.
  It fully supports empty directories. It can store file names with up to 256
  characters, provided that the file name can be split at directory separator in
  two parts, first of them being at most 155 bytes long. So, in most cases the
  maximum file name length will be shorter than 256 characters.

Closes #1830.

  [0]: https://www.gnu.org/software/automake/manual/html_node/List-of-Automake-options.html

Merge branch 'install-plugins-at-build-time' into 'master'

Configure plugins at build time

See merge request graphviz/graphviz!1581

Remove now obsolete exec of dot -c for Windows CI builds

Add install of plugin configuration file config6 for Windows MSBuild builds

Add gd and gdiplus plugins dependency to dot for MSBuild

This will ensure that the plugin configuration file config6, which
will be generated directly after dot is built in an upcoming commit,
contains all plugins.

Remove now obsolete exec of dot -c for Linux CMake CI builds

Add install of plugin configuration file config6 for CMake builds

Merge branch '06e77d83-9cd7-4e6a-b814-d74136617645' into 'master'

remove more unused files

See merge request graphviz/graphviz!1589

remove unused sfvalue()

remove unused sfulen()

remove unused sfstacked()

remove unused sfputu() and variants

remove unused sfputm() and variants

remove unused sfputl() and variants

remove unused sfputd() and variants

remove unused sfllen() and variants

remove unused sferror()

remove unused sfeof()

remove unused sfdlen() and variants

remove unused sfclrerr()

Related to #19. I do not believe this is a full fix. It just bypasses the first
error in the referenced issue.

configure.ac: Add e2k platform

Elbrus (also known as Elbrus 2000) is a Russian 64-bit LE VLIW.

Merge branch '22783486-1007-4063-bcc5-c902a9bf8dc7' into 'master'

remove direct use of 'environ'

See merge request graphviz/graphviz!1587

remove build system detection of _NSGetEnviron() and crt_externs.h

Neither of these are used any more.

remove now unnecessary environ abstraction code

remove direct use of environ

This code was manually parsing environment variable settings for some reason. It
also (incorrectly) assumed $_ would always be the first environment variable.

remove unused uid_t #define

Merge branch '1785-4' into 'master'

disambiguate #includes (4/5)

See merge request graphviz/graphviz!1588

disambiguate tclpkg/tclpathplan #includes

Related to #1242 and #1785.

remove unnecessary tclpkg/tclpathplan include path

disambiguate tclpkg/tcldot #includes

Related to #1242 and #1785.

remove unnecessary tclpkg/tcldot include path

disambiguate tclpkg/gv #includes

Related to #1242 and #1785.

remove unnecessary tclpkg/gv include path

Merge branch '4a751607-966c-4130-84cb-77a26d194de8' into 'master'

drop explicit libstdc++ references, allowing linking with libc++

Closes #163

See merge request graphviz/graphviz!1586

remove explicit linking of libstdc++

The autotools build should now be compatible with libc++. Closes #163.

force the C++ compiler to be used when linking C++-dependent targets

This looks pretty hacky, but is apparently the libtool-sanctioned way of doing
this [0]. Related to #163.

  [0]: https://www.gnu.org/software/automake/manual/automake.html#Libtool-Convenience-Libraries

remove duplicated Makefile content

Merge branch '23CD380E-C5C0-414F-B23F-122C42B26417' into 'master'

remove testing pieces from CMake build

Closes #1795

See merge request graphviz/graphviz!1584

remove testing pieces from CMake build

The existing test fragments in the CMake files had two undesirable effects:

  1. Projects using Graphviz as a subdirectory would pick up this test suite and
     it would get run as part of their own test suite. See #1795.

  2. The fragments incorrectly gave the impression this was the canonical set of
     Graphviz tests and/or that this is what was run in CI. The current reality
     is that CI ignores this path to a subset of the tests, and most of the
     maintainers do not run this either.

remove build system workaround for GCC 3

We no longer support anything below GCC 4.4.7.

don't squash -O2 when using ICC

ICC is Intel's C Compiler. All recent versions of it *do* support `-O2`.

Merge branch '803B960B-999F-41B4-981C-21E1EB0BB124' into 'master'

remove eFence support

Closes #1827

See merge request graphviz/graphviz!1585

remove eFence support

The eFence memory debugging tool has these days been superseded by Valgrind and
Address Sanitizer. Closes #1827.

remove commented out library checks

remove legacy comment

Merge branch '6B1AB424-B66A-42CC-B5CF-BAC9AEEF6129' into 'master'

miscellaneous cleanup

See merge request graphviz/graphviz!1583

remove ignore rules from some legacy files

libgraph was removed in d5f98556f654b41e5fdabd2d6f7528256bdea519.

remove commented out strToL

remove Windows strto[u]ll redirects

These functions exist on Windows. There is no need to remap them.

remove unused 'external'

remove unused 'cast'

Merge branch 'fix-linux-cmake-and-add-test-job-to-ci' into 'master'

Add Linux CMake test job to CI (including needed generation of a .deb package and correction of some linking problems)

See merge request graphviz/graphviz!1578

Add skip of format png:gd also for Linux CMake rtest

Add skip of PNG diff for also for Linux CMake rtest

Add skip of test_1813 also for Linux CMake

Add ubuntu18-04-cmake-test job to CI

Also adapt existing test jobs to accomodate this.

Fix lib/pack objs not included in lib/gvc dynamic lib for CMake builds

This is what the autotools builds do.

For some reason this was not a problem on Windows.

Without this, the rtest/test_examples.py gave errors like this for neatopack.c:

neatopack.c:(.text+0x81): undefined reference to `ccomps'