Merge branch 'use-gitlab-containers' into 'master'

Pull images from GitLab Container Registry in CI

See merge request graphviz/graphviz!1750

Pull images from GitLab Container Registry in CI

GitLab's container registry is pretty well-integrated with building the images, easier than pushing to DockerHub.

Less keys to manage too.

Towards #1943

Merge branch 'powertools' into 'master'

Attempt fixing CentOS8 Dockerfile

See merge request graphviz/graphviz!1749

Attempt fixing CentOS8 Dockerfile

Apparently this is now powertools, lowercase:

https://serverfault.com/questions/997896/how-to-enable-powertools-repository-in-centos-8

Towards #1945

Merge branch '1910' into 'master'

Delete ubuntu-19.10 CI

Closes #1944

See merge request graphviz/graphviz!1748

Delete ubuntu-19.10 CI

It's discontinued release, can't build Dockerfile any more.

Fixes #1944

Merge branch 'docker-in-ci' into 'master'

Touch all the Dockerfiles to rebuild them

See merge request graphviz/graphviz!1747

Touch all the Dockerfiles to rebuild them

And push them to the GitLab Container Registry

There's probably a nicer way to do this, but I'm not sure what it is. Maybe a manual job, or a variable that you can enable to trigger it?

Towards #1943

Merge branch 'docker-in-ci' into 'master'

Add CI building of all Docker images

See merge request graphviz/graphviz!1746

Add CI building of all Docker images

Only when we edit their respective Dockerfiles.

Towards #1943

Merge branch 'smattr/EFD0F12C-1BD3-412D-8DEE-AF6E0B80C486' into 'master'

fix memory leak in mingle

See merge request graphviz/graphviz!1736

Merge branch 'docker-in-ci' into 'master'

Experimental: build docker image in GitLab CI

See merge request graphviz/graphviz!1745

fix: stop manually managing kdTree memory in mingle

This actually fixes a bug where the first kdTree allocation was being leaked due
to never being deleted. Related to #1832.

remove manual memory management of dists in mingle

Related to #1832.

remove manual memory management of nnIdx in mingle

Related to #1832.

fix typo in mingle comment

remove unnecessary using directive and #includes

This file does not reference any symbols from these.

Experimental: build docker image in GitLab CI

Starting with just centos7

Following the instructions at https://about.gitlab.com/blog/2019/08/28/building-build-images/.

Haven't set up the environment variables yet.

Merge branch 'smattr/a8939803-4471-4750-b95b-5aa9fcb70802' into 'master'

remove unnecessary casts of cdt functions

See merge request graphviz/graphviz!1735

remove no-op cast

remove unnecessary casts of dtsearch() return value

These pointers implicitly coerce.

remove unnecessary casts of dtprev() return value

These pointers implicitly coerce.

remove unnecessary casts of dtlast() return values

These pointers implicitly coerce.

remove unnecessary casts of dtnext() return value

These pointers implicitly coerce.

remove unnecessary cast of dtfirst() return value

These pointers implicitly coerce.

remove unnecessary casts of dtmatch() return values

These pointers implicitly coerce.

Merge branch 'smattr/87474991-d226-4c2a-8e48-daedb4cdde1c' into 'master'

Keep the edge order in json output (!1728 plus test case)

See merge request graphviz/graphviz!1734

Keep the edge order in json output

expand a wrapper around calloc()

We're about to #include another header in this file that contains its own
conflicting definition of NEW().

add some test cases of the JSON output

Merge branch 'smattr/DC781A13-F00C-434C-A18F-6ACC0AF4B3CC' into 'master'

clean up some legacy unused config checks

See merge request graphviz/graphviz!1733

fix: remove orphaned MALLOC_H define

The logic to populate this was removed in
59bf93da69561600015ee3f2b747691d52228188.

remove mstats() detection

Graphviz does not depend on this.

remove mallinfo() detection

Nothing in Graphviz depends on this.

remove mallopt() detection

Nothing in Graphviz depends on this.

Merge branch 'smattr/34694182-7a40-4f06-abc7-e1218f680248' into 'master'

fix incorrect PS colors generated by Lefty and enable C99 warnings

See merge request graphviz/graphviz!1730

