Bug 233008

Summary: net/tcpdump: #includes deprecated sys/capability.h header
Product: Ports & Packages Reporter: Ed Maste <emaste>
Component: Individual Port(s)Assignee: Renato Botelho <garga>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (garga)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 228878    

Description Ed Maste freebsd_committer freebsd_triage 2018-11-05 23:49:58 UTC
sys/capability.h will be removed before FreeBSD 13. The (now) correct header to use on all supported (and some unsupported) versions of FreeBSD is sys/capsicum.h. There's no need for autoconf checks etc. to check for both (in the ports tree) but upstream may want to if there's a desire to build on older releases.
Comment 1 Renato Botelho freebsd_committer freebsd_triage 2018-11-06 10:50:49 UTC
I'll open a bug in upstream and then add a patch on ports
Comment 2 commit-hook freebsd_committer freebsd_triage 2018-11-06 17:18:09 UTC
A commit references this bug:

Author: garga
Date: Tue Nov  6 17:17:33 UTC 2018
New revision: 484310
URL: https://svnweb.freebsd.org/changeset/ports/484310

Log:
  - Do not include deprecated header sys/capability.h [1]
  - Remove unnecessary include of bsd.port.pre.mk
  - Add LICENSE_FILE
  - Fix USES order
  - Bump PORTREVISION

  PR:		233008 [1]
  Reported by:	emaste@ [1]
  Sponsored by:	Rubicon Communications, LLC (Netgate)

Changes:
  head/net/tcpdump/Makefile
Comment 3 Renato Botelho freebsd_committer freebsd_triage 2018-11-06 17:20:55 UTC
PR submitted to upstream at https://github.com/the-tcpdump-group/tcpdump/pull/715