[Bug 198073] [new port] databases/influxdb

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=198073

Dave Cottlehuber <dch at skunkwerks.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dch at skunkwerks.at

--- Comment #3 from Dave Cottlehuber <dch at skunkwerks.at> ---
Thanks for the huge amount of work in this port, the rewriting between 0.8 to
0.9 is ... extensive.


I tidied up the port structure a bit, see
https://github.com/skunkwerks/freebsd-ports/commit/4c9fa11 in particular:

- trying to get the daemon name (influxd) and the port name (influxdb) lined up
again.
- the shas in distinfo didn't match what the scripts expect, so those are fixed
too. I think this is go version drift in action.
- I added a dedicated influxdb user account for cleaner security
- used /usr/sbin/daemon to manage influxd itself. While influxdb *may* provide
this functionality now (I didn't look closely enough), daemon is more trusted
code than a relatively new go service.

Unfortunately I've not gotten this all working completely yet, but it seems
subjectively cleaner to me this way. I hope its useful. portlint & initial
building on 10.1 amd64.

```
❯❯❯ sudo make deinstall clean fetch makesum stage stage-qa check-orphans
check-plist package install

===>  Deinstalling for influxd
===>   Deinstalling influxd-0.9.0.rc5
Updating database digests format: 100%
Checking integrity... done (0 conflicting)
Deinstallation has been requested for the following 1 packages (of 0 packages
in the universe):

Installed packages to be REMOVED:
        influxd-0.9.0.rc5

The operation will free 20 MiB.
[1/1] Deinstalling influxd-0.9.0.rc5...
[1/1] Deleting files for influxd-0.9.0.rc5: 100%
===>  Cleaning for influxd-0.9.0.rc5
===>  License MIT accepted by the user
===>   influxd-0.9.0.rc5 depends on file: /usr/local/sbin/pkg - found
===> Fetching all distfiles required by influxd-0.9.0.rc5 for building
===>  License MIT accepted by the user
===>   influxd-0.9.0.rc5 depends on file: /usr/local/sbin/pkg - found
===> Fetching all distfiles required by influxd-0.9.0.rc5 for building
===>  License MIT accepted by the user
===>   influxd-0.9.0.rc5 depends on file: /usr/local/sbin/pkg - found
===> Fetching all distfiles required by influxd-0.9.0.rc5 for building
===>  Extracting for influxd-0.9.0.rc5
=> SHA256 Checksum OK for influxd-0.9.0.rc5.tar.gz.
=> SHA256 Checksum OK for BurntSushi-toml-443a628.tar.gz.
=> SHA256 Checksum OK for bmizerany-pat-b8a3500.tar.gz.
=> SHA256 Checksum OK for boltdb-bolt-fba9b58.tar.gz.
=> SHA256 Checksum OK for kimor79-gollectd-1d0fc88.tar.gz.
=> SHA256 Checksum OK for peterh-liner-29f6a64.tar.gz.
=> SHA256 Checksum OK for rakyll-statik-4a16c83.tar.gz.
=> SHA256 Checksum OK for c3294304d93f48a37d3bed1d382882a9c2989f99.tar.gz.
=> SHA256 Checksum OK for 1351f936d976c60a0a48d728281922cf63eafb8d.tar.gz.
=> SHA256 Checksum OK for 35bc42037350f0078e3c974c6ea690f1926603ab.tar.gz.
===>  Patching for influxd-0.9.0.rc5
===>   influxd-0.9.0.rc5 depends on package: go>=1.4.0 - found
===>   influxd-0.9.0.rc5 depends on executable: hg - found
===>   influxd-0.9.0.rc5 depends on executable: git - found
===>   influxd-0.9.0.rc5 depends on file: /usr/local/bin/go - found
===>  Configuring for influxd-0.9.0.rc5
===>  Building for influxd-0.9.0.rc5
===>  Staging for influxd-0.9.0.rc5
===>   Generating temporary packing list
/bin/mkdir -p /usr/ports/skunkwerks/influxdb/work/stage/usr/local/etc/influxd 
/usr/ports/skunkwerks/influxdb/work/stage/var/db/influxd/ 
/usr/ports/skunkwerks/influxdb/work/stage/var/run/influxd/ 
/usr/ports/skunkwerks/influxdb/work/stage/var/log/influxd/
install   -o root -g wheel -m 555
/usr/ports/skunkwerks/influxdb/work/bin/influxd
/usr/ports/skunkwerks/influxdb/work/stage/usr/local/bin/influxd
install   -o root -g wheel -m 555
/usr/ports/skunkwerks/influxdb/work/bin/influx
/usr/ports/skunkwerks/influxdb/work/stage/usr/local/bin/influx
install  -o root -g wheel -m 0644 
/usr/ports/skunkwerks/influxdb/work/src/github.com/influxdb/influxdb/etc/config.sample.toml

/usr/ports/skunkwerks/influxdb/work/stage/usr/local/etc/influxd/influxd.conf.sample
====> Compressing man pages (compress-man)
===> Staging rc.d startup script(s)
====> Running Q/A tests (stage-qa)
====> Running Q/A tests (stage-qa)
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
===> No pkg-plist issues found (check-plist)
===>  Building package for influxd-0.9.0.rc5
===>  Installing for influxd-0.9.0.rc5
===>  Checking if influxd already installed
===>   Registering installation for influxd-0.9.0.rc5
Installing influxd-0.9.0.rc5...

❯❯❯portlint

WARN: Makefile: possible use of absolute pathname "/var/db/${PORTNAME}/".
WARN: Makefile: possible use of absolute pathname "/var/log/${PORTNAME}...".
WARN: Makefile: possible use of absolute pathname "/var/run/${PORTNAME}...".
FATAL: Makefile: PORTVERSION looks illegal. You should modify "0.9.0.rc5".
1 fatal error and 3 warnings found.
```

-- 
You are receiving this mail because:
You are the assignee for the bug.