Allow testing docker_push step in my own repo

With my own container registry

Before, I mistakenly thought that merge requests ran with the
credentials of graphviz/graphviz, now I see they only run with the
credentials of the source repo (markhnsn/graphviz). So it's safe to run
this on any master branch, even forks.

Merge branch 'smoosh' into 'master'

docker: Split building SHA and pushing to :latest

See merge request graphviz/graphviz!1776

docker: Split building SHA and pushing to :latest

Previously, we guarded the :latest tag by limiting it to master only,
not merge requests.

Now we need to split this:
- it's safe for merge requests to push to their own SHA, nobody else is
  using that SHA
- the merge request tests assume that a docker image is tagged with a
  SHA, so the merge request tests require the docker_build step to run.

I'm a bit disappointed I couldn't get this to work in a single step
(hence the undesirable duplication) -- perhaps there's a way to lock
down :latest within a single CI job that I'm missing?

This commit is loosely based off
https://blog.callr.tech/building-docker-images-with-gitlab-ci-best-practices/

Merge branch 'smoosh' into 'master'

Bugfix: invalid spec

See merge request graphviz/graphviz!1775

Bugfix: invalid spec

Forgot to capitalize this

Caused:
ERROR: Preparation failed: failed to pull image
"registry.gitlab.com/graphviz/graphviz/:8437020a9de5bbd812cdc4cb09c94c349366631c"
with specified policies [always]: invalid reference format
(manager.go:205:0s)

Merge branch 'smoosh' into 'master'

Stop pushing to deprecated per-image tags

See merge request graphviz/graphviz!1774

Stop pushing to deprecated per-image tags

The tests are now all using separate images rather than tags on the one
image.

Merge branch 'smoosh' into 'master'

Use Git SHAs fetching build and test docker images

See merge request graphviz/graphviz!1773

Use Git SHAs fetching build and test docker images

Should fix a race condition on the :latest label

Took the chance to do a bit of deduplication. Not sure if I love the
result, but it does mean there's one place to edit for the schema of the
docker label to use.

Merge branch 'smoosh' into 'master'

Bugfix: should have been /$IMAGE:latest

See merge request graphviz/graphviz!1772

Bugfix: should have been /$IMAGE:latest

This was pushing all the different OS images to the top-level
graphviz/graphviz:latest package before.

Merge branch 'smoosh' into 'master'

No-op: Use CI_IMAGE_REGISTRY variable

See merge request graphviz/graphviz!1771

No-op: Use CI_IMAGE_REGISTRY variable

Reduces some typing. Expands to the same thing.

Merge branch 'smoosh' into 'master'

Reuse docker images where possible

See merge request graphviz/graphviz!1770

Reuse docker images where possible

Should speed up CI docker image building.

Inspiration from
https://blog.callr.tech/building-docker-images-with-gitlab-ci-best-practices/

Merge branch 'smoosh' into 'master'

Push Docker images to 'latest' tag too

See merge request graphviz/graphviz!1769

Push Docker images to 'latest' tag too

We'll use this for caching soon

Inspired by
https://blog.callr.tech/building-docker-images-with-gitlab-ci-best-practices/

Merge branch 'smoosh' into 'master'

Stop docker complaining about password argument

See merge request graphviz/graphviz!1768

Stop docker complaining about password argument

Fixes this warning:

```
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
```

With thanks to
https://blog.callr.tech/building-docker-images-with-gitlab-ci-best-practices/
for the inspiration.

Merge branch 'smoosh' into 'master'

Fix error: "docker build" requires exactly 1 argument.

See merge request graphviz/graphviz!1767

Fix error: "docker build" requires exactly 1 argument.

Oops, forgot the dot last commit.

Merge branch 'smoosh' into 'master'

Push to git-SHA Docker image label

See merge request graphviz/graphviz!1766

Push to git-SHA Docker image label

Keep the old image labels for backwards compatibility, just for now

Remove the filename scope: I want this to run every time we push, just
for now. We'll make this faster with caching soon.

Merge branch 'smoosh' into 'master'

No-op: consolidate Docker Image building YAML

See merge request graphviz/graphviz!1765

No-op: consolidate Docker Image building YAML

This should have the same effect while making it easier to change the
common docker image building code.

Merge branch 'smattr/faedcca5-e012-4920-8e45-5457d2ff984b' into 'master'

misc clean up

See merge request graphviz/graphviz!1740

squash some -Wimplicit-fallthrough compiler warnings

remove some unnecessary void* casts

remove our own CHAR_BIT definitions

We can just use those in the C standard library.

Merge branch 'smattr/150C537B-D311-4C5E-B421-A4874CE8C5F4'

add Fedora to the noted list of downstream consumers

Related to #1941.

Merge branch 'smattr/E488F2BC-9394-4031-AB3D-A8E6BC313197' into 'master'

remove some unnecessary sfstropen use

See merge request graphviz/graphviz!1738

remove another use of sfstropen in GVPR

Related to #1873.

remove an unnecessary use of sfstropen in GVPR

Related to #1873.

