Bug 1245755

Summary:	10-qt5-check-opengl2.sh doesn't work if more than one screen
Product:	[Fedora] Fedora	Reporter:	Henrique Martins <fedora>
Component:	qt5-qtbase	Assignee:	Than Ngo <than>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	22	CC:	jgrulich, jreznik, kevin, ltinkl, rdieter, rnovacek, than
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-01-08 14:44:05 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1229922
Bug Blocks:

Description Henrique Martins 2015-07-22 16:39:10 UTC

Description of problem:
Problem is actually on qt5-qtbase-gui, but bugzilla doesn't recognize that as a component of f22 (will look later where to file that bug :-).

/etc/X11/xinit/xinitrc.d/10-qt5-check-opengl2.sh errors out if DISPLAY has more than one screen.

Version-Release number of selected component (if applicable):
qt5-qtbase-gui-5.4.2-2.fc22.x86_64

How reproducible:
Always

Steps to Reproduce:
1. Computer with more than one screen, e.g. :0.0 and :0.1
2. Start X (at least via XDM)
3. look in ~/.xsession-errors

Actual results:
~/.xsession-errors contains:
/etc/X11/xinit/xinitrc.d/10-qt5-check-opengl2.sh: line 3: [: 3
3: integer expression expected

as the output of glxinfo contains two lines with OpenGL version string, one for each screen

Expected results:
No error in ~/.xsession-errors

Additional info:
Seems to be of no other consequence if OpenGL is 2 or greater

Comment 1 Henrique Martins 2015-07-22 16:40:44 UTC

Looks like a simple "head -1" in the pipe that sets OPENGL_VERSION would work.

Comment 2 Kevin Kofler 2015-08-03 12:58:42 UTC

Thanks for the report. One correction though: "head -1" is deprecated, "head -n 1" is the POSIX syntax.

So the line should read:
OPENGL_VERSION=`LANG=C glxinfo | grep '^OpenGL version string: ' | head -n 1 | sed -e 's/^OpenGL version string: \([0-9]\).*$/\1/g'`

Comment 3 Fedora Update System 2015-08-08 23:41:31 UTC

qt5-qtbase-5.5.0-15.fc23 has been submitted as an update for Fedora 23.
https://admin.fedoraproject.org/updates/qt5-qtbase-5.5.0-15.fc23

Comment 4 Kevin Kofler 2015-08-09 11:45:09 UTC

What's the roadmap for 5.5 for F22? Shall I backport this trivial fix to our 5.4 packaging or are we pushing 5.5 in the next few hours?

Comment 5 Rex Dieter 2015-08-10 14:14:41 UTC

Good question, let's discuss it in tomorrow's kde-sig meeting.

Tentatively marking this as depending on 5.5 for now

Comment 6 Fedora Update System 2015-08-18 05:23:41 UTC

qt5-qtbase-5.5.0-15.fc23 has been pushed to the Fedora 23 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 7 Rex Dieter 2016-01-08 14:44:05 UTC

f22 has qt-5.5 now too for awhile, closing