Re: NEW sysutils/openvox-server

Hi Klemens,

same here, tried to address your concerns. Updated tarball attached.

See inline below.

On Sat, Jan 31, 2026 at 2:58 PM Klemens Nanni <kn@openbsd.org> wrote:

28.01.2026 16:48, Sebastian Reitenbach пишет:
> Hi, 
>
> cat pkg/DESCR: 
> OpenVox is configured in an agent-server architecture, in which a
> primary server node manages the configuration information for a fleet of
> agent nodes. OpenVox Server acts as the primary server node. OpenVox
> Server is a Ruby and Clojure application that runs on the Java Virtual
> Machine (JVM). OpenVox Server runs Ruby code for compiling Puppet
> catalogs and for serving files in several JRuby interpreters. It also
> provides a certificate authority through Clojure.
>
> It is a drop in replacement for Puppetserver.
>
> with that, the legacy sysutils/puppetserver/8 port could bite the dust. The version there is loooong time EOL.

That'd be nice.

>
> between OpenVox server 8.11.x and 8.12.x they removed the Makefile, therefore the old Makefile is taken and stored in the files section. otherwise, could implement it directly in the do-install: section in the ports Makefile. But I found this way easier and a bit cleaner, but if there are concerns, I'd take it on and migrate the target into the do-install: if necessary.

See my openvoxdb response.  In addition to that

- MODJAVA_RUN_DEPENDS is not needed in RUN_DEPENDS,
  the java module already does that

removed.

 

- as mentioned in openvoxserver-ca, I suggest dropping
  ruby flavors, so the dep here would need adjusting

adjusted.

 

- SUBST_CMD purely on WRKSRC files seems odd, why not
  do that in post-patch?

done, 

 

This files/Makefile looks simpler and lots of repeated lines
could definitely be merged with globbing or .for loops in make,
which would result in a perhaps shortish, but still simple
and boring do-install.

The Makefile is taken from 8.11.0 version distfile and I'm not that well 

versed with loops in Makefile to update it. Same as in openvoxdb, I'm going

to check, if they might be open to re-add the Makefile, as it was quite convenient.

 

Then we don't need gmake and stuff like 'rundir = /var/run'
and 'rm -rf ${PREFIX}/var' would cancel themselves out.

I added the ?= to the Makefile, to be able to use the FAKE_FLAGS to 

override what is in the file. No USE_GMAKE needed.

 

thanks,

Sebastian