lirc init script seems to load things backwards

Bug #697999 reported by Joel Ebel
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
lirc (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: lirc

The lirc manual describes how to load multiple devices here:
http://www.lirc.org/html/configure.html#multiple

Assuming that the primary device get started on lircd and the secondary device gets started on lircd1:

In short the proper method is to start lircd1 with --listen, and then lircd with --connect

The lircd init script does this backwards, lircd with --listen, and then lircd1 with --connect.

The result is that only the primary device is visible on lircd. Both devices are visible on lircd1, however, most applications will require additional configuration in order to communicate with both devices.

Tags: patch
Revision history for this message
Joel Ebel (jbebel) wrote :

Attached is a patch which does what I expect. It pains me to not include my patch for bug 698007 in the patch, but it's a separate bug, so I'm leaving that fix there. Please consider it as well.

Revision history for this message
Joel Ebel (jbebel) wrote :

As much as it pains me, I realized the rest of the file used tabs, so here's a new patch with tabs instead of spaces.

tags: added: patch
Revision history for this message
Kees Cook (kees) wrote :

I've asked Mario to take a look at this bug (and698007 ) to review the patch. Hopefully we can get this into Natty asap. Thanks for the work on this!

Changed in lirc (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Joel Ebel (jbebel) wrote :

Got around to setting up Precise on my mythtv box, and I'm sad to see that this bug hasn't been fixed yet. I provided a patch, which still applies cleanly to lirc in Precise.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package lirc - 0.9.0-0ubuntu5

---------------
lirc (0.9.0-0ubuntu5) trusty; urgency=medium

  * debian/lirc.postinst, debian/lirc.templates, debian/po: drop a number of
    debconf templates that are no longer being prompted for at all due to
    the dropping of the lirc-modules-source package and the lirc.config some
    time ago.
  * debian/lirc.postinst: clean up the remaining debconf handling to not
    override settings in /etc/lirc/hardware.conf.
  * debian/lirc.postinst: drop code for removing /var/log/lircd, which has
    been gone for more than a decade.
  * fix blacklist handling, which creates files not used by modprobe, and
    doesn't put all the contents in a single file.
  * drop obsolete Ubuntu migration code that dates from jaunty and lucid.
  * fix a bug that causes transmitter device/driver settings to always be
    cleared on config change.
  * Adjust lircd handling for multiple devices so that events for all of
    them will appear on /dev/lircd as intended. Thanks to Joel Ebel for
    the patch. LP: #697999.
  * Use /run/lirc, the canonical name for /var/run/lirc. LP: #474701.
  * Fix the init script to not report failures when stopping an
    already-stopped daemon.
 -- Steve Langasek <email address hidden> Sun, 16 Mar 2014 05:35:47 -0700

Changed in lirc (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.