In compiz-0.8.8_9, compiz-manager doesn't cover the possibility of duplicate output from glxinfo at line 238, which causes check_texture_size() to fail. The possible solution is adding head -n 1 to the command: TEXTURE_LIMIT=$(glxinfo -l | grep GL_MAX_TEXTURE_SIZE | head -n 1 | sed 's/.*=[^0-9]//g')
(In reply to Vladimir Bychik from comment #0) @Vladimir This seems reasonable. I tested your suggestion against my running copy w/o incident. I've created an svn diff (patch) that implements your suggestion. But before I attach it. Would you kindly re-title this pr to: x11-wm/compiz: compiz-manager texture size parsing error so that it better reflects the port in question? There is no wm category, nor compiz-manager port. :-) Thanks for taking the time to report this, and provide a working solution! :-) --Chris
Created attachment 215870 [details] x11-wm/compiz: correct texture size parse error Uses proposed solution to texture parse error generated in compiz-manager. Proposed solution by Vladimir Bychik Patch provided by MAINTAINER Thanks! --Chris
A commit references this bug: Author: 0mp Date: Sun Oct 4 19:04:20 UTC 2020 New revision: 551440 URL: https://svnweb.freebsd.org/changeset/ports/551440 Log: x11-wm/compiz: Fix compiz-manager texture size parsing error compiz-manager doesn't cover the possibility of duplicate output from glxinfo at line 238, which causes check_texture_size() to fail. Let's fix it by limiting the number of output lines to 1. PR: 247476 Submitted by: Vladimir Bychik Approved by: maintainer (Chris Hutchinson) MFH: 2020Q4 Changes: head/x11-wm/compiz/Makefile head/x11-wm/compiz/files/compiz-manager.in
Thanks for the patch! Let's see, if we can get it MFH'ed now...
A commit references this bug: Author: 0mp Date: Sun Oct 4 23:08:38 UTC 2020 New revision: 551469 URL: https://svnweb.freebsd.org/changeset/ports/551469 Log: MFH: r551440 x11-wm/compiz: Fix compiz-manager texture size parsing error compiz-manager doesn't cover the possibility of duplicate output from glxinfo at line 238, which causes check_texture_size() to fail. Let's fix it by limiting the number of output lines to 1. PR: 247476 Submitted by: Vladimir Bychik Approved by: maintainer (Chris Hutchinson) Approved by: ports-secteam (fluffy) Changes: _U branches/2020Q4/ branches/2020Q4/x11-wm/compiz/Makefile branches/2020Q4/x11-wm/compiz/files/compiz-manager.in