Page MenuHomeFreeBSD

Remove support for FreeBSD < 11.2

Authored by rene on Oct 27 2018, 8:24 PM.



Remove support for FreeBSD < 11.2 after FreeBSD 10.4 support ends (2018-11-01).

All ports that work only on FreeBSD 10 have already been marked DEPRECATED and
will be removed when they expire.

Some ports are not (fully) covered in this patch:

  • ports-mgmt/poudriere[-devel] -> build ports on an old system (*)
  • lang/perl5.26 -> required by ports-mgmt/poudriere-devel
  • emulators/i386-wine*
  • any patch in files/

Tools/scripts/tindex is untouched by this patch.

(*) Mk/ does include a patch to raise the minimum supported version
to 11.2 though.

Test Plan
  • building INDEX on 13-amd64 works fine

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

Clean up some pkg-* files too.

dns/bind* and emulators/qemu* are left out for now.

Rebase to r483291 and fix devel/qtcreator

`make check-sanity' on looks fine on all changed ports.

From my POV:

  • Drop multimedia/aom/files/patch-av1_encoder_rd.h

From gecko POV:

  • Drop mail/thunderbird/files/patch-ipc_glue_MessageChannel.cpp
  • Drop www/firefox-esr/files/patch-ipc_glue_MessageChannel.cpp
  • Drop www/firefox/files/patch-ipc_glue_MessageChannel.cpp

From office POV:

  • devel/boost-libs: drop PLIST_SUB=FIBER in Makefile and expand %%FIBER%% in pkg-plist
  • devel/boost-all/ make C++17 unconditional

From misc POV:

  • Drop clang60 conditionals but keep USE_GCC from USES=compiler (non-trivial)
  • Drop print/harfbuzz/files/
  • Drop lang/phantomjs/files/
  • Drop net-mgmt/netdata/files/patch-src_freebsd__sysctl.c
  • Drop print/lyx/files/patch-src_TexRow.cpp
  • Drop sysutils/munin-common/files/
  • Drop __FreeBSD__ < 11 conditional from graphics/mesa-dri/files/patch-src_util_futex.h
  • Drop || (defined(__FreeBSD__) && __FreeBSD__ < 11) but keep in games/keeperrl/files/patch-stack__printer.cpp
13 ↗(On Diff #49726)

I can't approve leaving cruft in such a unconspicuous manner. Either don't touch or clean up better.

Instead of leaving .if ${OPSYS} == FreeBSD as cruft consult DeltaPorts or assume DragonFly irrelevant/unmaintainable (due to lack of a liaison), letting interested folks restore the support if desired.

29 ↗(On Diff #49726)

DragonFly currently has GCC 8 in base system, before that it was GCC 5. Both support C++11 just fine.

36 ↗(On Diff #49726)

Better convert to USES=compiler:gcc-c++11-lib (requires fixing build). It'd be safer on libc++ systems.

26 ↗(On Diff #49726)

Looks specific to modern Clang, regardless of OS type.

24 ↗(On Diff #49726)
26 ↗(On Diff #49726)

Required on FreeBSD to avoid conflict in <sys/socket.h>. Harmless on DragonFly

65 ↗(On Diff #49726)

DragonFly dropped i386 support eons ago. If conditional is really desired it should be limited to ${CHOSEN_COMPILER_TYPE} == clang.

258 ↗(On Diff #49726)

Required on FreeBSD to avoid conflict in <sys/socket.h>. Harmless on DragonFly

79 ↗(On Diff #49726)

Makes sense. MAP_NORESERVE still exists on DragonFly.

6 ↗(On Diff #49726)

cuse4bsd is still required if base system was built WITHOUT_CUSE. Maybe make CUSE port option unconditional, here and in sysutils/uhidd.

53 ↗(On Diff #49726)

DragonFly has neither P_KTHREAD nor P_KPROC. The port builds fine beacause of #elif defined(__DragonFly__).

68 ↗(On Diff #49726)

DragonFly has neither P_KTHREAD nor P_KPROC. The port builds fine beacause the substitution only affects #if defined(__FreeBSD__).

82 ↗(On Diff #49726)

Maybe this port should be DEPRECATED instead.

58 ↗(On Diff #49726)

Required on FreeBSD to avoid conflict in <sys/socket.h>. Harmless on DragonFly

46 ↗(On Diff #49726)

Required on FreeBSD to avoid conflict in <sys/socket.h>. Harmless on DragonFly

22 ↗(On Diff #49726)

Similar to net-mgmt/seafile-client but DragonFly support is missing.

37 ↗(On Diff #49726)

Doesn't build on DragonFly either way.

21 ↗(On Diff #49726)

Doesn't build on DragonFly either way.

165 ↗(On Diff #49726)

Maybe convert to .if exists(/usr/bin/fortune) (or reverse).

40 ↗(On Diff #49726)

Drop the conditional entirely and re-run make makesum instead of leaving cruft.

123 ↗(On Diff #49726)

DragonFly doesn't have linuxulator.

128 ↗(On Diff #49726)

DragonFly doesn't have linuxulator.

Rebase onto r483643, add www/node10

rene marked 24 inline comments as done.Nov 1 2018, 9:30 PM
rene added inline comments.
36 ↗(On Diff #49726)

This feels like a separate issue to me.

6 ↗(On Diff #49726)

Ah, good catch. I'll leave pkg-message unmodified.

82 ↗(On Diff #49726)

Perhaps, it is already marked BROKEN unconditionally in r481399 (2018-10-06). So it will probably be purged in less than half a year if it remains BROKEN.

165 ↗(On Diff #49726)

Converting to exists(/usr/bin/fortune) seems the cleanest to me.

rene marked an inline comment as done.

Process most of jbeich's suggestions and rebase onto r483743

This revision is now accepted and ready to land.Nov 2 2018, 1:01 PM
This revision was automatically updated to reflect the committed changes.