Bug 1245755 - 10-qt5-check-opengl2.sh doesn't work if more than one screen
Summary: 10-qt5-check-opengl2.sh doesn't work if more than one screen
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: qt5-qtbase
Version: 22
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Than Ngo
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: qt-5.5
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-07-22 16:39 UTC by Henrique Martins
Modified: 2016-01-08 14:44 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-01-08 14:44:05 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.