On Sat, September 16, 2023 04:57, Anthony J. Bentley wrote:
> Kirill Bychkov writes:
>> Attached is a port of VCMI-1.3.1, an open source Heroes
>> of Might and Magic III engine.
>> Played some campaigns and scenarios.
>
> Cool, this works with my GOG copy. Some comments:
>
> - The patches against included minizip are unnecessary since the port
> uses system minizip.
I pushed vcmi to openbsd-wip 10 years ago. I guess it is a heritage of the
days we have no minizip in ports. I'll fix that.
>
> - The second hunk of patch-lib_Interprocess.h seems unintentional. The
> first hunk also doesn't appear to be needed either?
Peobably. Will test without it.
>
> - Is -DUSE_FILE32API another leftover that can be removed?
I'll check.
>
> - CXXFLAGS=-I${X11BASE}/include can be rolled into CMAKE_CXX_FLAGS.
Sure.
>
> - It takes a long time to compile, so debug packages should be enabled,
> imo.
>
> - Can this use GH_* instead of a self-hosted distfile, so portroach can
> catch updates?
I'm self-hosting distfile because it needs to be recursively checked out
from git due to external submodules.
>
> - Looks like 1.3.2 was released this morning.
>
Nice. If there is no problems with new version, I'll update the port.
Thanks!
