MASTER_SITES needs updating, perhaps it's also worth considering migrating to CMake as that's what upstream uses for CI and fix make test.
It also seems like it incorrectly overrides default CFLAGS -g0 -Os (optimization enabled), -g0 -O0 if disabled (which is fine I guess).

dns/c-ares: Update 1.15.0_1 -> 1.16.1

Harbormaster completed remote builds in B31557: Diff 72813.Jun 8 2020, 5:00 AM

Why does MASTER_SITES need updating? To what?

Also, I don't see anything in the port that changes CFLAGS?

Hi, sorry for being vague

http://obsd.isc.org/pub/OpenBSD/distfiles/ doesn't resolve and ZI's sites doesn't carry it.

During compilation I see the following: (without OPTIMIZED CFLAGS enabled), -Os enabled
https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/dads-cflags.html

libtool: compile:  cc -DHAVE_CONFIG_H -I. -I. -DCARES_BUILDING_LIBRARY -DCARES_SYMBOL_HIDING -fvisibility=hidden -pipe -mcpu=cortex-a53 -fno-strict-aliasing -Qunused-arguments -g0 -O0 -MT libcares_la-ares__read_line.lo -MD -MP -MF .deps/libcares_la-ares__read_line.Tpo -c ares__read_line.c -o libcares_la-ares__read_line.o >/dev/null 2>&1

From what I can tell this comes from
L19735 and line 19736 in ./configure

Also, converting to CMake would be a bad idea as it would introduce circular depdencies sorry for not realizing that earlier.

Removed obsd.isc.org from MASTER_SITES

Harbormaster completed remote builds in B31631: Diff 72968.Jun 11 2020, 4:58 AM

bhughes retitled this revision from dns/c-ares: Update 1.15.0_1 -> 1.16.0 to dns/c-ares: Update 1.15.0_1 -> 1.16.1.Jun 11 2020, 4:59 AM

bhughes edited the summary of this revision. (Show Details)

This revision was not accepted when it landed; it landed in state Needs Review.Jun 19 2020, 4:56 AM

Closed by commit rP539584: dns/c-ares: Update 1.15.0_1 -> 1.16.1 (authored by bhughes). · Explain Why

This revision was automatically updated to reflect the committed changes.

bhughes added a commit: rP539584: dns/c-ares: Update 1.15.0_1 -> 1.16.1.