[mythtv] Re: [mythtv-commits] mythtv commit: r7392 - in trunk/mythtv by danielk

[Bruce Markey]

mythtv at cvs.mythtv.org wrote:
>       Author: danielk
>         Date: 2005-10-05 13:48:47 +0000 (Wed, 05 Oct 2005)
> New Revision: 7392
>    Changeset: http://cvs.mythtv.org/trac/changeset/7392
> 
> Modified:
> 
>    trunk/mythtv/docs/mythtv-HOWTO.sgml
>    trunk/mythtv/setup/backendsettings.cpp
>    trunk/mythtv/setup/backendsettings.h
>    trunk/mythtv/setup/main.cpp
> 
> Log:
> 
> Condenses the two questions in when you start-up mythtv-setup
> onto one dialog. This also removes the warning in the howto
> and replaces it with a check for a running mythbackend on
> startup. If it is seen running locally, a warning about what
> isn't safe while the backend is running is added to this
> initial dialog.
> 
> The check for mythbackend is very simple, it just runs
> "ps -ae | grep mythbackend". So if you rename the backend
> executanble it won't detect the backend. It also won't dedect
> a master backend running on a different host, etc. It just
> catches the common case.

Couldn't it just try connecting to the MBE control port as a
slave or frontend would?


I really like this. I though it would be six of one half a dozen
of the other but this is potentially much better.

Couple things. I see the text for the two old and one new help
text items truncated on the first line. For exanple:

 "Would you like to clear all program data and channel settings befo" [sic]
 "configuration? This will not affect any existing recordings.:"

The text "re starting " does not appear on screen.

The initial focus should be on the "OK" button. Checking a box
is a very execptional thing and the user should not be lead to
believe they should check them.

When the backend warning is appiled, there should also be an "Exit"
button that gets the inital focus.

--  bjm