Created attachment 157761 [details] Bump net-mgmt/lldpd to 0.7.15, silence build a little Hi there This small patch should bump net-mgmt/lldpd to its latest release 0.7.15. The added doxygen parameter just reduces the noise during build process (I don't think there is much value in having doxygen doc built for normal use) Looking forward to someone pushing the changes for me Thanks in advance, Mathieu
> The added doxygen parameter just reduces the noise during build process (I don't think there is much value in having doxygen doc built for normal use) A port should not use any dependencies not listed in *_DEPENDS, so I suggest to either leave the flag there or add a DOXYGEN option with proper dependency handling.
Created attachment 157821 [details] Only bump net-mgmt/lldpd to 0.7.15
Adding Doxygen (as of now) adds a hole slew of additional dependencies for properly build all doxygen documentation. Therefore I'd prefer to only get the new version first and later look at properly adding a doxygen option. Thanks for the feedback, Mathieu
Uh, I'm terribly sorry - for some reason I've read the original patch conversely and though that you were removing --disable-doxygen-doc. Actually adding it is completely correct and is exactly what I've advocated for. Adding an option would be nice, but I too think that it's doesn't have much value. Committing original patch, thanks!
A commit references this bug: Author: amdmi3 Date: Wed Jun 17 16:52:21 UTC 2015 New revision: 389941 URL: https://svnweb.freebsd.org/changeset/ports/389941 Log: - Update to 0.7.15 - Remove hidden dependency on doxygen PR: 200876 Submitted by: freebsd@simweb.ch (maintainer) Changes: head/net-mgmt/lldpd/Makefile head/net-mgmt/lldpd/distinfo head/net-mgmt/lldpd/pkg-plist