> - should be using upstream's distribution plus whatever patches are

I'm working with the ZoneMinder lead, Isaac Connor, to get this ported.  Right
now, he maintains a separate OpenBSD GitHub branch that is occasionally
synced to the master branch, thus falling behind version numbers slightly.
In turn, I've (temporarily) forked ZoneMinder into my own repository and am
making changes that get merged back upstream via GitHub PRs (I do not have or
need direct commit access to the main ZM GitHub repository).  I'm sure we can
fix this separation and need for my own repository in the near future, so
consider this as part of the testing process for now.

> - LIB_DEPENDS are only used where there is an actual library dependency

I must admit that I don't understand the difference between LIB_DEPENDS,
WANTLIB, and SHARED_LIBS. My current Makefile (which is no doubt not fully
correct) appears to work, but when I tried WANTLIB things seemed to go wrong
(which I am sure is due to a combination of lack of understanding or
experience with the ports process).  I'll try some variations and report
back.  Advice and guidance is welcome.

> - zm should run with nginx too

While I'm sure this is technically correct, I'm not sure this is the
supported and recommended configuration from the ZoneMinder project.  Let me
ask Isaac and get back to you.  Assuming this is confirmed, is there an
overall issue with an Apache dependency from this port?  Or should the
user need to follow separate instructions to install and configure Apache? I
would assume this goes for MySQL/MariaDB as well.  Bear in mind, these kind
of assumed dependencies come from the Linux world where this is generally
accepted, so if this is not the OpenBSD way they can easily be removed.

> - generally I wouldn't force deps on both mysql and pgsql extensions

I have no idea why I included pgsql, don't think it's needed. I'll remove it.
I don't think ZoneMinder supports pgsql anyway.

I will clean up and try to incorporate other changes & advice I've received
and post another version for testing.  Bear in mind that the source will
probably still come from my GitHub fork until I can work out the details
with Isaac.

-vansid

Reply via email to