This patch is syntactically incorrect, which leads me to question how
much it's been tested :-)

+       if [ -d /sys/class/rc -a "$DISABLE_KERNEL_SUPPORT" = "true"];
then

There's a missing space before the ] there.  Also, I would generally
advise avoiding the use of -a and -o inside [ ] as the rules are arcane
and not as portable as you might hope.  So I would write this as:

  if [ -d /sys/class/rc ] && [ "$DISABLE_KERNEL_SUPPORT" = true ]; then

Also, all changes need to be made in the development release (currently
quantal) first.  If they meet the criteria in
https://wiki.ubuntu.com/StableReleaseUpdates, you can then work through
those to apply the patch to precise.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1038371

Title:
  disabling in-kernel support should be optional

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/lirc/+bug/1038371/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to