Summary: | net-mgmt/collectd5: Update to 5.5.1 | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Krzysztof <ports> | ||||||||||||||
Component: | Individual Port(s) | Assignee: | Guido Falsi <madpilot> | ||||||||||||||
Status: | Closed FIXED | ||||||||||||||||
Severity: | Affects Many People | CC: | madpilot | ||||||||||||||
Priority: | --- | Keywords: | patch, patch-ready | ||||||||||||||
Version: | Latest | ||||||||||||||||
Hardware: | Any | ||||||||||||||||
OS: | Any | ||||||||||||||||
Attachments: |
|
Description
Krzysztof
2016-01-27 03:08:17 UTC
maintainer-feedback not needed unless in response to ? I'm not sure if patch for src/zfs_arc.c is correctly made (specialy for version number). I hope that someone will check it and correct me. Thanks a lot. I'm testing the port, I'll need a little time for proper testing. In the while I performed a little research about the "stolen" ZFS stat via subversion: It was removed from head in r286762, from August 14th 2015. This commit was merged to stable in r288580, on October 3rd 2015. This means that the kstat.zfs.misc.arcstats.stolen sysctl IS available on 10.2-RELEASE machines and will be for all future patches to that release, but not on 10-STABLE machines after October 3rd 2015. I'd propose to use the 1002501 __FreeBSD_version value as a discriminator in the patch, which was created on October 8th, really near to October 3rd. This will allow people on 10.2-RELEASE to use that value. Apart from this the ZFS patch looks fine to me to the best of my knowledge. Do you agree with my analysis? I think I should :-))) Telling seriosuly I'm not so familiar with __FreeBSD_version values. That's why I asked for check my patch. The simplest way was to remove kstat.zfs.misc.arcstats.stolen completely, but I thaought that some people still could be interested in collecting this values. So I agree with you. Created attachment 166361 [details]
Failure log
Hi,
I made a test with all options enabled and the port fails to compile, for some reason the configure script fails to detect certain dependencies.
Can you test and perhaps provide a patch?
Thanks.
OK, I've checked this error log. Yes, that's my fault - I did not checked these plugins before correctly. Now modbus plugin is corrected. But there is problem with write_mongo plugin. It depends on mongo-c-driver library. And mongo-c-driver depends on libbson. Both of them are out of date and I have to update them first. And then try to build mongodb plugin again. Current log is here: https://rtm.bsdserwis.com/poudriere/data/a1amd64-testing/2016-02-01_00h36m12s/logs/errors/collectd5-5.5.1.log Now as you can check modbus support is correctly impolemented: https://rtm.bsdserwis.com/poudriere/data/a1amd64-testing/2016-02-01_00h54m38s/logs/collectd5-5.5.1.log So I've changed patch for port. I need some time to update mongo plugin dependencies. Created attachment 166370 [details]
collectd-5.5.1.patch - update 2
Comment on attachment 166171 [details]
collectd-5.5.1.patch - update
It's obsoleted. A new one is added
Created attachment 166385 [details] patch collectd 5.5.1 - update After many tries I had to disable mongodb plugin. According to https://github.com/collectd/collectd/issues/492 there is a problem with this plugin for long time. Maybe it will be resolved soon, so MONGODB will be enabled again. I've forgotten add log from poudriere: https://rtm.bsdserwis.com/poudriere/data/a1amd64-testing/2016-02-01_10h03m35s/logs/collectd5-5.5.1.log (In reply to Krzysztof from comment #9) Hi, the new patch fails to apply cleanly. Looks like it was generated against an already modified ports tree. Could you regenerate the patch against a fresh port checkout? Thanks. Created attachment 166411 [details]
collectd-5.5.1.patch - update 3
Yes, previous patch was made on modified net-mgmt/collectd5 files. Now this patch is correct - I tested it on fresh ports tree.
I'm going to test this patch but please note that there a re still a few problems with it. First of all this patch removes pkg-descr, and the startup scripts in files. I recovered those. Also you did not update the __FreeBSD_version on the value we agreed upon. While I can(and did) fix these, please try to generate correct patches, this reduces duplicate work and helps avoiding making mistakes and committing unwanted parts to the tree. BTW I also added a "(BROKEN)" text to the mongodb option description, so it is clear to users and slightly reworded the BROKEN message to: fails to configure with write_mongodb option which is clearer in the context the ports infrastructure is going to use the message. Created attachment 166430 [details]
patch collectd 5.5.1 - update 4
OK, sorry for my mistakes. This patch is corrected according to our conversation and nothing is missing.
Once again sorry for my mistakes.
Committed! Thanks. A commit references this bug: Author: madpilot Date: Tue Feb 2 23:08:46 UTC 2016 New revision: 407901 URL: https://svnweb.freebsd.org/changeset/ports/407901 Log: - Update to 5.5.1 - Mark MONGODB option as BROKEN PR: 206660 Submitted by: ports at bsdserwis.com (maintainer) Changes: head/net-mgmt/collectd5/Makefile head/net-mgmt/collectd5/distinfo head/net-mgmt/collectd5/files/patch-Makefile.am head/net-mgmt/collectd5/files/patch-configure.ac head/net-mgmt/collectd5/files/patch-src__Makefile.am head/net-mgmt/collectd5/files/patch-src__Makefile.in head/net-mgmt/collectd5/files/patch-src__daemon__Makefile.am head/net-mgmt/collectd5/files/patch-src__daemon__collectd.h head/net-mgmt/collectd5/files/patch-src__dns.c head/net-mgmt/collectd5/files/patch-src__modbus.c head/net-mgmt/collectd5/files/patch-src__varnish.c head/net-mgmt/collectd5/files/patch-src_config.h.in head/net-mgmt/collectd5/files/patch-src_disk.c head/net-mgmt/collectd5/files/patch-src_write__mongodb.c head/net-mgmt/collectd5/files/patch-src_zfs__arc.c head/net-mgmt/collectd5/files/patch-version-gen.sh head/net-mgmt/collectd5/pkg-plist Hi, I'm reopening this bug because I have been reported by danfe that there is a problem with my commit. The patch in files/patch-src__Makefile.in is really uneeded, since the port has USES=autoreconf, so Makefile.in files are being regenerated from the Makefile.am ones. All the work is being done already in the files/patch-src__Makefile.am patch. I'm testing the port without files/patch-src__Makefile.in and if everything works I'll fix the port. Such followup commits don't really need you approval, but I'm anyway reopening this bug and informing you. I also forgot to mention that in the commit I had to put back the files/patch-Makefile.am patch, otherwise the port was creating stage orphans. Thanks. A commit references this bug: Author: madpilot Date: Wed Feb 3 15:54:30 UTC 2016 New revision: 407956 URL: https://svnweb.freebsd.org/changeset/ports/407956 Log: Remove uneeded patch file added by mistake. PR: 206660 Reported by: danfe Changes: head/net-mgmt/collectd5/files/patch-src__Makefile.in Unneeded patch file removed. |