Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#5583 closed enhancement (fixed)

LIRC improvements

Reported by: danielk Owned by: danielk
Priority: minor Milestone: unknown
Component: mythtv Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Improves LIRC handling in MythTV.

  • Allows user to connect to the local LIRC daemon, even if it is running on say /dev/lircd2 rather than /dev/lircd. This is very useful if you have multiple LIRC daemons running, say one for transmit and nother for recv, you do not need to go through any contortions to make sure the LIRC daemon for recv is always on /dev/lircd
  • Allows user to connect to a remote LIRC daemon, using host[:port], ip[:port] instead of the default '/dev/lircd'.

Attachments (5)

5583-v1.patch (24.0 KB) - added by danielk 11 years ago.
lirc-5583-v2.patch (24.6 KB) - added by mrand 11 years ago.
Resync'ed patch with trunk. mythtv/libs/libmythui/lirc.cpp had a major conflicts - it would be advisable to double that everything is correct
02-5583-lirc-5583-v2.patch (24.6 KB) - added by Matthew Wire <devel@…> 11 years ago.
Resynced for trunk 19114. lirc.h was failing.
5583-v3.patch (25.9 KB) - added by danielk 11 years ago.
Updated patch
5583-v4.patch (28.2 KB) - added by danielk 11 years ago.
Adds SIGUSR2 handler, inspired by #5929.

Download all attachments as: .zip

Change History (10)

Changed 11 years ago by danielk

Attachment: 5583-v1.patch added

Changed 11 years ago by mrand

Attachment: lirc-5583-v2.patch added

Resync'ed patch with trunk. mythtv/libs/libmythui/lirc.cpp had a major conflicts - it would be advisable to double that everything is correct

Changed 11 years ago by Matthew Wire <devel@…>

Attachment: 02-5583-lirc-5583-v2.patch added

Resynced for trunk 19114. lirc.h was failing.

comment:1 Changed 11 years ago by Matthew Wire <devel@…>

Unfortunately this doesn't currently compile for me. I'm getting the following errors and can't work out why: In file included from mythmainwindow.cpp:29: lirc.h:16: erreur: function definition does not declare parameters mythmainwindow.cpp:118: erreur: ISO C++ forbids declaration of ‘LIRC’ with no type

comment:2 Changed 11 years ago by Dibblah

Status: newassigned

Changed 11 years ago by danielk

Attachment: 5583-v3.patch added

Updated patch

Changed 11 years ago by danielk

Attachment: 5583-v4.patch added

Adds SIGUSR2 handler, inspired by #5929.

comment:3 Changed 11 years ago by danielk

Resolution: fixed
Status: assignedclosed

(In [19499]) Fixes #5583. Allow LIRC device to be set and allow LIRC to be reset externally. This makes setting up a machine with multiple LIRC devices a simpler affair.

comment:4 Changed 11 years ago by danielk

(In [19523]) Refs #5583. Make sure we handle modifier keys correctly for LIRC. Qt4's QKeySequence requires that the modifier name be followed by a '+' instead of the lircrc standard of '-'. also we want to release the modifier keys after releasing the character key, so the key release loop has been inverted.

comment:5 Changed 11 years ago by danielk

(In [19547]) Refs #5583. Make LIRC a little less verbose, after first unsuccessful attempt to connect to LIRC downgrade init warnings and errors down to VB_FILE, with the exception of the message saying we're going to stop trying to connect after a large number of retries and the message saying we successfully connected to the daemon.

Note: See TracTickets for help on using tickets.