[mythtv-commits] mythtv commit: r15932 - in trunk/mythtv/libs by cpinkham

[mythtv at cvs.mythtv.org]

      Author: cpinkham
        Date: 2008-02-12 04:53:58 +0000 (Tue, 12 Feb 2008)
New Revision: 15932
   Changeset: http://cvs.mythtv.org/trac/changeset/15932

Modified:

   trunk/mythtv/libs/libmyth/dbutil.cpp
   trunk/mythtv/libs/libmyth/dbutil.h
   trunk/mythtv/libs/libmyth/mythcontext.cpp
   trunk/mythtv/libs/libmyth/mythcontext.h
   trunk/mythtv/libs/libmythtv/dbcheck.cpp

Log:

Various changes related to the new Database Backup that runs before any schema
upgrade.

- Don't bother running the backup if the existing DB Schema version is empty 
  (ie, when the DB is first created).
  
- Use system() instead of myth_system() so that we can get the result of
  the mysqldump command.
  
- Break out the compression into a separate call so that we can get the
  result of mysqldump instead of the result of the gzip.

- If the user chooses to upgrade the schema, display a warning if the
  database was not successfully backed up.

- If the DB was succesfully backed up, display the filename along with a
  message telling the user where they can locate the backup file if they
  experience issues.

References #4613.

WARNING: This bumps the MYTH_BINARY_VERSION version because of changes to
  MythContext::PromptForSchemaUpgrade(), so a recompile of mythtv and all
  plugins is necessary.