Page MenuHomeFreeBSD

Make libglvnd a LIB_DEPEND in various usages
ClosedPublic

Authored by kbowling on Aug 8 2021, 8:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 9:31 PM
Unknown Object (File)
Fri, Apr 19, 10:45 PM
Unknown Object (File)
Wed, Apr 17, 11:05 AM
Unknown Object (File)
Mon, Apr 8, 12:26 AM
Unknown Object (File)
Fri, Mar 29, 8:05 PM
Unknown Object (File)
Fri, Mar 29, 1:07 AM
Unknown Object (File)
Mar 23 2024, 2:27 PM
Unknown Object (File)
Mar 17 2024, 7:52 AM
Subscribers
None

Details

Summary

There are still reports of people with old systems running into issues on pkg upgrade. I think libglvnd must be a LIB_DEPEND in the main branch now to help with dependency solving.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kbowling created this revision.

I forgot to add USES+=gl, but even with that the LIB_DEPENDS wasn't working as expected. Convert to explicit LIB_DEPENDS.

Looks OK (except inline nits):

  • www/qt5-webengine and all of its dependencies still build fine
  • x11/alacritty + emulators/yuzu-qt5 still install/run fine
graphics/mesa-devel/Makefile
31

Sort by port origin instead of the library name for consistency with current style here and my other ports. I don't agree with everything portlint -C, portfmt or portclippy suggest.

graphics/mesa-libs/Makefile
10

Why += instead of =? .include appears at the end of this Makefile which itself isn't sourced/included unlike mesa-dri/Makefile.common.

graphics/s2tc/Makefile
14 ↗(On Diff #93498)

Neither an OpenGL provider nor requires libOpenGL at runtime. Disabling dlopen via CONFIGURE_ARGS=--disable-runtime-linking exposes libtxc_dxtn.so in poudriere testport output but not libOpenGL, suggesting only headers are used. Try the following smoke test:

$ pkg install s2tc
$ pkg delete -f libglvnd
$ pkg info -l s2tc | sed -n '/bin/p; /lib\//p' | xargs strings | grep GL.\*so
$ fetch https://people.math.sc.edu/Burkardt/data/tga/earth.tga
$ s2tc_compress -i earth.tga -o earth.dds -t DXT5
$ file earth.dds
earth.dds: Microsoft DirectDraw Surface (DDS): 512 x 512, compressed using DXT5
This revision is now accepted and ready to land.Aug 11 2021, 12:30 AM
This revision now requires review to proceed.Aug 11 2021, 2:22 AM

On landing maybe bump PORTREVISION in the modified ports to make upgrades more predictable (e.g., for portmaster users) and discover fallout early (if any but unlikely).

This revision is now accepted and ready to land.Aug 11 2021, 2:28 PM