databases/influxdb2-cli: New port
ClosedPublic
Actions

Authored by driesm on Dec 20 2021, 9:57 AM.

Details

Reviewers

philip
0mp

Summary

databases/influxdb2-cli: New port

CLI for managing resources in InfluxDB v2.

 Approved by:		tbd (mentor)
 Diferential Revision:	https://reviews.freebsd.org/D33582

This gets working ports out of my WIP in the tree, so users can start using it :) and my local diff gets smaller.

Test Plan

Poudriere testport looks good and linters are happy.

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 43500
Build 40388: arc lint + arc unit

Event Timeline

driesm created this revision.Dec 20 2021, 9:57 AM

Herald added a subscriber: mat. · View Herald TranscriptDec 20 2021, 9:57 AM

driesm requested review of this revision.Dec 20 2021, 9:57 AM

Harbormaster completed remote builds in B43494: Diff 100333.Dec 20 2021, 9:57 AM

driesm edited the summary of this revision. (Show Details)Dec 20 2021, 9:57 AM

driesm edited the summary of this revision. (Show Details)

driesm added reviewers: philip, 0mp.

driesm edited the summary of this revision. (Show Details)

0mp added inline comments.Dec 20 2021, 10:46 AM

databases/influxdb2-cli/Makefile
14	I'm not sure if this is correct. I don't see any other examples of `DEPENDS_USES` in the tree. Is it a typo?

driesm planned changes to this revision.Dec 20 2021, 10:59 AM

driesm added inline comments.

databases/influxdb2-cli/Makefile
14	Indeed!

Use TEST_DEPENDS

Harbormaster completed remote builds in B43500: Diff 100346.Dec 20 2021, 11:42 AM

LGTM. If it builds, it seems ready to go.

Just a note: TEST_DEPENDS= ...gmake... looks fishy, but I guess that USES=gmake does not cover this use case.

This revision is now accepted and ready to land.Jan 25 2022, 12:30 AM

Yea indeed, I just checked uses/go.mk, and it has a test target on its own. I'm gonna commit this without the modified test target. This way I can also drop gmake as dependency.

driesm mentioned this in R11:ec44ccc6b369: databases/influxdb2-cli: New port.Jan 26 2022, 5:34 PM

driesm closed this revision.Jan 26 2022, 5:41 PM