enable some additional C99 compiler warnings

These can now be turned on without adding warnings to the Autotools build.

fix incorrect colors in Lefty postscript output

This code intended to print color data to its output file in postscript (PS)
format. However it miscalculated the low nibble of each non-zero byte as 1 due
to an incorrect operator. This may have not been noticed in the resulting image
due to low precision. This was exposed by the compiler flag -Wlogical-op. Fixes
#1927.

At first glance, it looks like this could be written without shifting and
masking at all by simply printing the byte padded to width 2, but maybe I am
missing some subtlety.

Merge branch 'smattr/1FE7F116-BD57-44DB-A463-C712472A1CA6' into 'master'

add details of updating the website to the release procedure

See merge request graphviz/graphviz!1729

add details of updating the website to the release procedure

This is addressing the following point I made in the 2.46.0 retrospective:

  We probably need to incorporate www2.graphviz.org and graphviz.gitlab.io
  actions into the release steps in DEVELOPERS.md.

It would be nice to further automate some of this in future.

Merge branch 'smattr/7654BD6A-00FB-419C-9CEE-58F314E9471A' into 'master'

fix HTML validation issues in FAQ.html

See merge request graphviz/graphviz!1727

fix HTML validation issues in FAQ.html

Related to #1861.

Merge branch 'smattr/E0AA0C62-2E41-45C8-8F0E-C7CF13B57772' into 'master'

remove Python 2 pytest installation in CI

Closes #1886

See merge request graphviz/graphviz!1726

remove Python 2 pytest installation in CI

We only support running the test suite with Python 3. Closes #1886.

Merge branch 'smattr/eb990658-b367-436d-84a5-fcb9dec533c3' into 'master'

remove dead suffix parsing code

Closes #1933

See merge request graphviz/graphviz!1725

delete no longer used strton

remove dead code for suffix handling of parsed numbers

This code attempted to handle a numeric suffix indicating a multiplier (e.g. 'k'
for 1024). However, the preceding logic that constructs a temporary buffer for
it to parse from does not anticipate suffixes and copy them to the buffer. The
(inactive) suffix code is considered only useful for describing file sizes, so
not relevant to enable for Graphviz. Closes #1933.

Merge branch 'smattr/D1537418-F027-4305-8650-1E45D1B2F6B1' into 'master'

update Graphviz URLs in comments

See merge request graphviz/graphviz!1724

update Graphviz URLs in comments

Merge 'Start 2.46.2 development series' into master

Start 2.46.2 development series

Merge branch 'smattr/63291F41-E610-4E9F-B12E-1566969C0FDA' into 'master'

Stable Release 2.46.1

See merge request graphviz/graphviz!1732

Stable Release 2.46.1

Merge branch 'smattr/DD8B4ECD-399B-4BEF-AD77-3EF40BA4402E' into 'master'

Properly fill graphviz_version.h (!1706 plus test case)

See merge request graphviz/graphviz!1723

Properly fill graphviz_version.h

Make sure the variables used inside the header template are populated.

Prior to this patch we ended up with this in the generated
graphviz_version.h:
  #define PACKAGE_VERSION ""

Edit by Matthew Fernandez: Added changelog entry and flipped polarity on the
CMake test case for this.

add a test of PACKAGE_VERSION

Related to !1706.

Merge branch 'smattr/D39EA669-23A2-4D0E-A8BD-7FC55FC1C35E' into 'master'

apply some C99 to vmalloc

See merge request graphviz/graphviz!1722

#pragma once instead of old-style header guards in vmalloc

These are supported by all recent compilers.

remove Vim modeline markers from vmalloc

These have no effect for non-Vim users or Vim users with modeline disabled.
clang-format is a better modern solution to this problem.

take advantage of some C99 features in vmalloc

compile vmalloc tests with C99

This should have been included in fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc.

Merge branch 'smattr/B7B7989C-A893-415A-B27A-B365C1D44997' into 'master'

drop support for Fedora 31

See merge request graphviz/graphviz!1721

drop support for Fedora 31

Fedora 31 was EOLed in November 2020 [0].

  [0]: https://lists.fedoraproject.org/archives/list/announce@lists.fedoraproject.org/thread/NU5AENRUFG4XK5D34SJN5FZPLYMZF6ZQ/

