[mythtv-commits] mythtv commit: r21955 - in trunk by cpinkham

[mythtv at cvs.mythtv.org]

      Author: cpinkham
        Date: 2009-09-20 02:29:45 +0000 (Sun, 20 Sep 2009)
New Revision: 21955
   Changeset: http://cvs.mythtv.org/trac/changeset/21955

Modified:

   trunk/mythplugins/mythmusic/mythmusic/dbcheck.cpp
   trunk/mythplugins/mythvideo/mythvideo/dbcheck.cpp
   trunk/mythtv/libs/libmyth/dbutil.cpp
   trunk/mythtv/libs/libmyth/dbutil.h
   trunk/mythtv/libs/libmyth/schemawizard.cpp
   trunk/mythtv/libs/libmyth/schemawizard.h
   trunk/mythtv/libs/libmythdb/mythversion.h
   trunk/mythtv/libs/libmythtv/dbcheck.cpp

Log:

Fixes some confusing DB schema upgrade messages in the SchemaUpgradeWizard
by adding a string that gets printed in the logs to indicate which schema
we are checking.  Currently only MythTV, MythVideo, and MythMusic are
using the schema wizard, but the log messages could be confusing when the
relatively low MythVideo and/or MythMusic version numbers were printed out
with the same format message as the higher MythTV schema version number.
All messages should now have 'MythTV', 'MythVideo', 'MythMusic' in them.

Modify the schema wizard and DB backup util's BackupDB() methods to return
an enum instead of boolean.  Now we can distinguish whether a backup
was made, failed, skipped for empty DB, disabled, etc..  The schema wizard's
screen is now fixed to show the proper message based on the DB upgrade
status.  Previously we would tell the user a backup was located in an
empty location if they had backups disabled (including Windows users where
backups are hardcoded to be disabled).

Fix a few places where we were manually upgrading the DB schema version
rather than using the performActualUpdate() routine.  This meant that in
a few places, we weren't displaying the 'Upgrading to schema version X'
message for that specific version upgrade.  These messages are now
printed manually in the upgrades that bypass performActualUpdate().