Fwd: NEW databases/openvoxdb

Index: user.list
===================================================================
RCS file: /cvs/ports/infrastructure/db/user.list,v
diff -u -r1.480 user.list
--- user.list 25 Jan 2026 17:24:41 -0000 1.480
+++ user.list 1 Feb 2026 10:42:41 -0000
@@ -227,7 +227,7 @@
716 _c-icap _c-icap www/c-icap/c-icap
717 _uptimed _uptimed sysutils/uptimed
718 _stuntman _stuntman telephony/stuntman
-719 _puppetdb _puppetdb databases/puppetdb
+719 _puppetdb _puppetdb databases/{puppetdb,openvoxdb}
#720 _lldpd _lldpd net/lldpd
721 _dkimproxy _dkimproxy mail/dkimproxy
722 _salt _salt sysutils/salt

Forgot to include ports@ in my reply.

---------- Forwarded message ---------

Hi Klemens,

I think I addressed all your concerns, see below.

Updated tarball attached.

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

28.01.2026 16:55, Sebastian Reitenbach пишет:
> Hi,
>
> cat pkg/DESCR:
> OpenVoxDB is a fork of Open Source PuppetDB.
>
> OpenVoxDB is the fast, scalable, and reliable data warehouse for OpenVox.
> It caches data generated by OpenVox, and gives you advanced features at
> awesome speed with a powerful API.
>
> Similarly to OpenVox server, it will eventually replace PuppetDB. To eventually support multiple major versions, same structure used as databases/puppetdb.

Makes sense.

I don't think '@pkgpath databases/puppetdb/7' is needed.

removed 

 

infrastructure/db/user.list should mention openvoxdb as well.

see attached user.list.diff file

 

patches/ harcodes /var while the .rc file uses LOCALSTATEDIR;
both is fine to me really, but it should be consistent.

I went the /var route.

 

patches/ has files with 'env bash' shebangs and do-configure
sed-patches other files to use an absoloute path.  Both work,
but I'd prefer we stick to one way of fixing/using bash.

I chose the env bash version.

 

Strictly speaking, I think, MAKE_FLAGS should be FAKE_FLAGS
since only do-install uses them and NO_BUILD=Yes.

Renamed

 

Overall, it'd be nice if openvoxdb and openvox-server Makefiles
would match;  they share a lot, but have churn wrt. V/VERSION,
spaces/no spaces before =, etc. which makes reading harder than
it should be.

Should be better now.

 

>
> Same as with OpenVox server, between 8.11 and 8.12, they dropped the Makefile with the install target. Therefore using same approach here, and added the old Makefile to the files directory.

I lack the details/background here, but the approach seems fine
except for

- 'rubylibdir ?= $(shell ruby ...)':
  - needs USE_GMAKE=Yes (only openvox-server sets it)
  - should probably use MODRUBY_* stuff to get the binary name
  or not use a shell command at all

- you default 'confidir ?= /etc' and others, bu then also pass
  them in the ports Makefile;  we control both, so wouldn' it
  be simpler to use SYSCONFDIR in files/Makefile right away?

To the TWO points above:

I don't know yet, why they removed the Makefile, I wanted to check with upstream.

Therefore I intended to keep the way it was working with puppetdb.

Actually, what is in the file, i.e. the 'rubylibdir ?= $(shell ruby ...)' is not executed,

because it's overridden by the FAKE_FLAGS. therefore no USE_GMAKE anymore,

must have been an old leftover....

 

- does it really need to have all the targets we don't use?

removed the superfluous targets.

 

>
> Any concerns, test reports or any other feedback welcome.

I don't run an openvox server/db setup (yet).

>
> Cheers,
> Sebastian