Bug 233092 - lang/go: relies on deprecated sys/capability.h header for Capsicum
Summary: lang/go: relies on deprecated sys/capability.h header for Capsicum
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Julien Laffaye
URL: https://github.com/golang/sys/commit/...
Keywords:
Depends on:
Blocks: 228878 233091 233090
  Show dependency tree / graph
 
Reported: 2018-11-09 18:17 UTC by Ed Maste
Modified: 2019-03-08 17:48 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (jlaffaye)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Maste freebsd_committer 2018-11-09 18:17:57 UTC 
sys/capability.h will be removed before FreeBSD 13. The (now) correct header to use on all supported (and most 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.

go/vendor/golang.org/x/sys/unix/mkerrors.sh
82:#include <sys/capability.h>
Comment 1 Ed Maste freebsd_committer 2018-11-12 17:53:16 UTC 
Upstream pull request at https://github.com/golang/go/pull/28742

Port can probably use REINPLACE_CMD to modify the file while waiting on the change to percolate through, or just wait if it gets merged/released quickly.
Comment 2 Ed Maste freebsd_committer 2018-11-12 17:55:00 UTC 
Canonical change URL: https://go-review.googlesource.com/c/go/+/148836
Comment 3 Ed Maste freebsd_committer 2018-11-24 22:14:28 UTC 
Link to commit that addresses this.
Comment 4 w.schwarzenfeld freebsd_triage 2019-02-11 12:05:17 UTC 
ping!
Comment 5 Dmitri Goutnik 2019-03-08 17:48:32 UTC 
This was fixed by the upstream [1] and included in go1.12

[1] https://github.com/golang/go/commit/cc5fb5d93dd5880ba9832b1aa5f0914cca5cc0bd#diff-0a9f0278821c88f523571ee62e282314L83