Skip to content

Commit

Permalink
doc/go1.18: document FreeBSD 11.x will no longer be supported
Browse files Browse the repository at this point in the history
Mention future versions will require the COMPAT_FREEBSD12 kernel option
set in the kernel.

For #47694.

Change-Id: Ia94c4f9dbb38c68025d3c1d12dd2e241a5480a6b
Reviewed-on: https://go-review.googlesource.com/c/go/+/354971
Reviewed-by: Ian Lance Taylor <[email protected]>
Reviewed-by: Tobias Klauser <[email protected]>
  • Loading branch information
paulzhol authored and dmitshur committed Oct 11, 2021
1 parent 702e337 commit f2141a7
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions doc/go1.18.html
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,11 @@ <h2 id="language">Changes to the language</h2>

<h2 id="ports">Ports</h2>

<p>
TODO: complete this section, or delete if not needed
<p id="freebsd">
Go 1.18 is the last release that is supported on FreeBSD 11.x, which has

This comment has been minimized.

Copy link
@dmgk

dmgk Jun 7, 2022

Member

There's still FreeBSD 11 compatibility code introduced by e.g. dc6eb20 and golang/sys@5535b4e which still requires COMPAT_FREEBSD11 in the kernel. Saying that FreeBSD 12.2 is an oldest supported version and at the same time requiring a kernel built with COMPAT_FREEBSD11 sounds confusing to me.

Are there plans to remove FreeBSD 11 compat code from Go and x/sys/unix before go1.19 release?

This comment has been minimized.

Copy link
@ianlancetaylor

ianlancetaylor Jun 7, 2022

Contributor

@dmgk Note that we use Gerrit for code review. GitHub is a mirror. Very few people see comments on GitHub commits. If you want to comment on this change, please comment on https://go.dev/cl/354971. Thanks.

That said, this concern would probably be best raised on the golang-dev group or as an issue. Thanks.

already reached end-of-life. Go 1.19 will require FreeBSD 12.2+ or FreeBSD
13.0+.
FreeBSD 13.0+ will require a kernel with the COMPAT_FREEBSD12 option set (this is the default).
</p>

<h2 id="tools">Tools</h2>
Expand Down

0 comments on commit f2141a7

Please sign in to comment.