move Fedora CMake testing to Fedora 32

Fedora 31 was EOLed in November 2020 [0].

  [0]: https://lists.fedoraproject.org/archives/list/announce@lists.fedoraproject.org/thread/NU5AENRUFG4XK5D34SJN5FZPLYMZF6ZQ/

Merge branch 'smattr/77EC4CE8-9933-43C1-B3BB-B9F65F3901ED' into 'master'

remove alternate vsnprintf code

See merge request graphviz/graphviz!1720

remove some unnecessary casts

remove vsnprintf detection

We no longer depend on this and simply assume vsnprintf is available.

remove alternate vsnprintf paths

As of fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc, we build with C99 where
vsnprintf is require. Hence we no longer need other code paths.

Merge branch 'smattr/F614351E-E612-4E14-A869-F958C2FDF07E' into 'master'

use release version for generic package version when possible

See merge request graphviz/graphviz!1719

use release version for generic package version when possible

This addresses my comment from the 2.46.0 retrospective:

  I realized after the fact that, though the generic package versions have a
  restricted format, our release versions fit that format. I.e. we could have
  the generated 0.0.287364… versions for commit snapshots, but for packages that
  contain a release we could use a version that is the same as the release
  version itself, e.g. 2.46.0.

Now generic packages that correspond to a stable release will get the same
generic package version as the release version itself. This should avoid some
user confusion in future.

reorder version computation steps in deployment

abbreviate some use of enumerate()

Merge branch 'smattr/DA75A76F-4160-460E-8F0E-E2159B71E1C2' into 'master'

migrate test suite to Python 3.6+

Closes #1870

See merge request graphviz/graphviz!1718

use f-strings in Python test scripts

Now that we have Python 3.6 as a base line requirement, we can use f-strings
pervasively. Related to #1870.

use Pathlib for path-typed variables in the test suite

As of Python 3.4, this is the Pythonic way of dealing with file system paths.

set a new base line of Python 3.6 for running the test suite

Closes #1870.

Merge branch 'smattr/19153a68-1b2c-4716-962d-c07b057af4ed' into 'master'

fix: no longer lose \n within quoted strings

Closes #1931

See merge request graphviz/graphviz!1717

fix: no longer lose \n within quoted strings

9bdd24cae96d01fc6b2a940bc87d27edeb5d84a7 addressed a problem where line number
tracking was not working across new line characters embedded within quoted
strings. However it accidentally unmasked another bug involving a parsing rule
that swallowed new lines which was incorrectly enabled within quoted strings.
Fixes #1931.

test case for #1931

Merge branch 'smattr/56867813-653e-4fc1-822a-4d4f56154de5' into 'master'

fix: add header guard to graphviz_version.h

Closes #1929

See merge request graphviz/graphviz!1716

fix: add header guard to graphviz_version.h

To prevent users #including this multiple times getting compiler warnings.
Closes #1929.

Merge branch 'smattr/850CBB25-AAEA-471D-BB06-0C826F1276EF' into 'master'

some rtest.py tweaks and generate a .tar.xz portable source

Closes #454

See merge request graphviz/graphviz!1715

also build a .tar.xz archive for distribution

Closes #454.

fix: teach rtest.py how to find its default files regardless of CWD

If rtest.py is run from a different directory -- e.g. from a top-level make
check-recursive -- it could not find its default inputs.

fix: make tools findable by rtest.py in a `make check-recursive`

fix path to rtest.py when triggered by `make check-recursive`

Merge !1714 into master

Remove dead assignments from SparseMatrix_import_matrix_market

Remove dead assignment from extoken_fn

Remove dead assignment from makeDotGraph

Remove dead assignment from GFXinit

Remove dead assignment from scan_num

Remove dead assignments from mincross_step

Remove dead assignment from SparseMatrix_coordinate_form_add_entries

Remove dead assignment from constrained_majorization_new

Remove dead assignments from self(Bottom|Top|Right|Left)

Remove dead assignment from parseSegs

Remove dead assignment from mincross

Remove dead assignment from edge_distinct_coloring

Remove dead assignment from map_path