Bug 200876 - net-mgmt/lldpd: Bump to 0.7.15
Summary: net-mgmt/lldpd: Bump to 0.7.15
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Dmitry Marakasov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-06-15 15:05 UTC by Mathieu Simon
Modified: 2015-06-17 16:52 UTC (History)
2 users (show)

See Also:


Attachments
Bump net-mgmt/lldpd to 0.7.15, silence build a little (1.43 KB, patch)
2015-06-15 15:05 UTC, Mathieu Simon
no flags Details | Diff
Only bump net-mgmt/lldpd to 0.7.15 (1.11 KB, patch)
2015-06-17 04:38 UTC, Mathieu Simon
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mathieu Simon 2015-06-15 15:05:37 UTC
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
Comment 1 Dmitry Marakasov freebsd_committer 2015-06-15 20:29:28 UTC
> 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.
Comment 2 Mathieu Simon 2015-06-17 04:38:56 UTC
Created attachment 157821 [details]
Only bump net-mgmt/lldpd to 0.7.15
Comment 3 Mathieu Simon 2015-06-17 04:40:27 UTC
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
Comment 4 Dmitry Marakasov freebsd_committer 2015-06-17 16:52:24 UTC
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!
Comment 5 commit-hook freebsd_committer 2015-06-17 16:52:31 UTC
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