Bug 202487
| Summary: | [PATCH] net-mgmt/collectd5: Add write_kafka support | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | xoan <xoan333> | ||||||||||
| Component: | Individual Port(s) | Assignee: | Jason Unovitch <junovitch> | ||||||||||
| Status: | Closed FIXED | ||||||||||||
| Severity: | Affects Some People | CC: | junovitch, ports | ||||||||||
| Priority: | --- | Keywords: | patch | ||||||||||
| Version: | Latest | Flags: | ports:
maintainer-feedback+
|
||||||||||
| Hardware: | Any | ||||||||||||
| OS: | Any | ||||||||||||
| Attachments: |
|
||||||||||||
Created attachment 160134 [details]
collectd5.patch to add librdkafka
I've made some small corrections to your patch. So this one is correct.
Comment on attachment 160107 [details]
collectd5.patch
Please make your patch as obsolete - I can not do it.
Created attachment 160461 [details]
collectd5_librdkafka.patch
Add corrected patch - previous one does not apply cleanly
I have tested build of collectd5 with librdkafka. You can see log at https://rtm.bsdserwis.com/poudriere/data/a1amd64-testing/2015-08-29_02h44m06s/logs/collectd5-5.5.0_2.log There were no error at all. So I hope this patch will be commited soon :-))) Take. Created attachment 160849 [details] collectd5-5.5.0_3.patch * Obsolete prior patch * Krzysztof, this is just a small cleanup on the patch. Mainly to sort the OPTIONS_GROUP_OUTPUT that we are adding, sort two missorted _DESC= values, and keep the KAFKA PORT_OPTIONS we just added alphabetically sorted between JSON and LOGSTASH. Final log for my mentor review: net-mgmt/collectd5: add support for write_kafka plugin While here, also sort OPTIONS_GROUP_OUTPUT and two DESC items PR: 202487 Submitted by: xoan333@yahoo.com Approved by: Krzysztof <ports@bsdserwis.com> (maintainer) QA wise, nothing actionable to address at the moment. Portlint still shows a false hit because of the capitalized "CGI" WARN: Makefile: [169]: IGNORE messages should begin with a lowercase letter and end without a period. Poudriere with all port options is compiling now and was successful on 9.3 amd64. Compile with default options was good on 9, 10, and 11 so I am not foreseeing any issue. Can you do one thing when you have the time? Take a look at the various revised helpers that have been added in the Porter's Handbook. This should shrink the Makefile significantly. For now, let's get the PR for this feature addition out of the way now and follow up on this in the future commit just to clean this port up. https://www.FreeBSD.org/doc/en_US.ISO8859-1/books/porters-handbook/makefile-options.html#ports-options-practical-use Thanks! A commit references this bug: Author: junovitch Date: Fri Sep 11 03:32:14 UTC 2015 New revision: 396645 URL: https://svnweb.freebsd.org/changeset/ports/396645 Log: net-mgmt/collectd5: add support for write_kafka plugin While here, also sort OPTIONS_GROUP_OUTPUT and two DESC items PR: 202487 Submitted by: xoan333@yahoo.com Approved by: Krzysztof <ports@bsdserwis.com> (maintainer), feld (mentor) Changes: head/net-mgmt/collectd5/Makefile head/net-mgmt/collectd5/pkg-plist Committed. Thank you both! Krzysztof, I'll you some details and example ports for the option helper conversions and we'll pick that up in another PR when you are able to have a look. |
Created attachment 160107 [details] collectd5.patch Add write_kafka support to collectd5 with the net/librdkafka port