Merge branch 'smattr/25B6AB1C-1F36-4CEE-87BD-F608C7459480' into 'master'

remove recapitulated prototypes of strcasecmp

See merge request graphviz/graphviz!1741

remove recapitulated prototypes of strcasecmp

These actually cause warnings in CMake Windows builds:

  …\lib\cgraph/strcasecmp.h(15,47): warning C4273: '_stricmp': inconsistent
    dll linkage […\graphviz\build\plugin\gd\gvplugin_gd.vcxproj]
  C:\Program Files (x86)\Windows Kits\10\Include\10.0.18362.0\ucrt\string.h(193,22):
    message : see previous definition of '_stricmp'
    […\build\plugin\gd\gvplugin_gd.vcxproj]

and MSBuild Windows builds:

  …\lib\cgraph\strcasecmp.h(23,64): warning C4211: nonstandard extension used:
    redefined extern to static […\graphviz\lib\cgraph\cgraph.vcxproj]
  …\lib\cgraph\strcasecmp.h(27,75): warning C4211: nonstandard extension used:
    redefined extern to static […\lib\cgraph\cgraph.vcxproj]

and Cygwin MinGW builds:

  …/lib/cgraph/strcasecmp.h:15:5: warning: '_stricmp' redeclared without
    dllimport attribute: previous dllimport ignored [-Wattributes]
     15 | int strcasecmp(const char *s1, const char *s2);
        |     ^~~~~~~~~~
  …/lib/cgraph/strcasecmp.h:16:5: warning: '_strnicmp' redeclared without
    dllimport attribute: previous dllimport ignored [-Wattributes]
     16 | int strncasecmp(const char *s1, const char *s2, size_t n);
        |     ^~~~~~~~~~~

Related to #1940.

Merge branch 'add-gvpr-to-cmake-builds' into 'master'

Add gvpr to CMake builds

Closes #1878

See merge request graphviz/graphviz!1684

Remove unnecessary declaration of libc time function

Is available through '#include <time.h>'.

Remove unused sfexit.c and replace waitpid declaration with std

Remove gvpr from tools not built with CMake in ci/tests.py

Add skip of test_gvpr_example for 64-bit Windows builds

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

Add skip of gvpr bbox & col test also for Windows CMake builds

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

Fix gvpr -? to actually print usage and exit non-zero

Add build of cmd/gvpr using CMake

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

Add build of lib/gvpr using CMake

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

Add build of lib/expr using CMake

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

Add build of lib/ast using CMake

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

Add build of lib/sfio using CMake

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

Add missing lib/pack CMake dependency to lib/gvc

Merge branch 'smattr/10F637A0-9721-4680-A7BA-37B3CBE3BC2F' into 'master'

silence some markdownlint warnings

See merge request graphviz/graphviz!1761

silence some markdownlint warnings

Merge branch 'smattr/056A5DAD-1EFB-41E9-BDBF-737CB8FA0DA2' into 'master'

fix: use public inheritance in VPSC hierarchy

See merge request graphviz/graphviz!1737

remove now-unnecessary cast

Related to #1874.

fix: use public inheritance in VPSC hierarchy

This removes some C++ undefined behavior where libvpsc casts of IncVPSC pointers
to VPSC pointers were invalid due to private inheritance. There was no need to
use private inheritance here, so we now simply use public inheritance. Fixes
#1874.

Merge branch 'smattr/8d9e1781-cb6e-4eb7-a5fb-7d1cca23f690' into 'master'

remove use of regex for identifying plugin libraries

See merge request graphviz/graphviz!1731

remove use of regex for identifying plugin libraries

This removes some dynamic allocations and failure paths, accelerating the
determination of the "is this library a Graphviz plugin?" check. Though the root
motivation here is removing a dependency on regular expressions. The resulting
code being more efficient is a pleasant side effect. A step along the way to
#1919.

remove unused DMKEY

Merge branch 'smoosh' into 'master'

Smoosh some Dockerfile layers into one

See merge request graphviz/graphviz!1753

Smoosh some Dockerfile layers into one

This is kind of a test for the new automatic building of dockerfiles.

This should make for smaller images and slightly faster CI runs.

The tradeoff is slower local builds if you'd cached the earlier bits.

But now we can easily build these on the cloud when changed.

Merge branch 'remove-comment' into 'master'

Remove my hacky comment that forced CI builds of Dockerfiles

See merge request graphviz/graphviz!1752

Remove my hacky comment that forced CI builds of Dockerfiles

Cleanup for #1943

Merge branch 'list' into 'master'

Fix python typing warning

See merge request graphviz/graphviz!1744

Merge branch 'cmake' into 'master'

Install cmake in ubuntu-18.04 image

See merge request graphviz/graphviz!1751

Install cmake in ubuntu-18.04 image

I suppose sometime, canonical must have removed cmake from the deps of another package we installed.

Towards #1946

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 python typing warning

"""
List expression not allowed in type annotation
  Use List[T] to indicate a list type or Union[T1, T2] to indicate a union typePylance
"""

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.