Opened 13 years ago

Closed 13 years ago

#1915 closed patch (fixed)

mythfrontend does not entirely heed the -display option

Reported by: rl@… Owned by: danielk
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

myhtfrontend advertises the -display command line option, but does not always heed it. In particular, when I use my nVidia FX 5200 in dual-monitor mode, and from screen 0, I run "mythfrontend -display :0.1" to display myth on screen 1, almost everything works. But when it comes time to actually playback a recording (using XvMC), the screen goes blank.

If I set the DISPLAY environment variable to :0.1 and do not use -display, everything works, and the playback is normal on screen 1. (Thus, I have a simple, completely effective workaround.)

I checked the source code, and I see several instances of XOpenDisplay(NULL), which, of course, opens the default display, not the display that I specified with the -display command line.

Attachments (1)

1915.patch (7.6 KB) - added by danielk 13 years ago.
possible fix

Download all attachments as: .zip

Change History (6)

comment:1 Changed 13 years ago by danielk

Milestone: 0.20
Owner: changed from Isaac Richards to danielk
Version: head

Changed 13 years ago by danielk

Attachment: 1915.patch added

possible fix

comment:2 Changed 13 years ago by danielk

Type: defectpatch

rl, can you give this a try?

I only fixed mythfrontend, but if that works this should be trivial to port to mythtv. I don't have a good test setup at the moment, so this is untested with multiple X11 screens.

comment:3 Changed 13 years ago by rl@…

Resolution: fixed
Status: newclosed

The patch seems to work fine. Thanks!

comment:4 Changed 13 years ago by anonymous

Resolution: fixed
Status: closedreopened

patch has not yet been applied.

comment:5 Changed 13 years ago by danielk

Resolution: fixed
Status: reopenedclosed

(In [10257]) Closes #1915. Heads -display param in non-Qt based XOpenDisplay() calls.

This grabs the "-display" param before QApplication steals it from us and saves it for later use in the context. We then use this in MythXOpenDisplay() which is used in place of XOpenDisplay() in the various places where we open the display ourselves.

This changes the binary revision so you will need to recompile the plugins.

Note: See TracTickets for help on using tickets.