Comment 3 for bug 1038371

Revision history for this message
Colin Watson (cjwatson) wrote :

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.