Shutdown doesn't work from frontend in MythTV 0.23

Bug #574877 reported by Richard Brooklyn
24
This bug affects 3 people
Affects Status Importance Assigned to Milestone
MythTV
Fix Released
Unknown
mythtv (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: mythtv

Since 0.23.0+fixes24158-0ubuntu2, a patch has been applied which causes mythtv to use DBus to attempt to shut down the system. Whilst this is a step in the right direction for the 'doing it right' argument. It breaks existing behaviour and causes dependencies which don't appear to be satisfied by the mythtv package itself.

Case in point;
My set-up was a custom installed ubuntu lucid system. Installed with minimal X and only whatever else I could get away with. Before this patch came in, I was using two custom commands to shut down and restart the system. Now they no longer work, and the dbus method appears to not work either. The only way I can shutdown currently is to log into the system and issue the command manually. The 'dbus' package is already installed, but it appears to not be enough to make this work. I do not know which package needs installing or configuring and this may add extra overhead this system cannot afford.

No error messages appear at all to tell me why anything has failed, so I have not included logs.

This patch appears to have made it in very late in Lucid's development cycle. At the very least the extra dependencies need to be added to the mythtv package so the system can shutdown and reboot. Ideally, I feel this update should be reversed until it has received more testing, as it did suddenly break a couple of machines when a feature freeze was in effect.

The behaviour in ubuntu1 version was fine, and worked as expected.

Revision history for this message
Lomion (lomion) wrote :

This is not working in my environment either!
Is there any way to work around it (an additional package or a manual build)?

Revision history for this message
Thomas Mashos (tgm4883) wrote :

Thank you for your bug report. Could you please describe your environment, how you set up your minimal system, what packages you installed, etc. I've tried reproducing this in a VM but was unable to.

Changed in mythtv (Ubuntu):
status: New → Incomplete
Revision history for this message
Jan (jan-tux) wrote :

Since proper_shutdown_behavior_for_ubuntu.patch my mythfrontend will not longer shutdown my client.

I tried the follow command on the console with success:
dbus-send --print-reply --system --dest=org.freedesktop.Hal /org/freedesktop/Hal/devices/computer org.freedesktop.Hal.Device.SystemPowerManagement.Shutdown

Additional I wrote a liltle testprogram with the content of the DBusHalt function. Works perfect also.

So, if the DBusHalt is called then everything should by fine.

But only if CONFIG_QTDBUS defined. Is it?

Revision history for this message
Jan (jan-tux) wrote :

For myself the proper_shutdown_behavior_for_ubuntu.patch makes no sense, because I configured my own shutdown script, to suspend my client. The suspend was the only way the wake-up my frontend with my remote control again.

Please remove the proper_shutdown_behavior_for_ubuntu.patch!!

Revision history for this message
Thomas Mashos (tgm4883) wrote :

Shutdown != suspend. Perhaps mythtv needs to add suspend functionality to that menu.

Revision history for this message
Lomion (lomion) wrote :

I am not using any scripts ... I'ts just not shutting down anymore.

I have a ZOTAC ION ITX-F Mainboard in a standalone, diskless mythfrontend.
Originally I installed Mythbuntu 9.10 which was working fine for me (including the shutdown feature) but with MYthtv 0.22.
Then I upgraded to Mythbuntu 10.04 which did not break anything but the I moved to the weekly Mythtv builds afterwards which broke the shutdown behaviour ...

Does this help?

Revision history for this message
Mario Limonciello (superm1) wrote :

@Lombion:

Do you have consolekit and policykit installed? They're not explicit dependencies, and depending on how minimalistic your setup is, I think it's possible they won't be there...

Revision history for this message
Jan (jan-tux) wrote :

@Mario
It will not shutdown (see #3) with policykit and consolekit installed:

ii policykit-1 0.96-2
ii policykit-1-gnome 0.96-2ubuntu2
ii consolekit 0.4.1-3ubuntu1

But as I said before it will shutdown with a dbus-send and my "myth-DBusHalt" program.

@Thomas
You be right! But why removing functionality, that give some people the possibility to configure a better solution for their own.
The normal user have always the change to leave the configurable shutdown or reboot command in the GUI blank.

Revision history for this message
Richard Brooklyn (rbrooklyn) wrote :

Hi Thomas,

My install is fairly basic. I used the 'alternative' cd i386 image to install a 'minimal' system, and added X, mythfrontend and other bits from that. I have mythfrontend starting automatically from init scripts. very bare bones.

Unless mythtv or X pulled in anything dbus/policykit/consolekit related, then it's not installed.

Is it not possible to go back to the old behaviour? And maybe have the 'default' options to call dbus scripts to shutdown if the fields are left blank by the user? I think this would be the best of both worlds.

Revision history for this message
Thomas Mashos (tgm4883) wrote :
Changed in mythtv (Ubuntu):
status: Incomplete → Confirmed
summary: - proper_shutdown_behavior_for_ubuntu.patch breaks existing behaviour and
- may require new unknown dependancies
+ Shutdown doesn't work from frontend in MythTV 0.23
Changed in mythtv:
status: Unknown → New
Revision history for this message
Thomas Mashos (tgm4883) wrote :

I've pushed the patch from upstream to auto-builds, it should be there in the next build.

Changed in mythtv (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Thomas Mashos (tgm4883) wrote :

@Jan

Sorry, missed your post. That is a different issue. You will need to file a separate bug for that.

Revision history for this message
Thomas Mashos (tgm4883) wrote :

Should be fixed in 0.23.0+fixes25111 Please test

Revision history for this message
Douglas Mackay (o-launchpad-mackayd-co-uk) wrote :

I've tested 0.23.0+fixes25111 and shutdown is working.

Revision history for this message
Jan (jan-tux) wrote :

Tested with 0.23.0+fixes25129-0ubuntu0+mythbuntu2 and shutdown is working.
... and my girlfriend is happy again :-)

Revision history for this message
Jan (jan-tux) wrote :

@Thomas #12
Which issue: #3 or #8

Revision history for this message
Thomas Mashos (tgm4883) wrote :

@Jan
#8. #3 is fixed

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

This bug was fixed in the package mythtv - 0.23.0+fixes25138-0ubuntu1

---------------
mythtv (0.23.0+fixes25138-0ubuntu1) maverick; urgency=low

  [ Mario Limonciello ]
  * new upstream checkout (25138)

  [ Thomas Mashos ]
  * Added missingok for mirobridge log in logrotate (LP: #581283)
  * Backend now waits for udev to finish (LP: #556204)
  * Added patch to properly detect dbus during compile (LP: #574877)
 -- Mario Limonciello <email address hidden> Sun, 20 Jun 2010 00:11:00 -0500

Changed in mythtv (Ubuntu):
status: Fix Committed → Fix Released
Changed in mythtv:
status: New → Fix Released
Revision history for this message
hinkie (hinkie) wrote :

Thomas Mashos wrote on 2010-06-11: #5
Shutdown != suspend. Perhaps mythtv needs to add suspend functionality to that menu.

I'd like to vote for the inclusion of a suspend option from the menus please. it would allow suspend from an IR remote, and WOL from a phone, alt PC whatever...

Revision history for this message
Dave Walker (davewalker) wrote :

@hinkie, This issue seemed to be targeted towards "Shutdown". If you have an interest in the addition of a "Suspend" feature, Please can you open a new bug. Thanks :)

Revision history for this message
hinkie (hinkie) wrote :
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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