Remove support for FreeBSD 11.4

[rene]

Summary

*: mark FreeBSD 11.4 as unsupported
"git grep 11.4"
Clean up OSVER*
Readd net/intel-em-kmod

Test Plan

make index

[jbeich]

multimedia/vlc is OK on behalf of multimedia. What is removed was originally added by me (see dda7c32cfd52d).

My ports look OK but can you drop more stuff? See attached example.

• devel/aml: drop files/patch-freebsd11
• emulators/rpcs3: drop files/patch-rpcs3_Emu_system__config.h
• emulators/yuzu: drop files/patch-freebsd11
• graphics/mesa-devel: drop files/patch-freebsd11
• multimedia/aom: drop FreeBSD < 12 conditionals in files/patch-aom_\_ports_arm__cpudetect.c
• multimedia/aom: drop FreeBSD < 12 conditionals in files/patch-aom_\_ports_ppc__cpudetect.c
• multimedia/dav1d: drop files/patch-neon-freebsd11
• multimedia/libvpx: drop FreeBSD < 12 conditionals in files/patch-vpx_\_ports_arm__cpudetect.c
• multimedia/libvpx: drop FreeBSD < 12 conditionals in files/patch-vpx_\_ports_ppc__cpudetect.c
• net/waypipe: drop files/patch-neon-freebsd11
• sysutils/keyd: drop -D_WITH_GETLINE
• x11-toolkits/wlroots: drop FreeBSD 11.* notes in pkg-message
• x11-toolkits/wlroots: drop files/patch-include_wlr_types_wlr__seat.h
• x11-wm/gamescope: drop -D_WITH_DPRINTF
• x11-wm/picom: drop files/patch-src_log.c
• x11/taiga: drop -D_WITH_GETLINE
• x11/waybar: drop -D_WITH_GETLINE
• (for gecko@) www/firefox: drop files/patch-freebsd11

D32008-followup.patch22 KBDownload

[ashish]

net-im/ejabberd changes look good to me. Thank you for wrangling this.

[tremere_cainites.net]

OK for dns/powerdns and dns/powerdns-recursor

[jbeich]

All other instances of _WITH_GETLINE and _WITH_DPRINTF can be dropped as well. See attached example.

$ rg -l -e _WITH_GETLINE -e _WITH_DPRINTF
audio/surge-synthesizer-lv2/files/patch-vstgui.surge_vstgui_lib_platform_linux_x11fileselector.cpp
audio/vst3sdk/files/patch-vstgui4_vstgui_lib_platform_linux_x11fileselector.cpp
chinese/fcitx-libpinyin/files/patch-src__CMakeLists.txt
chinese/ibus-libpinyin/files/patch-src_PYLibPinyin.cc
chinese/libpinyin/Makefile
devel/android-tools-simpleperf/files/Makefile
devel/bear/files/extra-patch-source_intercept_source_report_libexec_Logger.cc
devel/evemu/Makefile
mail/thunderbird/files/patch-cubeb-oss
misc/lxi-tools/Makefile
multimedia/obs-studio/Makefile
multimedia/subtitleripper/Makefile
security/libfido2/Makefile
sysutils/cfengine-devel/files/patch-cf-execd_cf-execd-runagent.c
sysutils/cfengine318/files/patch-cf-execd_cf-execd-runagent.c
sysutils/hexpeek/Makefile
sysutils/netevent/Makefile
sysutils/sleuthkit/Makefile
textproc/dwdiff/Makefile
www/firefox-esr/files/patch-cubeb-oss
x11-drivers/xf86-video-intel/Makefile

D32008-another-followup.patch13 KBDownload

devel/bear/Makefile

37

Can you drop the actual file as well? files/extra-patch-source_intercept_source_report_libexec_Logger.cc still exists.

[adridg]

For kde@ ports (net/qt5-network, devel/qca) approved.
For adridg@ ports (net-im/mtxclient) approved.

Feel free to remove the qca-legacy port as well.

Mk/Uses/qca.mk

25

This can be more thorough -- the legacy port devel/qca-legacy can be removed, as well as the variable naming it, here. But that is cleanup kde@ can do afterwards.

[jbeich]

All -fuse-ld=lld instances can be dropped. FreeBSD < 13 powerpc* has Clang/LLD disabled by default and should be using GNU bfd from devel/binutils via USES=compiler:c11 or similar. See attached example.

D32008.bfd.diff24 KBDownload

[royger]

For Xen bits.

[jhb]

security/isal-kmod is ok.

One suggestion: it is really hard to review a large change like this as it took me quite a while to figure out which part was relevant for me to review. I would consider breaking this up into multiple reviews/commits in the future if possible. Even one per category might be a decent compromise. Another option might be to split it along functional lines (e.g. one commit to remove all the 'IGNORE_FreeBSD_11' and a separate commit to remove OSVERSION checks, etc.). Some of the followup commits to remove classes of things might also be good candidates for splitting into separate commits/reviews as well.

[brooks]

llvm bits look good

[Alexander88207_protonmail.com]

Looks good for emulators/i386-wine and emulators/i386-wine-devel, thanks!

[ehaupt]

Changes to:

mail/isync
net/httping

LGTM

[brnrd]

LGTM for my ports

[riggs]

LGTM for multimedia/mkvtoolnix, multimedia/vlc

[woodsb02]

Given 11.4 EOL is 30th September, is the plan to remove support for 11.4 from ports before the 2021Q4 quarterly is branched?

[delphij]

All changes to my port looked fine.

(IMO early 12.x prior to 12-STABLE branch point 1200500 should be removed too).

[mandree]

graphics/rawtherapee LGTM

[mandree]

Comments to portmgr, skimming through more than just my own patches,

