[Bug 1096641] Re: Patch to remove Bashism in /usr/share/acpi-support/power-funcs

[Antonis Kanouras]

Hello and thanks for the fast reply :)

I guess an argument for it would be for use as a helper in local
scripts.

While writing a local workaround for #467825, it happened to suit my
needs. I would have written a proper one if it didn't exist though.
However, since then I caved in and patched upowerd, meaning I have no
use for it anymore.

Another issue with it is that if it's used by multiple scripts, it'll
only work for the first one that happens to be called by acpid per power
state change.

So it's up to you in the end. If you decide its functionality is useful
indeed, and since I seem to have stirred the waters here :), I could
send you an improved version of it (or of the whole file while I'm at
it).

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

Title:
  Patch to remove Bashism in /usr/share/acpi-support/power-funcs

Status in “acpi-support” package in Ubuntu:
  Incomplete

Bug description:
  Hello,

  Please accept this patch to remove a bashism in /usr/share/acpi-
  support/power-funcs .

  Thanks

  ProblemType: Bug
  DistroRelease: Ubuntu 12.10
  Package: acpi-support 0.141 [modified: usr/share/acpi-support/power-funcs]
  ProcVersionSignature: Ubuntu 3.5.0-22.33-generic 3.5.7.2
  Uname: Linux 3.5.0-22-generic x86_64
  NonfreeKernelModules: nvidia
  ApportVersion: 2.6.1-0ubuntu9
  Architecture: amd64
  Date: Sun Jan  6 19:17:16 2013
  MarkForUpload: True
  ProcEnviron:
   LANGUAGE=el_GR:en_GB:en_US:en
   TERM=xterm
   PATH=(custom, no user)
   LANG=el_GR.UTF-8
   SHELL=/bin/bash
  SourcePackage: acpi-support
  UpgradeStatus: No upgrade log present (probably fresh install)

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/acpi-support/+bug/1096641/+subscriptions