Review: Approve
A couple of nitpicks:

86: cmd = sqliteexe + u' "' + olddb + u'" .dump'
might be better as 
  cmd= u'%s  "%s" .dump' % (sqliteexe, olddb)


-- 
https://code.launchpad.net/~j-corwin/openlp/migration/+merge/13537
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to