• the diversity of OpenSSL 1.0.2 workaround and refusals weasling out was astonishing. While the 1.1.1 -> 3.0.0 transition isn't as hard as 1.0.2 -> 1.1.1 in my experience, we may want to offer standard workarounds or requirements in the framework. Some means to tell the framework "this port needs OpenSSL 3.0.0".
• to some lesser extent, we may have had FreeBSD 11 exclusions due to insufficient compiler support, and we may want the ports framework to cater for compiler upgrades in an easy manner, too

Comment to @fluffy - opensmtpd-devel (2016) looks much older than opensmtpd 6.8.0 (p2 released in 2020), do we need to keep it or should it go before 2021Q4? Or at least marked DEPRECATED and expiring? It seems opensmtpd is a good migration patch.

[rene]

• security/ipsec-tools: Re-add support for older FreeBSD versions
• Address change requests.

Mk/Uses/qca.mk

25

Yes, initially I was more enthusiastic in cleaning this up, but decided leaving the variables in would be more flexible for future updates.

lang/perl5-devel/Makefile

201

Oops, that one fell out.

misc/mc/Makefile

94 ↗

(On Diff #95305)

Fair enough, the other option would be to remove the OPSYS check entirely (or do we support DragonFlyBSD here) ?

[rene]

• More cleanup from jbeich@

[kai]

LGTM for devel/py-keystone-engine and lang/python310.

[dbaio]

doceng: misc/freebsd-doc-en

[sunpoet]

Thanks, @rene.
LGTM for databases/rocksdb and ftp/bsdftpd-ssl.
As perl@, LGTM for security/p5-Crypt-OpenSSL-CA and security/p5-OpenCA-OpenSSL (though the SSL_DEFAULT check needs to be improved by adding at least openssl).
As ruby@, LGTM for devel/rubygem-rice.

[dch]

LGTM and thanks for the 🚀 prep work to get this organised!

• ✅ databases/couchdb3
• ✅ databases/foundationdb
• ✅ databases/foundationdb-devel
• ✅ lang/zig
• ✅ lang/zig-devel
• ✅ www/h2o
• ✅ www/h2o-devel
• ✅ sysutils/kvmclock-kmod

[dch]

with erlang hat on DELETE devel/erlang-exmpp entirely

• ⏱ last upstream patches more than 10 years ago
• 🔴 port has no dependencies
• doesn't support modern erlang anymore ☣

I don't believe anybody is using this port at all. If not, we urgently need to save them from themselves.

[jamie_catflap.org]

LGTM

[freqlabs]

openzfs* changes look reasonable to me.

[sbz]

LGTM for security/dsniff and security/openssl, thanks.

[ronald_klop.ws]

mongodb changes are fine by me

[danfe]

misc/mc/Makefile

94 ↗

(On Diff #95305)

Fair enough, the other option would be to remove the OPSYS check entirely (or do we support DragonFlyBSD here)?

Actually, we should be really checking if user's diff is GNU or BSD one rather then checking OSVERSION (and/or OPSYS). I've replaced the check just now, so it's no longer an issue.

[fernape]

LGTM.

Thanks!

[rene]

• security/ipsec-tools: Re-add support for older FreeBSD versions
• Address change requests.
• More cleanup from jbeich@
• More cleanup from jbeich@
• Chase more *_FreeBSD_11
• Chase more OSVER*

[fluffy]

In D32008#722311, @mandree wrote:

Comment to @fluffy - opensmtpd-devel (2016) looks much older than opensmtpd 6.8.0 (p2 released in 2020), do we need to keep it or should it go before 2021Q4? Or at least marked DEPRECATED and expiring? It seems opensmtpd is a good migration patch.

Yeah, I've dropped it in favor of mail/opensmtpd today.

[rene]

• security/ipsec-tools: Re-add support for older FreeBSD versions
• Address change requests.
• More cleanup from jbeich@
• More cleanup from jbeich@
• Chase more *_FreeBSD_11
• Chase more OSVER*
• Chase WITH_GETLINE removal, from jbeich@
• Chase _WITH_DPRINTF removal, from jbeich@
• Remove obsolete devel/qca-legacy, from adridg@
• Remove now obsolete GNU bfd workarounds, from jbeich@
• Remove obsolete devel/erlang-exmpp, from dch@

[rene]

• security/ipsec-tools: Re-add support for older FreeBSD versions
• Address change requests.
• More cleanup from jbeich@
• More cleanup from jbeich@
• Chase more *_FreeBSD_11
• Chase more OSVER*
• Chase WITH_GETLINE removal, from jbeich@
• Chase _WITH_DPRINTF removal, from jbeich@
• Remove obsolete devel/qca-legacy, from adridg@
• Remove now obsolete GNU bfd workarounds, from jbeich@
• Remove obsolete devel/erlang-exmpp, from dch@
• Chase a few more OSVERSION / OSREL cleanups
• More cleanups

[rene]

• security/ipsec-tools: Re-add support for older FreeBSD versions
• Address change requests.
• More cleanup from jbeich@
• More cleanup from jbeich@
• Chase more *_FreeBSD_11
• Chase more OSVER*
• Chase WITH_GETLINE removal, from jbeich@
• Chase _WITH_DPRINTF removal, from jbeich@
• Remove obsolete devel/qca-legacy, from adridg@
• Remove now obsolete GNU bfd workarounds, from jbeich@
• Remove obsolete devel/erlang-exmpp, from dch@
• Chase a few more OSVERSION / OSREL cleanups
• More cleanups
• Some cleanups for security/p5-*OpenSSL* from sunpoet@
• emulators/mesen: simplify after 11.X and 12.0 EOL