Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 692212 - media-libs/cogl: Add support for media-libs/libglvnd
Summary: media-libs/cogl: Add support for media-libs/libglvnd
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: Normal enhancement (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 692206
  Show dependency tree
 
Reported: 2019-08-15 10:55 UTC by Maik
Modified: 2019-09-03 12:23 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch against cogl-1.22.2.ebuild removing test useflag. (cogl-1.22.2.ebuild-remove-test.patch,2.70 KB, text/plain)
2019-09-03 12:23 UTC, Maik
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Maik 2019-08-15 10:55:43 UTC
media-libs/cogl has a dependency on eselect-opengl when USE="test" is set, thus blocking unmasking libglvnd usage. Stub for now to work as blocker.
Comment 1 Maik 2019-08-15 16:07:03 UTC
Nice comment in ebuild:
# Use swrast for tests, llvmpipe is incomplete and "test_sub_texture" fails
# NOTE: recheck if this is needed after every mesa bump
Question is, when was the last recheck of this. So needs recheck and hopefully the dependency can simply be dropped.
Comment 2 Matt Turner gentoo-dev 2019-08-24 17:23:16 UTC
FWIW, cogl's test suite passes for me with llvmpipe and fails with the classic swrast driver.
Comment 3 Maik 2019-09-03 12:23:41 UTC
Created attachment 588886 [details]
Patch against cogl-1.22.2.ebuild removing test useflag.

I've taken a deeper look at the ebuild and this seems quite absurd. It has dependencies to mesa[classic] and eselect-opengl when USE="test" but tests are disabled anyway with RESTRICT=test because those always fail(ed). Moreover, it uses
LIBGL_DRIVERS_PATH="${EROOT}/usr/$(get_libdir)/mesa", looks like some leftover from ancient times. At least I wouldn't know how that path could be used nowadays.
Conclusion: tests doesn't work anyway, is unmaintained, so just remove it. Attaching a patch doing so.