Bug 256072 – x11-misc/xscreensaver fails to build with USE="-opengl"

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Goran Mekić 2009-01-23 11:18:18 UTC]

Title said it all

Reproducible: Always

Steps to Reproduce:

[Comment 1 Goran Mekić 2009-01-23 11:20:27 UTC]

Created attachment 179434 [details, diff]
add-gl-and-glu-libs.patch

[Comment 2 Goran Mekić 2009-01-23 11:31:21 UTC]

Created attachment 179437 [details, diff]
add-gl-and-glu-libs.patch

[Comment 3 Goran Mekić 2009-01-23 11:32:35 UTC]

Created attachment 179439 [details, diff]
xscreensaver-5.08.ebuild.patch

[Comment 4 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2009-01-23 20:37:41 UTC]

Comment on attachment 179437 [details, diff]
add-gl-and-glu-libs.patch

"-lGL -lGLU" are libraries (LIBS), not LDFLAGS.

[Comment 5 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2009-01-23 20:39:50 UTC]

Reopen with build log attached and 'emerge --info'.

[Comment 6 Maxim Britov 2009-01-24 18:34:57 UTC]

Created attachment 179573 [details, diff]
Fix for wrong retire glforrestfire

[Comment 7 Maxim Britov 2009-01-24 18:41:45 UTC]

This issue with retired glforrestfire.
glforrestfire should be build only with USE="opengl", but

Makefile.in:7399 is
RETIRED_EXES = glforestfire
RETIRED_GL_EXES = @RETIRED_GL_EXES@

and Makefile.in:9666 is
all: $(EXES) $(RETIRED_EXES)

With all above it will compile glforrestfire with and without enabled opengl.
Without opengl it fail.

[Comment 8 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2009-01-25 00:59:47 UTC]

Reopening.

[Comment 9 Jouni Rinne 2009-02-15 21:12:52 UTC]

Confirmed, glforestfire patch allows xscreensaver-5.08 to compile with USE="-opengl". Compiling fails without it.

[Comment 10 Maciej Mrozowski 2009-03-13 19:58:04 UTC]

Confirmed on ~amd64 box, solves linking issue.

[Comment 11 Tomáš Chvátal (RETIRED) 2009-03-13 20:32:09 UTC]

Patch in the tree.
Sorry for the delay.
Maintainer: slap me later for adding it if you dont like it. We need it working for kscreensavers where pple report bug to us kde.

[Comment 12 Torsten Veller (RETIRED) 2009-03-14 13:26:00 UTC]

The patch doesnot work with xscreensaver-5.07.

[Comment 13 Tomáš Chvátal (RETIRED) 2009-03-14 13:44:29 UTC]

(In reply to comment #12)
> The patch doesnot work with xscreensaver-5.07.
> 

Indeed it does not, In 5.07 it is in correct GL_EXES part. The problem seems to be that i tested with 5.08:
emerge -v1 xscreensaver
and then i tested with versioned 5.07 with this command:
emerge -v1 =xscreensaver-5.08
So i accidentaly tested double the same version and was sure that it apply on 07 too :(

Sorry for mess