[mythtv] Ticket #3910: Mythshutdown cleanup

Daniel Kristjansson danielk at cuymedia.net
Wed Jan 14 13:44:53 UTC 2009


On Sat, 2009-01-10 at 10:36 +0000, Matthew Wire wrote:
> Thankyou for taking the time to review this patch.  I understand about
> the isRunning stuff although I'm not aware of any platform independent
> way of detecting if processes are running?

You can connect to the backends to check on those and check the job
queue for the other processes.

> As for the changes to mythdb get/set settings code I am hoping you may
> reconsider.  It was originally designed to save a few lines of code
> within mythshutdown but quickly grew out of that when I realised that
> there was a whole load of code duplication within the dbsettings code.
> The patch looks a lot more complex than it actually is because of the
> way it has been generated (00-3910-mythdb-dbsettings.patch).

Open another ticket for that, I'm not going to apply it anytime soon
myself but it will be there for anyone planning to work on the DB
stuff.

-- Daniel





More information about the mythtv-dev mailing list