[Pm-utils] [PATCH] sleep.d/55NetworkManager: don't call nonexistent dbus_send

[Victor Lowther]

On Wed, Jun 30, 2010 at 2:39 AM, Uri Okrent <uokrent at gmail.com> wrote:
> dbus provides dbus-send, not dbus_send. This typo means that
> NetworkManager was not actually shutting down interfaced or bringing
> them back up on suspend/resume.
>
> Signed-off-by: Uri Okrent <uokrent at gmail.com>

NAK, we provide dbus_send in the functions file we source at the
beginning of the script.

> ---
>  pm/sleep.d/55NetworkManager |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/pm/sleep.d/55NetworkManager b/pm/sleep.d/55NetworkManager
> index f3c6df5..89dcb34 100755
> --- a/pm/sleep.d/55NetworkManager
> +++ b/pm/sleep.d/55NetworkManager
> @@ -12,7 +12,7 @@ suspend_nm()
>  {
>        # Tell NetworkManager to shut down networking
>         printf "Having NetworkManager put all interaces to sleep..."
> -       dbus_send --system                         \
> +       dbus-send --system                         \
>                --dest=org.freedesktop.NetworkManager  \
>                /org/freedesktop/NetworkManager        \
>                org.freedesktop.NetworkManager.sleep && \
> @@ -23,7 +23,7 @@ resume_nm()
>  {
>        # Wake up NetworkManager and make it do a new connection
>        printf "Having NetworkManager wake interfaces back up..."
> -        dbus_send --system                        \
> +        dbus-send --system                        \
>                --dest=org.freedesktop.NetworkManager \
>                /org/freedesktop/NetworkManager       \
>                org.freedesktop.NetworkManager.wake && \
> --
> 1.7.0.4
>
> _______________________________________________
> Pm-utils mailing list
> Pm-utils at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/pm-utils
>