Bug 236082 - net-mgmt/nfdump: Add InfluxDB option
Summary: net-mgmt/nfdump: Add InfluxDB option
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Kurt Jaeger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-02-27 16:42 UTC by Jose Luis Duran
Modified: 2019-02-27 17:35 UTC (History)
0 users

See Also:
pi: maintainer-feedback+


Attachments
Add InfluxDB option (1.47 KB, patch)
2019-02-27 16:42 UTC, Jose Luis Duran
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jose Luis Duran 2019-02-27 16:42:59 UTC
Created attachment 202412 [details]
Add InfluxDB option

- Add InfluxDB option
- portlint

NOTE: There is a warning in `stage-qa` that lib/libnfdump-1.6.17.so is not stripped. Should I change `INSTALL_DATA` for `INSTALL_LIB`?
Comment 1 Kurt Jaeger freebsd_committer 2019-02-27 17:22:32 UTC
thanks for the patch, committed!
Comment 2 commit-hook freebsd_committer 2019-02-27 17:23:19 UTC
A commit references this bug:

Author: pi
Date: Wed Feb 27 17:22:29 UTC 2019
New revision: 494074
URL: https://svnweb.freebsd.org/changeset/ports/494074

Log:
  net-mgmt/nfdump: Add InfluxDB option, default OFF

  - and portlint
  - INSTALL_LIB to strip libnfdump.so

  PR:		236082
  Submitted by:	Jose Luis Duran <jlduran@gmail.com>

Changes:
  head/net-mgmt/nfdump/Makefile
Comment 3 Jose Luis Duran 2019-02-27 17:29:58 UTC
(In reply to Jose Luis Duran from comment #0)

Thank you!

But, I did NOT add the stripping part to the library. I was just asking if I should. I'll attach another patch, but don't know if I should bump PORTREVISION again?
Comment 4 Jose Luis Duran 2019-02-27 17:35:19 UTC
(In reply to Jose Luis Duran from comment #3)

Never mind! Thank you!