← Back to team overview

openlp-core team mailing list archive

[Merge] lp:~springermac/openlp/packaging-py3 into lp:openlp/packaging

 

Jonathan Springer has proposed merging lp:~springermac/openlp/packaging-py3 into lp:openlp/packaging.

Requested reviews:
  OpenLP Core (openlp-core)

For more details, see:
https://code.launchpad.net/~springermac/openlp/packaging-py3/+merge/218500

Ran 2to3 and changed line lengths to 120 characters.
Also fixed a few python3 issues and add code to include the default theme's json file.
-- 
https://code.launchpad.net/~springermac/openlp/packaging-py3/+merge/218500
Your team OpenLP Core is requested to review the proposed merge of lp:~springermac/openlp/packaging-py3 into lp:openlp/packaging.
=== modified file 'osx/macosx-builder.py'
--- osx/macosx-builder.py	2013-02-03 14:56:42 +0000
+++ osx/macosx-builder.py	2014-05-06 19:57:38 +0000
@@ -118,7 +118,7 @@
 import sys
 from shutil import copy, rmtree
 from subprocess import Popen, PIPE
-from ConfigParser import SafeConfigParser as ConfigParser
+from configparser import ConfigParser
 from argparse import ArgumentParser
 
 
@@ -128,7 +128,7 @@
     """
     for path in os.environ["PATH"].split(os.pathsep):
         if os.access(os.path.join(path, command), os.X_OK):
-            print "%s/%s" % (path, command)
+            print("%s/%s" % (path, command))
             return "%s/%s" % (path, command)
 
 
@@ -137,6 +137,7 @@
     The :class:`MacosxBuilder` class encapsulates everything that is needed
     to build a Mac OS X .dmg file.
     """
+
     def __init__(self):
         self.setup_args()
         self.setup_system_paths()
@@ -150,7 +151,7 @@
         """
         if len(args) > 0:
             text = text % tuple(args)
-        print text
+        print(text)
 
     def _print_verbose(self, text, *args):
         """
@@ -173,7 +174,7 @@
             self._print(error)
             raise Exception(err_msg)
         return output
- 
+
     def _get_directory_size(self, directory):
         """
         Return directory size - size of everything in the dir.
@@ -184,16 +185,15 @@
                 filename = os.path.join(path, file)
                 dir_size += os.path.getsize(filename)
         return dir_size
-     
+
     def _get_mountpoints(self):
         """
         Return list of mounted disks on Mac.
         """
         # Get the output in plist format.
         paths = []
-        output = self._run_command([self.hdiutil, 'info', '-plist'],
-                u'Detecting mount points failed.')
-        pl = plistlib.readPlistFromString(output)
+        output = self._run_command([self.hdiutil, 'info', '-plist'], 'Detecting mount points failed.')
+        pl = plistlib.readPlistFromBytes(output)
         for image in pl['images']:
             for se in image['system-entities']:
                 if se.get('mount-point'):
@@ -207,33 +207,25 @@
         """
         parser = ArgumentParser()
         parser.add_argument('-b', '--branch', metavar='BRANCH', dest='branch',
-            help='Specify the path to the branch you wish to build.')
-        parser.add_argument('--devel', dest='devel',
-            action='store_true', default=False,
-            help='Development build does not have set icons for .dmg file '
-                'and .dmg filename contains bzr revision number.')
-        parser.add_argument('-d', '--documentation', metavar='DOCS',
-            dest='docs',
-            help='Specify the path to the documentation branch.')
+                            help='Specify the path to the branch you wish to build.')
+        parser.add_argument('--devel', dest='devel', action='store_true', default=False,
+                            help='Development build does not have set icons for .dmg file '
+                                 'and .dmg filename contains bzr revision number.')
+        parser.add_argument('-d', '--documentation', metavar='DOCS', dest='docs',
+                            help='Specify the path to the documentation branch.')
         parser.add_argument('-c', '--config', metavar='CONFIG', dest='config',
-            help='Specify the path to the configuration file.',
-            default=os.path.abspath(os.path.join('.', 'config.ini.default')))
-        parser.add_argument('-u', '--skip-update', dest='skip_update',
-            action='store_true', default=False,
-            help='Do NOT update the branch before building.')
-        parser.add_argument('-t', '--skip-translations',
-            dest='skip_translations', action='store_true', default=False,
-            help='Do NOT update the language translation files.')
-        parser.add_argument('--transifex',
-            dest='update_translations', action='store_true', default=False,
-            help='Update the language translation from Transifex.')
-        parser.add_argument('--transifex-user',
-            dest='transifex_user', help='Transifex username.')
-        parser.add_argument('--transifex-pass',
-            dest='transifex_pass', help='Transifex password.')
-        parser.add_argument('-v', '--verbose', dest='verbose',
-            action='store_true', default=False,
-            help='Print out additional information.')
+                            help='Specify the path to the configuration file.',
+                            default=os.path.abspath(os.path.join('.', 'config.ini.default')))
+        parser.add_argument('-u', '--skip-update', dest='skip_update', action='store_true', default=False,
+                            help='Do NOT update the branch before building.')
+        parser.add_argument('-t', '--skip-translations', dest='skip_translations', action='store_true', default=False,
+                            help='Do NOT update the language translation files.')
+        parser.add_argument('--transifex', dest='update_translations', action='store_true', default=False,
+                            help='Update the language translation from Transifex.')
+        parser.add_argument('--transifex-user', dest='transifex_user', help='Transifex username.')
+        parser.add_argument('--transifex-pass', dest='transifex_pass', help='Transifex password.')
+        parser.add_argument('-v', '--verbose', dest='verbose', action='store_true', default=False,
+                            help='Print out additional information.')
         self.args = parser.parse_args()
 
     def read_config(self):
@@ -241,10 +233,8 @@
         Read the configuration from the configuration file.
         """
         self.config = ConfigParser(defaults={
-            u'here': self.script_path,
-            u'projects': os.path.abspath(os.path.join(self.script_path,
-                '..', '..')),
-        })
+            'here': self.script_path,
+            'projects': os.path.abspath(os.path.join(self.script_path, '..', '..')), })
         self.config.read(os.path.abspath(self.args.config))
 
     def setup_system_paths(self):
@@ -258,14 +248,13 @@
         """
         Set up the paths to the executables we use.
         """
-        self.sphinx = _which(self.config.get(u'executables', u'sphinx'))
-        self.pyinstaller = os.path.abspath(
-            self.config.get(u'executables', u'pyinstaller'))
-        self.lrelease = _which(self.config.get(u'executables', u'lrelease'))
-        self.diskutil = _which(self.config.get(u'executables', u'diskutil'))
-        self.hdiutil = _which(self.config.get(u'executables', u'hdiutil'))
-        self.osascript = _which(self.config.get(u'executables', u'osascript'))
-        
+        self.sphinx = _which(self.config.get('executables', 'sphinx'))
+        self.pyinstaller = os.path.abspath(self.config.get('executables', 'pyinstaller'))
+        self.lrelease = _which(self.config.get('executables', 'lrelease'))
+        self.diskutil = _which(self.config.get('executables', 'diskutil'))
+        self.hdiutil = _which(self.config.get('executables', 'hdiutil'))
+        self.osascript = _which(self.config.get('executables', 'osascript'))
+
     def setup_paths(self):
         """
         Set up a variety of paths that we use throughout the build process.
@@ -273,168 +262,172 @@
         if self.args.branch:
             self.branch_path = os.path.abspath(self.args.branch)
         else:
-            self.branch_path = self.config.get(u'paths', u'branch')
+            self.branch_path = self.config.get('paths', 'branch')
         if self.args.docs:
             self.docs_path = os.path.abspath(self.args.docs)
         else:
-            self.docs_path = self.config.get(u'paths', u'documentation')
+            self.docs_path = self.config.get('paths', 'documentation')
 
-        self.openlp_script = os.path.abspath(
-            os.path.join(self.branch_path, u'openlp.py'))
-        self.hooks_path = os.path.abspath(os.path.join(
-            self.branch_path, self.config.get(u'paths', u'hooks')))
-        self.mac_icon = os.path.abspath(
-            self.config.get(u'paths', u'macicon'))
-        self.bundle_info = os.path.abspath(
-            self.config.get(u'paths', u'bundleinfo'))
-        self.dmg_background_img = os.path.abspath(
-            self.config.get(u'paths', u'dmg_background'))
-        self.i18n_utils = os.path.join(self.branch_path, u'scripts',
-            u'translation_utils.py')
-        self.source_path = os.path.join(self.branch_path, u'openlp')
-        self.manual_path = os.path.join(self.docs_path, u'manual')
-        self.manual_build_path = os.path.join(self.manual_path, u'build')
-        self.i18n_path = os.path.join(self.branch_path, u'resources', u'i18n')
-        self.build_path = os.path.join(self.branch_path, u'build')
-        self.dist_app_path = os.path.join(self.branch_path, u'dist', u'OpenLP.app')
-        self.dist_path = os.path.join(self.branch_path, u'dist', u'OpenLP.app',
-                'Contents', 'MacOS')
+        self.openlp_script = os.path.abspath(os.path.join(self.branch_path, 'openlp.py'))
+        self.hooks_path = os.path.abspath(os.path.join(self.branch_path, self.config.get('paths', 'hooks')))
+        self.mac_icon = os.path.abspath(self.config.get('paths', 'macicon'))
+        self.bundle_info = os.path.abspath(self.config.get('paths', 'bundleinfo'))
+        self.dmg_background_img = os.path.abspath(self.config.get('paths', 'dmg_background'))
+        self.i18n_utils = os.path.join(self.branch_path, 'scripts', 'translation_utils.py')
+        self.source_path = os.path.join(self.branch_path, 'openlp')
+        self.manual_path = os.path.join(self.docs_path, 'manual')
+        self.manual_build_path = os.path.join(self.manual_path, 'build')
+        self.i18n_path = os.path.join(self.branch_path, 'resources', 'i18n')
+        self.build_path = os.path.join(self.branch_path, 'build')
+        self.dist_app_path = os.path.join(self.branch_path, 'dist', 'OpenLP.app')
+        self.dist_path = os.path.join(self.branch_path, 'dist', 'OpenLP.app', 'Contents', 'MacOS')
 
         # Path to Qt translation files.
         from PyQt4.QtCore import QCoreApplication
+
         qt_plug_dir = str(list(QCoreApplication.libraryPaths())[0])
-        self.qt_translat_path = os.path.join(os.path.dirname(qt_plug_dir),
-            'translations')
+        self.qt_translat_path = os.path.join(os.path.dirname(qt_plug_dir), 'translations')
 
     def update_code(self):
         """
         Update the code in the branch.
         """
         os.chdir(self.branch_path)
-        self._print(u'Reverting any changes to the code...')
-        bzr = Popen((u'bzr', u'revert'), stdout=PIPE)
-        output = bzr.communicate()[0]
-        code = bzr.wait()
-        if code != 0:
-            self._print(output)
-            raise Exception(u'Error reverting the code')
-        self._print(u'Updating the code...')
-        bzr = Popen((u'bzr', u'update'), stdout=PIPE)
-        output = bzr.communicate()[0]
-        code = bzr.wait()
-        if code != 0:
-            self._print(output)
-            raise Exception(u'Error updating the code')
+        self._print('Reverting any changes to the code...')
+        bzr = Popen(('bzr', 'revert'), stdout=PIPE)
+        output = bzr.communicate()[0]
+        code = bzr.wait()
+        if code != 0:
+            self._print(output)
+            raise Exception('Error reverting the code')
+        self._print('Updating the code...')
+        bzr = Popen(('bzr', 'update'), stdout=PIPE)
+        output = bzr.communicate()[0]
+        code = bzr.wait()
+        if code != 0:
+            self._print(output)
+            raise Exception('Error updating the code')
 
     def run_pyinstaller(self):
         """
         Run PyInstaller on the branch to build an executable.
         """
-        self._print(u'Running PyInstaller...')
+        self._print('Running PyInstaller...')
         os.chdir(self.branch_path)
         pyinstaller = Popen((self.python,
-            self.pyinstaller,
-            u'--noconfirm',
-            u'--windowed',
-            u'--noupx',
-            u'--additional-hooks-dir', self.hooks_path,
-            u'--runtime-hook', os.path.join(self.hooks_path, 'rthook_openlp_pyqt4.py'),
-            u'--log-level=ERROR',
-            u'-o', self.branch_path,
-            #u'-i', self.mac_icon,
-            u'-p', self.branch_path,
-            u'-n', u'OpenLP',
-            self.openlp_script),
-            stdout=PIPE)
+                             self.pyinstaller,
+                             '--noconfirm',
+                             '--windowed',
+                             '--noupx',
+                             '--additional-hooks-dir', self.hooks_path,
+                             '--runtime-hook', os.path.join(self.hooks_path, 'rthook_openlp_pyqt4.py'),
+                             '--log-level=ERROR',
+                             # '--distpath', self.branch_path,
+                             # '-i', self.mac_icon,
+                             '-p', self.branch_path,
+                             '-n', 'OpenLP',
+                             self.openlp_script),
+                            stdout=PIPE)
         output = pyinstaller.communicate()[0]
         code = pyinstaller.wait()
         if code != 0:
             self._print(output)
-            raise Exception(u'Error running PyInstaller')
+            raise Exception('Error running PyInstaller')
 
     def write_version_file(self):
         """
         Write the version number to a file for reading once installed.
         """
-        self._print(u'Writing version file...')
+        self._print('Writing version file...')
         os.chdir(self.branch_path)
-        bzr = Popen((u'bzr', u'tags', u'--sort', u'time'), stdout=PIPE)
+        bzr = Popen(('bzr', 'tags', '--sort', 'time'), stdout=PIPE)
         output = bzr.communicate()[0]
         code = bzr.wait()
         if code != 0:
-            raise Exception(u'Error running bzr tags')
+            raise Exception('Error running bzr tags')
         lines = output.splitlines()
         if len(lines) == 0:
-            tag = u'0.0.0'
-            revision = u'0'
+            tag = '0.0.0'
+            revision = '0'
         else:
-            tag, revision = lines[-1].split()
-        bzr = Popen((u'bzr', u'log', u'--line', u'-r', u'-1'), stdout=PIPE)
+            tag, revision = lines[-1].decode('utf-8').split()
+        bzr = Popen(('bzr', 'log', '--line', '-r', '-1'), stdout=PIPE)
         output, error = bzr.communicate()
         code = bzr.wait()
         if code != 0:
-            raise Exception(u'Error running bzr log')
-        output_ascii = unicode(output, errors=u'ignore')
-        latest = output_ascii.split(u':')[0]
-        self.version_string = u'%s-bzr%s' % (tag, latest)
+            raise Exception('Error running bzr log')
+        latest = output.decode('utf-8').split(':')[0]
+        self.version_string = '%s-bzr%s' % (tag, latest)
         self.version_tag = tag
-        version_file = open(os.path.join(self.dist_path, u'.version'), u'w')
+        version_file = open(os.path.join(self.dist_path, '.version'), 'w')
         # Release version does not contain revision in .dmg name.
         if self.args.devel:
-            version_file.write(self.version_string)
+            version_file.write(str(self.version_string))
         else:
-            version_file.write(self.version_tag)
+            version_file.write(str(self.version_tag))
         version_file.close()
 
+    def copy_default_theme(self):
+        """
+        Copy the default theme to the correct directory for OpenLP.
+        """
+        self._print('Copying default theme...')
+        source = os.path.join(self.source_path, 'core', 'lib', 'json')
+        dest = os.path.join(self.dist_path, 'core', 'lib', 'json')
+        for root, dirs, files in os.walk(source):
+            for filename in files:
+                if filename.endswith('.json'):
+                    dest_path = os.path.join(dest, root[len(source) + 1:])
+                    if not os.path.exists(dest_path):
+                        os.makedirs(dest_path)
+                    self._print_verbose('... %s', filename)
+                    copy(os.path.join(root, filename), os.path.join(dest_path, filename))
+
     def copy_plugins(self):
         """
         Copy all the plugins to the correct directory so that OpenLP sees that
         it has plugins.
         """
-        self._print(u'Copying plugins...')
-        source = os.path.join(self.source_path, u'plugins')
-        dest = os.path.join(self.dist_path, u'plugins')
+        self._print('Copying plugins...')
+        source = os.path.join(self.source_path, 'plugins')
+        dest = os.path.join(self.dist_path, 'plugins')
         for root, dirs, files in os.walk(source):
             for filename in files:
-                if not filename.endswith(u'.pyc'):
-                    dest_path = os.path.join(dest, root[len(source)+1:])
+                if not filename.endswith('.pyc'):
+                    dest_path = os.path.join(dest, root[len(source) + 1:])
                     if not os.path.exists(dest_path):
                         os.makedirs(dest_path)
-                    self._print_verbose(u'... %s', filename)
-                    copy(os.path.join(root, filename),
-                        os.path.join(dest_path, filename))
+                    self._print_verbose('... %s', filename)
+                    copy(os.path.join(root, filename), os.path.join(dest_path, filename))
 
     def copy_media_player(self):
         """
         Copy the media players to the correct directory for OpenLP.
         """
-        self._print(u'Copying media player...')
-        source = os.path.join(self.source_path, u'core', u'ui', u'media')
-        dest = os.path.join(self.dist_path, u'core', u'ui', u'media')
+        self._print('Copying media player...')
+        source = os.path.join(self.source_path, 'core', 'ui', 'media')
+        dest = os.path.join(self.dist_path, 'core', 'ui', 'media')
         for root, dirs, files in os.walk(source):
             for filename in files:
-                if not filename.endswith(u'.pyc'):
-                    dest_path = os.path.join(dest, root[len(source)+1:])
+                if not filename.endswith('.pyc'):
+                    dest_path = os.path.join(dest, root[len(source) + 1:])
                     if not os.path.exists(dest_path):
                         os.makedirs(dest_path)
-                    self._print_verbose(u'... %s', filename)
-                    copy(os.path.join(root, filename),
-                        os.path.join(dest_path, filename))
+                    self._print_verbose('... %s', filename)
+                    copy(os.path.join(root, filename), os.path.join(dest_path, filename))
 
     def copy_mac_bundle_files(self):
         """
         Copy Info.plist and OpenLP.icns to app bundle.
         """
-        copy(self.mac_icon, os.path.join(self.dist_app_path, 
-            'Contents', 'Resources', os.path.basename(self.mac_icon)))
+        copy(self.mac_icon, os.path.join(self.dist_app_path, 'Contents', 'Resources', os.path.basename(self.mac_icon)))
         # Add OpenLP version to Info.plist and put it to app bundle.
-        fr = open(self.bundle_info, u'r')
-        fw = open(os.path.join(self.dist_app_path, 
-            'Contents', os.path.basename(self.bundle_info)), 'w')
+        fr = open(self.bundle_info, 'r')
+        fw = open(os.path.join(self.dist_app_path, 'Contents', os.path.basename(self.bundle_info)), 'w')
         text = fr.read()
         text = text % {'openlp_version': self.version_string}
         fw.write(text)
-        
+
         fr.close()
         fw.close()
 
@@ -442,118 +435,104 @@
         """
         Copy all the OSX-specific files.
         """
-        self._print(u'Copying extra files for Mac OS X...')
-        self._print_verbose(u'... LICENSE.txt')
-        copy(os.path.join(self.script_path, u'LICENSE.txt'),
-            os.path.join(self.dist_path, u'LICENSE.txt'))
+        self._print('Copying extra files for Mac OS X...')
+        self._print_verbose('... LICENSE.txt')
+        copy(os.path.join(self.script_path, 'LICENSE.txt'), os.path.join(self.dist_path, 'LICENSE.txt'))
 
     def update_translations(self):
         """
         Update the translations.
         """
-        self._print(u'Updating translations...')
+        self._print('Updating translations...')
         if not self.config.has_section('transifex'):
-            raise Exception(u'No section named "transifex" found.')
+            raise Exception('No section named "transifex" found.')
         if not self.config.has_option('transifex', 'username'):
-            raise Exception(u'No option named "username" found.')
+            raise Exception('No option named "username" found.')
         if not self.config.has_option('transifex', 'password'):
-            raise Exception(u'No option named "password" found.')
+            raise Exception('No option named "password" found.')
         if self.args.transifex_user:
             username = self.args.transifex_user
         else:
-            username = self.config.get(u'transifex', u'username')
+            username = self.config.get('transifex', 'username')
         if self.args.transifex_pass:
             password = self.args.transifex_pass
         else:
-            password = self.config.get(u'transifex', u'password')
+            password = self.config.get('transifex', 'password')
         os.chdir(os.path.split(self.i18n_utils)[0])
-        translation_utils = Popen([self.python, self.i18n_utils, u'-qdpu',
-            u'-U', username, u'-P', password])
+        translation_utils = Popen([self.python, self.i18n_utils, '-qdpu', '-U', username, '-P', password])
         code = translation_utils.wait()
         if code != 0:
-            raise Exception(u'Error running translation_utils.py')
+            raise Exception('Error running translation_utils.py')
 
     def compile_translations(self):
         """
         Compile the translations for Qt.
         """
-        self._print(u'Compiling translations...')
+        self._print('Compiling translations...')
         files = os.listdir(self.i18n_path)
-        if not os.path.exists(os.path.join(self.dist_path, u'i18n')):
-            os.makedirs(os.path.join(self.dist_path, u'i18n'))
+        if not os.path.exists(os.path.join(self.dist_path, 'i18n')):
+            os.makedirs(os.path.join(self.dist_path, 'i18n'))
         for file in files:
-            if file.endswith(u'.ts'):
-                self._print_verbose(u'... %s', file)
+            if file.endswith('.ts'):
+                self._print_verbose('... %s', file)
                 source_path = os.path.join(self.i18n_path, file)
-                dest_path = os.path.join(self.dist_path, u'i18n',
-                    file.replace(u'.ts', u'.qm'))
-                lconvert = Popen((self.lrelease, u'-compress', u'-silent',
-                    source_path, u'-qm', dest_path))
+                dest_path = os.path.join(self.dist_path, 'i18n', file.replace('.ts', '.qm'))
+                lconvert = Popen((self.lrelease, '-compress', '-silent', source_path, '-qm', dest_path))
                 code = lconvert.wait()
                 if code != 0:
-                    raise Exception(u'Error running lconvert on %s' % \
-                        source_path)
-        self._print(u'Copying qm files...')
+                    raise Exception('Error running lconvert on %s' % source_path)
+        self._print('Copying qm files...')
         source = self.qt_translat_path
         files = os.listdir(source)
         for filename in files:
-            if filename.startswith(u'qt_') and filename.endswith(u'.qm') and \
-                len(filename) == 8:
-                self._print_verbose(u'... %s', filename)
-                copy(os.path.join(source, filename),
-                    os.path.join(self.dist_path, u'i18n', filename))
+            if filename.startswith('qt_') and filename.endswith('.qm') and len(filename) == 8:
+                self._print_verbose('... %s', filename)
+                copy(os.path.join(source, filename), os.path.join(self.dist_path, 'i18n', filename))
 
     def run_sphinx(self):
         """
         Run Sphinx to build an HTML Help project.
         """
-        self._print(u'Deleting previous manual build... %s',
-            self.manual_build_path)
+        self._print('Deleting previous manual build... %s', self.manual_build_path)
         if os.path.exists(self.manual_build_path):
             rmtree(self.manual_build_path)
-        self._print(u'Running Sphinx...')
+        self._print('Running Sphinx...')
         os.chdir(self.manual_path)
-        sphinx = Popen((self.sphinx, u'-b', u'htmlhelp', u'-d',
-            u'build/doctrees', u'source', u'build/htmlhelp'), stdout=PIPE)
+        sphinx = Popen((self.sphinx, '-b', 'htmlhelp', '-d', 'build/doctrees', 'source', 'build/htmlhelp'), stdout=PIPE)
         output, error = sphinx.communicate()
         code = sphinx.wait()
         if code != 0:
             self._print(output)
-            raise Exception(u'Error running Sphinx')
+            raise Exception('Error running Sphinx')
 
     def create_dmg_file(self):
         """
         Create .dmg file.
         """
-        self._print(u'Creating dmg file...')
+        self._print('Creating dmg file...')
 
         # Release version does not contain revision in .dmg name.
         if self.args.devel:
-            dmg_name = 'OpenLP-' + self.version_string + '.dmg'
+            dmg_name = 'OpenLP-' + str(self.version_string) + '.dmg'
         else:
-            dmg_name = 'OpenLP-' + self.version_tag + '.dmg'
+            dmg_name = 'OpenLP-' + str(self.version_tag) + '.dmg'
 
         dmg_file = os.path.join(self.branch_path, 'build', dmg_name)
         # Remove dmg if it exists.
         if os.path.exists(dmg_file):
             os.remove(dmg_file)
-        # Create empty dmg file. 
+        # Create empty dmg file.
         size = self._get_directory_size(self.dist_app_path)  # in bytes.
         size = size / (1024 * 1024)  # Convert to megabytes.
         size += 10  # Additional space in .dmg for other files.
-        self._print(u'... dmg disk size: %s' % size)
-        self._run_command([self.hdiutil, 'create', dmg_file, 
-            '-ov', '-megabytes', str(size), 
-            '-fs', 'HFS+', '-volname', 'OpenLP'],
-            u'Could not create dmg file.'
-        )
+        self._print('... dmg disk size: %s' % size)
+        self._run_command([self.hdiutil, 'create', dmg_file, '-ov', '-megabytes', str(size), '-fs', 'HFS+', '-volname',
+                           'OpenLP'], 'Could not create dmg file.')
 
         # Mount empty dmg file.
         old_mounts = self._get_mountpoints()
-        self._print(u'... mounting the dmg file: %s' % dmg_file)
-        self._run_command([self.hdiutil, 'attach', dmg_file],
-            u'Could not mount dmg file, cannot continue.'
-        )
+        self._print('... mounting the dmg file: %s' % dmg_file)
+        self._run_command([self.hdiutil, 'attach', dmg_file], 'Could not mount dmg file, cannot continue.')
         new_mounts = self._get_mountpoints()
         # Get the mount point from difference between paths
         # after mounting and before mounting the dmg file.
@@ -561,25 +540,19 @@
 
         # Copy OpenLP.app and other files to .dmg
         # TODO more reliable way to determine dmg_volume_path
-        self._print(u'... Copying the app to the dmg: ' + dmg_volume_path)
-        self._run_command(['cp', '-r', self.dist_app_path,
-            dmg_volume_path],
-            u'Could not copy app bundle, dmg creation failed.'
-        )
+        self._print('... Copying the app to the dmg: ' + dmg_volume_path)
+        self._run_command(['cp', '-r', self.dist_app_path, dmg_volume_path],
+                          'Could not copy app bundle, dmg creation failed.')
 
         # Set icon for dmg file.
         # http://endrift.com/blog/2010/06/14/dmg-files-volume-icons-cli/
         self._print('... Setting the dmg icon.')
         dmg_icon = os.path.join(dmg_volume_path, '.VolumeIcon.icns')
-        self._run_command(['cp', self.mac_icon, dmg_icon],
-            u'Could not copy the dmg icon file, dmg creation failed.'
-        )
+        self._run_command(['cp', self.mac_icon, dmg_icon], 'Could not copy the dmg icon file, dmg creation failed.')
         # Set proper dmg icon attributes.
-        self._run_command(['SetFile', '-c', 'icnC', dmg_icon],
-                'Could not set dmg icon attributes.')
+        self._run_command(['SetFile', '-c', 'icnC', dmg_icon], 'Could not set dmg icon attributes.')
         # Ensures dmg icon will be used while mounted.
-        self._run_command(['SetFile', '-a', 'C', dmg_volume_path],
-                'Could not set dmg icon attributes.')
+        self._run_command(['SetFile', '-a', 'C', dmg_volume_path], 'Could not set dmg icon attributes.')
 
         # Create symlink in dmg pointing to the /Applications directory on OS X.
         self._print('... Creating symlink to /Applications.')
@@ -591,15 +564,13 @@
             self._print('... Setting the background image.')
 
             os.mkdir(os.path.join(dmg_volume_path, '.background'))
-            self._run_command(['cp', self.dmg_background_img,
-                os.path.join(dmg_volume_path,
-                '.background/installer-background.png')],
-                u'Could not copy the background image, dmg creation failed.'
-            )
+            self._run_command(['cp', self.dmg_background_img, os.path.join(dmg_volume_path,
+                                                                           '.background/installer-background.png')],
+                              'Could not copy the background image, dmg creation failed.')
 
             self.adjust_dmg_view(os.path.basename(dmg_volume_path))
 
-        ## Unmount dmg file.
+        # Unmount dmg file.
         self._print('... unmounting the dmg.')
         # Sometimes it could happen that OSX Finder is blocking umount.
         # We need to find this process and kill it.
@@ -609,50 +580,42 @@
                 blocking_proc_pid = int(output.split()[0])
                 os.kill(int(blocking_proc_pid), signal.SIGKILL)
         except Exception as e:
-            print str(e)
+            print(str(e))
             self._print('... failed to kill process using %s' % dmg_volume_path)
         # Unmount dmg file.
         self._run_command([self.hdiutil, 'detach', dmg_volume_path],
-            'Could not unmount the dmg file, dmg creation failed.'
-        )
+                          'Could not unmount the dmg file, dmg creation failed.')
 
         # Compress dmg file.
         self._print('... compressing the dmg file')
-        compressed_dmg = os.path.join(self.branch_path, 'dist',
-            os.path.basename(dmg_file))  # Put dmg to 'dist' dir.
+        compressed_dmg = os.path.join(self.branch_path, 'dist', os.path.basename(dmg_file))  # Put dmg to 'dist' dir.
         # Remove dmg if it exists.
         if os.path.exists(compressed_dmg):
             os.remove(compressed_dmg)
-        self._run_command([self.hdiutil, 
-            'convert', dmg_file, '-format', 'UDZO',
-            '-imagekey', 'zlib-level=9', '-o', compressed_dmg],
-            u'Could not compress the dmg file, dmg creation failed.'
-        )
+        self._run_command([self.hdiutil, 'convert', dmg_file, '-format', 'UDZO', '-imagekey', 'zlib-level=9', '-o',
+                           compressed_dmg], 'Could not compress the dmg file, dmg creation failed.')
 
-        # Jenkins integration. 
+        # Jenkins integration.
         # Continuous integration server needs to know the filename of dmg.
         # Write java property file. For uploading dmg to openlp.
         if self.args.devel:
             fpath = os.path.join(self.branch_path, 'openlp.properties')
-            self._print('... writing property file for jenkins: %s' %
-                fpath)
+            self._print('... writing property file for jenkins: %s' % fpath)
             f = open(fpath, 'w')
             f.write('OPENLP_DMGNAME=' + os.path.basename(dmg_file) + '\n')
             f.close()
 
         # Dmg done.
-        self._print('Finished creating dmg file, resulting file: %s' %
-            compressed_dmg)
+        self._print('Finished creating dmg file, resulting file: %s' % compressed_dmg)
 
         self.dmg_file = compressed_dmg
 
     def adjust_dmg_view(self, dmg_volume_name):
         try:
             # TODO: Use only one applescript file. Remove one for osx 10.5.
-            f = open(os.path.join(self.script_path,
-                'applescript-adjust-dmg-view.master'))
+            f = open(os.path.join(self.script_path, 'applescript-adjust-dmg-view.master'))
             p = Popen([self.osascript], stdin=PIPE)
-            p.communicate(f.read() % (dmg_volume_name, 'OpenLP'))
+            p.communicate(bytes(f.read() % (dmg_volume_name, 'OpenLP'), 'utf-8'))
             f.close()
             result = p.returncode
             if (result != 0):
@@ -664,33 +627,32 @@
         """
         The main function to run the Mac OS X builder.
         """
-        self._print_verbose(u'OpenLP main script: ......%s',
-            self.openlp_script)
-        self._print_verbose(u'Script path: .............%s',
-            os.path.split(os.path.abspath(__file__))[0])
-        self._print_verbose(u'Branch path: .............%s', self.branch_path)
-        self._print_verbose(u'Source path: .............%s', self.source_path)
-        self._print_verbose(u'"dist.app" path: .........%s', self.dist_app_path)
-        self._print_verbose(u'"dist" path: .............%s', self.dist_path)
-        self._print_verbose(u'"hooks" path: ............%s', self.hooks_path)
-        self._print_verbose(u'PyInstaller: .............%s', self.pyinstaller)
-        self._print_verbose(u'Documentation branch path:%s', self.docs_path)
-        self._print_verbose(u'')
+        self._print_verbose('OpenLP main script: ......%s', self.openlp_script)
+        self._print_verbose('Script path: .............%s', os.path.split(os.path.abspath(__file__))[0])
+        self._print_verbose('Branch path: .............%s', self.branch_path)
+        self._print_verbose('Source path: .............%s', self.source_path)
+        self._print_verbose('"dist.app" path: .........%s', self.dist_app_path)
+        self._print_verbose('"dist" path: .............%s', self.dist_path)
+        self._print_verbose('"hooks" path: ............%s', self.hooks_path)
+        self._print_verbose('PyInstaller: .............%s', self.pyinstaller)
+        self._print_verbose('Documentation branch path:%s', self.docs_path)
+        self._print_verbose('')
         if not self.args.skip_update:
             self.update_code()
         self.run_pyinstaller()
         self.write_version_file()
         self.copy_mac_bundle_files()
+        self.copy_default_theme()
         self.copy_plugins()
         self.copy_media_player()
         # TODO creating help on Mac
         if os.path.exists(self.manual_path):
             self.run_sphinx()
         else:
-            self._print(u'')
-            self._print(u'WARNING: Documentation trunk not found. Mac OS X')
-            self._print(u'         Help file will not be included in build')
-            self._print(u'')
+            self._print('')
+            self._print('WARNING: Documentation trunk not found. Mac OS X')
+            self._print('         Help file will not be included in build')
+            self._print('')
         self.copy_macosx_files()
         if not self.args.skip_translations:
             if self.args.update_translations:
@@ -698,9 +660,9 @@
             self.compile_translations()
         self.create_dmg_file()
 
-        self._print(u'Done.')
+        self._print('Done.')
         raise SystemExit()
 
 
-if __name__ == u'__main__':
+if __name__ == '__main__':
     MacosxBuilder().main()

=== modified file 'pyinstaller-hooks/rthook_openlp_pyqt4.py'
--- pyinstaller-hooks/rthook_openlp_pyqt4.py	2013-01-01 01:04:23 +0000
+++ pyinstaller-hooks/rthook_openlp_pyqt4.py	2014-05-06 19:57:38 +0000
@@ -41,10 +41,10 @@
 
 import sip
 
-sip.setapi(u'QDate', 2)
-sip.setapi(u'QDateTime', 2)
-sip.setapi(u'QString', 2)
-sip.setapi(u'QTextStream', 2)
-sip.setapi(u'QTime', 2)
-sip.setapi(u'QUrl', 2)
-sip.setapi(u'QVariant', 2)
+sip.setapi('QDate', 2)
+sip.setapi('QDateTime', 2)
+sip.setapi('QString', 2)
+sip.setapi('QTextStream', 2)
+sip.setapi('QTime', 2)
+sip.setapi('QUrl', 2)
+sip.setapi('QVariant', 2)

=== modified file 'windows/windows-builder.py'
--- windows/windows-builder.py	2013-02-03 14:56:42 +0000
+++ windows/windows-builder.py	2014-05-06 19:57:38 +0000
@@ -129,7 +129,7 @@
 from shutil import copy, rmtree, move
 from distutils import dir_util
 from subprocess import Popen, PIPE
-from ConfigParser import SafeConfigParser as ConfigParser
+from configparser import ConfigParser
 from argparse import ArgumentParser
 
 
@@ -138,13 +138,14 @@
     The :class:`WindowsBuilder` class encapsulates everything that is needed
     to build a Windows installer.
     """
+
     def __init__(self):
         self.setup_args()
         self.setup_system_paths()
         self.read_config()
         self.setup_executables()
         self.setup_paths()
-        self.version = u''
+        self.version = ''
 
     def _print(self, text, *args):
         """
@@ -152,7 +153,7 @@
         """
         if len(args) > 0:
             text = text % tuple(args)
-        print text
+        print(text)
 
     def _print_verbose(self, text, *args):
         """
@@ -167,26 +168,20 @@
         """
         parser = ArgumentParser()
         parser.add_argument('-b', '--branch', metavar='BRANCH', dest='branch',
-            help='Specify the path to the branch you wish to build.',
-            default=None)
-        parser.add_argument('-d', '--documentation', metavar='DOCS',
-            dest='docs', default=None,
-            help='Specify the path to the documentation branch.')
+                            help='Specify the path to the branch you wish to build.', default=None)
+        parser.add_argument('-d', '--documentation', metavar='DOCS', dest='docs', default=None,
+                            help='Specify the path to the documentation branch.')
         parser.add_argument('-c', '--config', metavar='CONFIG', dest='config',
-            help='Specify the path to the configuration file.',
-            default=os.path.abspath(os.path.join('.', 'config.ini')))
-        parser.add_argument('-u', '--skip-update', dest='skip_update',
-            action='store_true', default=False,
-            help='Do NOT update the branch before building.')
-        parser.add_argument('-p', '--portable', metavar='PORTABLE',
-            dest='portable', default=None,
-            help='Specify the path to build the portable installation.')
-        parser.add_argument('-t', '--skip-translations',
-            dest='skip_translations', action='store_true', default=False,
-            help='Do NOT update the language translation files.')
-        parser.add_argument('-v', '--verbose', dest='verbose',
-            action='store_true', default=False,
-            help='Print out additional information.')
+                            help='Specify the path to the configuration file.',
+                            default=os.path.abspath(os.path.join('.', 'config.ini')))
+        parser.add_argument('-u', '--skip-update', dest='skip_update', action='store_true', default=False,
+                            help='Do NOT update the branch before building.')
+        parser.add_argument('-p', '--portable', metavar='PORTABLE', dest='portable', default=None,
+                            help='Specify the path to build the portable installation.')
+        parser.add_argument('-t', '--skip-translations', dest='skip_translations', action='store_true', default=False,
+                            help='Do NOT update the language translation files.')
+        parser.add_argument('-v', '--verbose', dest='verbose', action='store_true', default=False,
+                            help='Print out additional information.')
         self.args = parser.parse_args()
 
     def read_config(self):
@@ -194,10 +189,10 @@
         Read the configuration from the configuration file.
         """
         self.config = ConfigParser(defaults={
-            u'pyroot': self.python_root,
-            u'progfiles': self.program_files,
-            u'sitepackages': self.site_packages,
-            u'here': self.script_path
+            'pyroot': self.python_root,
+            'progfiles': self.program_files,
+            'sitepackages': self.site_packages,
+            'here': self.script_path
         })
         self.config.read(os.path.abspath(self.args.config))
 
@@ -208,40 +203,29 @@
         self.script_path = os.path.split(os.path.abspath(__file__))[0]
         self.python = sys.executable
         self.python_root = os.path.split(self.python)[0]
-        self.site_packages = os.path.join(self.python_root,
-            u'Lib', u'site-packages')
-        self.program_files = os.getenv(u'PROGRAMFILES')
+        self.site_packages = os.path.join(self.python_root, 'Lib', 'site-packages')
+        self.program_files = os.getenv('PROGRAMFILES')
 
     def setup_executables(self):
         """
         Set up the paths to the executables we use.
         """
-        self.innosetup = os.path.abspath(
-            self.config.get(u'executables', u'innosetup'))
-        self.sphinx = os.path.abspath(
-            self.config.get(u'executables', u'sphinx'))
-        self.pyinstaller = os.path.abspath(
-            self.config.get(u'executables', u'pyinstaller'))
-        self.vcbuild = os.path.abspath(
-            self.config.get(u'executables', u'vcbuild'))
-        self.hhc = os.path.abspath(
-            self.config.get(u'executables', u'htmlhelp'))
-        self.psvince = os.path.abspath(
-            self.config.get(u'executables', u'psvince'))
-        self.portableinstaller = os.path.abspath(
-            self.config.get(u'executables', u'portableinstaller'))
-        self.portablelauncher = os.path.abspath(
-            self.config.get(u'executables', u'portablelauncher'))
-        if os.path.exists(os.path.join(self.site_packages, u'PyQt4', u'bin')):
+        self.innosetup = os.path.abspath(self.config.get('executables', 'innosetup'))
+        self.sphinx = os.path.abspath(self.config.get('executables', 'sphinx'))
+        self.pyinstaller = os.path.abspath(self.config.get('executables', 'pyinstaller'))
+        self.vcbuild = os.path.abspath(self.config.get('executables', 'vcbuild'))
+        self.hhc = os.path.abspath(self.config.get('executables', 'htmlhelp'))
+        self.psvince = os.path.abspath(self.config.get('executables', 'psvince'))
+        self.portableinstaller = os.path.abspath(self.config.get('executables', 'portableinstaller'))
+        self.portablelauncher = os.path.abspath(self.config.get('executables', 'portablelauncher'))
+        if os.path.exists(os.path.join(self.site_packages, 'PyQt4', 'bin')):
             # Older versions of the PyQt4 Windows installer put their binaries
             # in the "bin" directory
-            self.lrelease = os.path.join(self.site_packages, u'PyQt4',
-                u'bin', u'lrelease.exe')
+            self.lrelease = os.path.join(self.site_packages, 'PyQt4', 'bin', 'lrelease.exe')
         else:
             # Newer versions of the PyQt4 Windows installer put their binaries
             # in the base directory of the installation
-            self.lrelease = os.path.join(self.site_packages, u'PyQt4',
-                u'lrelease.exe')
+            self.lrelease = os.path.join(self.site_packages, 'PyQt4', 'lrelease.exe')
 
     def setup_paths(self):
         """
@@ -250,270 +234,263 @@
         if self.args.branch:
             branch_path = self.args.branch
         else:
-            branch_path = self.config.get(u'paths', u'branch')
+            branch_path = self.config.get('paths', 'branch')
         self.branch_path = os.path.abspath(branch_path)
         if self.args.docs:
             docs_path = self.args.docs
         else:
-            docs_path = self.config.get(u'paths', u'documentation')
+            docs_path = self.config.get('paths', 'documentation')
         self.docs_path = os.path.abspath(docs_path)
         if self.args.portable:
             portable_path = self.args.portable
         else:
             try:
-                portable_path = self.config.get(u'paths', u'portable')
+                portable_path = self.config.get('paths', 'portable')
             except:
-                portable_path = u''
+                portable_path = ''
         if portable_path:
             self.portable_path = os.path.abspath(portable_path)
             self.args.portable = self.portable_path
         else:
-            self.portable_path = u''
-        self.openlp_script = os.path.abspath(
-            os.path.join(branch_path, u'openlp.py'))
-        self.hooks_path = os.path.abspath(self.config.get(u'paths', u'hooks'))
-        self.win32_icon = os.path.abspath(
-            self.config.get(u'paths', u'win32icon'))
-        self.i18n_utils = os.path.join(self.branch_path, u'scripts',
-            u'translation_utils.py')
-        self.source_path = os.path.join(self.branch_path, u'openlp')
-        self.manual_path = os.path.join(self.docs_path, u'manual')
-        self.manual_build_path = os.path.join(self.manual_path, u'build')
-        self.helpfile_path = os.path.join(self.manual_build_path, u'htmlhelp')
-        self.i18n_path = os.path.join(self.branch_path, u'resources', u'i18n')
-        self.winres_path = os.path.join(self.branch_path, u'resources',
-            u'windows')
-        self.build_path = os.path.join(self.branch_path, u'build')
-        self.dist_path = os.path.join(self.branch_path, u'dist', u'OpenLP')
-        self.pptviewlib_path = os.path.join(self.source_path, u'plugins',
-            u'presentations', u'lib', u'pptviewlib')
+            self.portable_path = ''
+        self.openlp_script = os.path.abspath(os.path.join(branch_path, 'openlp.py'))
+        self.hooks_path = os.path.abspath(self.config.get('paths', 'hooks'))
+        self.win32_icon = os.path.abspath(self.config.get('paths', 'win32icon'))
+        self.i18n_utils = os.path.join(self.branch_path, 'scripts', 'translation_utils.py')
+        self.source_path = os.path.join(self.branch_path, 'openlp')
+        self.manual_path = os.path.join(self.docs_path, 'manual')
+        self.manual_build_path = os.path.join(self.manual_path, 'build')
+        self.helpfile_path = os.path.join(self.manual_build_path, 'htmlhelp')
+        self.i18n_path = os.path.join(self.branch_path, 'resources', 'i18n')
+        self.winres_path = os.path.join(self.branch_path, 'resources', 'windows')
+        self.build_path = os.path.join(self.branch_path, 'build')
+        self.dist_path = os.path.join(self.branch_path, 'dist', 'OpenLP')
+        self.pptviewlib_path = os.path.join(self.source_path, 'plugins', 'presentations', 'lib', 'pptviewlib')
 
     def update_code(self):
         """
         Update the code in the branch.
         """
         os.chdir(self.branch_path)
-        self._print(u'Reverting any changes to the code...')
-        bzr = Popen((u'bzr', u'revert'), stdout=PIPE)
-        output = bzr.communicate()[0]
-        code = bzr.wait()
-        if code != 0:
-            self._print(output)
-            raise Exception(u'Error reverting the code')
-        self._print(u'Updating the code...')
-        bzr = Popen((u'bzr', u'update'), stdout=PIPE)
-        output = bzr.communicate()[0]
-        code = bzr.wait()
-        if code != 0:
-            self._print(output)
-            raise Exception(u'Error updating the code')
+        self._print('Reverting any changes to the code...')
+        bzr = Popen(('bzr', 'revert'), stdout=PIPE)
+        output = bzr.communicate()[0]
+        code = bzr.wait()
+        if code != 0:
+            self._print(output)
+            raise Exception('Error reverting the code')
+        self._print('Updating the code...')
+        bzr = Popen(('bzr', 'update'), stdout=PIPE)
+        output = bzr.communicate()[0]
+        code = bzr.wait()
+        if code != 0:
+            self._print(output)
+            raise Exception('Error updating the code')
 
     def run_pyinstaller(self):
         """
         Run PyInstaller on the branch to build an executable.
         """
-        self._print(u'Running PyInstaller...')
+        self._print('Running PyInstaller...')
         os.chdir(self.branch_path)
         pyinstaller = Popen((self.python, self.pyinstaller,
-            u'--noconfirm',
-            u'--windowed',
-            u'--noupx',
-            u'--additional-hooks-dir', self.hooks_path,
-            u'--runtime-hook', os.path.join(self.hooks_path, 'rthook_openlp_pyqt4.py'),
-            u'--log-level=ERROR',
-            u'-o', self.branch_path,
-            u'-i', self.win32_icon,
-            u'-p', self.branch_path,
-            u'-n', u'OpenLP',
-            self.openlp_script),
-            stdout=PIPE)
+                             '--noconfirm',
+                             '--windowed',
+                             '--noupx',
+                             '--additional-hooks-dir', self.hooks_path,
+                             '--runtime-hook', os.path.join(self.hooks_path, 'rthook_openlp_pyqt4.py'),
+                             '--log-level=ERROR',
+                             '--distpath', self.branch_path,
+                             '-i', self.win32_icon,
+                             '-p', self.branch_path,
+                             '-n', 'OpenLP',
+                             self.openlp_script),
+                            stdout=PIPE)
         output = pyinstaller.communicate()[0]
         code = pyinstaller.wait()
         if code != 0:
             self._print(output)
-            raise Exception(u'Error running PyInstaller')
+            raise Exception('Error running PyInstaller')
 
     def write_version_file(self):
         """
         Write the version number to a file for reading once installed.
         """
-        self._print(u'Writing version file...')
+        self._print('Writing version file...')
         os.chdir(self.branch_path)
-        bzr = Popen((u'bzr', u'tags', u'--sort', u'time'), stdout=PIPE)
+        bzr = Popen(('bzr', 'tags', '--sort', 'time'), stdout=PIPE)
         output = bzr.communicate()[0]
         code = bzr.wait()
         if code != 0:
-            raise Exception(u'Error running bzr tags')
+            raise Exception('Error running bzr tags')
         lines = output.splitlines()
         if len(lines) == 0:
-            tag = u'0.0.0'
-            revision = u'0'
+            tag = '0.0.0'
+            revision = '0'
         else:
-            tag, revision = lines[-1].split()
-        bzr = Popen((u'bzr', u'log', u'--line', u'-r', u'-1'), stdout=PIPE)
+            tag, revision = lines[-1].decode('utf-8').split()
+        bzr = Popen(('bzr', 'log', '--line', '-r', '-1'), stdout=PIPE)
         output, error = bzr.communicate()
         code = bzr.wait()
         if code != 0:
-            raise Exception(u'Error running bzr log')
-        output_ascii = unicode(output, errors=u'ignore')
-        latest = output_ascii.split(u':')[0]
-        version_string = latest == revision and tag or \
-            u'%s-bzr%s' % (tag, latest)
+            raise Exception('Error running bzr log')
+        latest = output.decode('utf-8').split(':')[0]
+        version_string = latest == revision and tag or '%s-bzr%s' % (tag, latest)
         # Save decimal version in case we need to do a portable build.
-        self.version = latest == revision and tag or\
-            u'%s.%s' % (tag, latest)
-        version_file = open(os.path.join(self.dist_path, u'.version'), u'w')
-        version_file.write(version_string)
+        self.version = latest == revision and tag or '%s.%s' % (tag, latest)
+        version_file = open(os.path.join(self.dist_path, '.version'), 'w')
+        version_file.write(str(version_string))
         version_file.close()
 
+    def copy_default_theme(self):
+        """
+        Copy the default theme to the correct directory for OpenLP.
+        """
+        self._print('Copying default theme...')
+        source = os.path.join(self.source_path, 'core', 'lib', 'json')
+        dest = os.path.join(self.dist_path, 'core', 'lib', 'json')
+        for root, dirs, files in os.walk(source):
+            for filename in files:
+                if filename.endswith('.json'):
+                    dest_path = os.path.join(dest, root[len(source) + 1:])
+                    if not os.path.exists(dest_path):
+                        os.makedirs(dest_path)
+                    self._print_verbose('... %s', filename)
+                    copy(os.path.join(root, filename), os.path.join(dest_path, filename))
+
     def copy_plugins(self):
         """
         Copy all the plugins to the correct directory so that OpenLP sees that
         it has plugins.
         """
-        self._print(u'Copying plugins...')
-        source = os.path.join(self.source_path, u'plugins')
-        dest = os.path.join(self.dist_path, u'plugins')
+        self._print('Copying plugins...')
+        source = os.path.join(self.source_path, 'plugins')
+        dest = os.path.join(self.dist_path, 'plugins')
         for root, dirs, files in os.walk(source):
             for filename in files:
-                if not filename.endswith(u'.pyc'):
-                    dest_path = os.path.join(dest, root[len(source)+1:])
+                if not filename.endswith('.pyc'):
+                    dest_path = os.path.join(dest, root[len(source) + 1:])
                     if not os.path.exists(dest_path):
                         os.makedirs(dest_path)
-                    self._print_verbose(u'... %s', filename)
-                    copy(os.path.join(root, filename),
-                        os.path.join(dest_path, filename))
+                    self._print_verbose('... %s', filename)
+                    copy(os.path.join(root, filename), os.path.join(dest_path, filename))
 
     def copy_media_player(self):
         """
         Copy the media players to the correct directory for OpenLP.
         """
-        self._print(u'Copying media player...')
-        source = os.path.join(self.source_path, u'core', u'ui', u'media')
-        dest = os.path.join(self.dist_path, u'core', u'ui', u'media')
+        self._print('Copying media player...')
+        source = os.path.join(self.source_path, 'core', 'ui', 'media')
+        dest = os.path.join(self.dist_path, 'core', 'ui', 'media')
         for root, dirs, files in os.walk(source):
             for filename in files:
-                if not filename.endswith(u'.pyc'):
-                    dest_path = os.path.join(dest, root[len(source)+1:])
+                if not filename.endswith('.pyc'):
+                    dest_path = os.path.join(dest, root[len(source) + 1:])
                     if not os.path.exists(dest_path):
                         os.makedirs(dest_path)
-                    self._print_verbose(u'... %s', filename)
-                    copy(os.path.join(root, filename),
-                        os.path.join(dest_path, filename))
+                    self._print_verbose('... %s', filename)
+                    copy(os.path.join(root, filename), os.path.join(dest_path, filename))
 
     def copy_windows_files(self):
         """
         Copy all the Windows-specific files.
         """
-        self._print(u'Copying extra files for Windows...')
-        self._print_verbose(u'... OpenLP.ico')
-        copy(os.path.join(self.script_path, u'OpenLP.ico'),
-            os.path.join(self.dist_path, u'OpenLP.ico'))
-        self._print_verbose(u'... LICENSE.txt')
-        copy(os.path.join(self.script_path, u'LICENSE.txt'),
-            os.path.join(self.dist_path, u'LICENSE.txt'))
-        self._print_verbose(u'... psvince.dll')
-        copy(self.psvince, os.path.join(self.dist_path, u'psvince.dll'))
-        if os.path.isfile(os.path.join(self.helpfile_path, u'OpenLP.chm')):
-            self._print_verbose(u'... OpenLP.chm')
-            copy(os.path.join(self.helpfile_path, u'OpenLP.chm'),
-                os.path.join(self.dist_path, u'OpenLP.chm'))
+        self._print('Copying extra files for Windows...')
+        self._print_verbose('... OpenLP.ico')
+        copy(os.path.join(self.script_path, 'OpenLP.ico'), os.path.join(self.dist_path, 'OpenLP.ico'))
+        self._print_verbose('... LICENSE.txt')
+        copy(os.path.join(self.script_path, 'LICENSE.txt'), os.path.join(self.dist_path, 'LICENSE.txt'))
+        self._print_verbose('... psvince.dll')
+        copy(self.psvince, os.path.join(self.dist_path, 'psvince.dll'))
+        if os.path.isfile(os.path.join(self.helpfile_path, 'OpenLP.chm')):
+            self._print_verbose('... OpenLP.chm')
+            copy(os.path.join(self.helpfile_path, 'OpenLP.chm'), os.path.join(self.dist_path, 'OpenLP.chm'))
         else:
-            self._print(u'... WARNING: Windows help file not found')
+            self._print('... WARNING: Windows help file not found')
 
     def update_translations(self):
         """
         Update the translations.
         """
-        self._print(u'Updating translations...')
+        self._print('Updating translations...')
         if not self.config.has_section('transifex'):
-            raise Exception(u'No section named "transifex" found.')
+            raise Exception('No section named "transifex" found.')
         if not self.config.has_option('transifex', 'username'):
-            raise Exception(u'No option named "username" found.')
+            raise Exception('No option named "username" found.')
         if not self.config.has_option('transifex', 'password'):
-            raise Exception(u'No option named "password" found.')
-        username = self.config.get(u'transifex', u'username')
-        password = self.config.get(u'transifex', u'password')
+            raise Exception('No option named "password" found.')
+        username = self.config.get('transifex', 'username')
+        password = self.config.get('transifex', 'password')
         os.chdir(os.path.split(self.i18n_utils)[0])
-        translation_utils = Popen([self.python, self.i18n_utils, u'-qdpu',
-            u'-U', username, u'-P', password])
+        translation_utils = Popen([self.python, self.i18n_utils, '-qdpu', '-U', username, '-P', password])
         code = translation_utils.wait()
         if code != 0:
-            raise Exception(u'Error running translation_utils.py')
+            raise Exception('Error running translation_utils.py')
 
     def compile_translations(self):
         """
         Compile the translations for Qt.
         """
-        self._print(u'Compiling translations...')
+        self._print('Compiling translations...')
         files = os.listdir(self.i18n_path)
-        if not os.path.exists(os.path.join(self.dist_path, u'i18n')):
-            os.makedirs(os.path.join(self.dist_path, u'i18n'))
+        if not os.path.exists(os.path.join(self.dist_path, 'i18n')):
+            os.makedirs(os.path.join(self.dist_path, 'i18n'))
         for file in files:
-            if file.endswith(u'.ts'):
-                self._print_verbose(u'... %s', file)
+            if file.endswith('.ts'):
+                self._print_verbose('... %s', file)
                 source_path = os.path.join(self.i18n_path, file)
-                dest_path = os.path.join(self.dist_path, u'i18n',
-                    file.replace(u'.ts', u'.qm'))
-                lconvert = Popen((self.lrelease, u'-compress', u'-silent',
-                    source_path, u'-qm', dest_path))
+                dest_path = os.path.join(self.dist_path, 'i18n', file.replace('.ts', '.qm'))
+                lconvert = Popen((self.lrelease, '-compress', '-silent', source_path, '-qm', dest_path))
                 code = lconvert.wait()
                 if code != 0:
-                    raise Exception(u'Error running lconvert on %s' % \
-                        source_path)
-        self._print(u'Copying qm files...')
-        source = os.path.join(self.site_packages, u'PyQt4', u'translations')
+                    raise Exception('Error running lconvert on %s' % source_path)
+        self._print('Copying qm files...')
+        source = os.path.join(self.site_packages, 'PyQt4', 'translations')
         files = os.listdir(source)
         for filename in files:
-            if filename.startswith(u'qt_') and filename.endswith(u'.qm') and \
-                len(filename) == 8:
-                self._print_verbose(u'... %s', filename)
-                copy(os.path.join(source, filename),
-                    os.path.join(self.dist_path, u'i18n', filename))
+            if filename.startswith('qt_') and filename.endswith('.qm') and len(filename) == 8:
+                self._print_verbose('... %s', filename)
+                copy(os.path.join(source, filename), os.path.join(self.dist_path, 'i18n', filename))
 
     def run_sphinx(self):
         """
         Run Sphinx to build an HTML Help project.
         """
-        self._print(u'Deleting previous help manual build... %s',
-            self.manual_build_path)
+        self._print('Deleting previous help manual build... %s', self.manual_build_path)
         if os.path.exists(self.manual_build_path):
             rmtree(self.manual_build_path)
-        self._print(u'Running Sphinx...')
+        self._print('Running Sphinx...')
         os.chdir(self.manual_path)
-        sphinx = Popen((self.sphinx, u'-b', u'htmlhelp', u'-d',
-            u'build/doctrees', u'source', u'build/htmlhelp'), stdout=PIPE)
+        sphinx = Popen((self.sphinx, '-b', 'htmlhelp', '-d', 'build/doctrees', 'source', 'build/htmlhelp'), stdout=PIPE)
         output, error = sphinx.communicate()
         code = sphinx.wait()
         if code != 0:
             self._print(output)
-            raise Exception(u'Error running Sphinx')
+            raise Exception('Error running Sphinx')
 
     def run_htmlhelp(self):
         """
         Run HTML Help Workshop to convert the Sphinx output into a manual.
         """
-        self._print(u'Running HTML Help Workshop...')
-        os.chdir(os.path.join(self.manual_build_path, u'htmlhelp'))
-        hhc = Popen((self.hhc, u'OpenLP.chm'), stdout=PIPE)
+        self._print('Running HTML Help Workshop...')
+        os.chdir(os.path.join(self.manual_build_path, 'htmlhelp'))
+        hhc = Popen((self.hhc, 'OpenLP.chm'), stdout=PIPE)
         output, error = hhc.communicate()
         code = hhc.wait()
         if code != 1:
-            self._print(u'Exit code:', code)
+            self._print('Exit code:', code)
             self._print(output)
-            raise Exception(u'Error running HTML Help Workshop')
+            raise Exception('Error running HTML Help Workshop')
 
     def create_innosetup_file(self):
         """
         Create an InnoSetup file pointing to the branch being built.
         """
-        self._print(u'Creating Inno Setup file...')
-        input = open(os.path.join(self.script_path,
-            u'OpenLP-2.0.iss.default'), u'r').read()
-        output = input.replace(u'%(branch)s', self.branch_path)
-        output = output.replace(u'%(display_version)s', self.version)
-        outfile = open(os.path.join(self.script_path,
-            u'OpenLP-2.0.iss'), u'w')
+        self._print('Creating Inno Setup file...')
+        input = open(os.path.join(self.script_path, 'OpenLP-2.0.iss.default'), 'r').read()
+        output = input.replace('%(branch)s', self.branch_path)
+        output = output.replace('%(display_version)s', self.version)
+        outfile = open(os.path.join(self.script_path, 'OpenLP-2.0.iss'), 'w')
         outfile.write(output)
         outfile.close()
 
@@ -522,13 +499,11 @@
         Checks the PortableApp directory structure amd creates
         missing subdirs
         """
-        self._print(u'  Checking PortableApps directory structure...')
-        launcher_path = os.path.join(self.portable_path, u'App',
-            u'Appinfo', u'Launcher')
+        self._print('  Checking PortableApps directory structure...')
+        launcher_path = os.path.join(self.portable_path, 'App', 'Appinfo', 'Launcher')
         if not os.path.exists(launcher_path):
             os.makedirs(launcher_path)
-        settings_path = os.path.join(self.portable_path, u'Data',
-            u'Settings')
+        settings_path = os.path.join(self.portable_path, 'Data', 'Settings')
         if not os.path.exists(settings_path):
             os.makedirs(settings_path)
 
@@ -536,14 +511,12 @@
         """
         Create a Portabbleapps appinfo.ini file.
         """
-        self._print(u'  Creating PortableApps appinfo file ...')
-        portable_version = self.version + '.0' * (3-self.version.count('.'))
-        input = open(os.path.join(self.script_path,
-            u'appinfo.ini.default'), u'r').read()
-        output = input.replace(u'%(display_version)s', self.version)
-        output = output.replace(u'%(package_version)s', portable_version)
-        outfile = open(os.path.join(self.portable_path, u'App',
-            u'Appinfo', u'appinfo.ini'), u'w')
+        self._print('  Creating PortableApps appinfo file ...')
+        portable_version = self.version + '.0' * (3 - self.version.count('.'))
+        input = open(os.path.join(self.script_path, 'appinfo.ini.default'), 'r').read()
+        output = input.replace('%(display_version)s', self.version)
+        output = output.replace('%(package_version)s', portable_version)
+        outfile = open(os.path.join(self.portable_path, 'App', 'Appinfo', 'appinfo.ini'), 'w')
         outfile.write(output)
         outfile.close()
 
@@ -551,13 +524,12 @@
         """
         Run InnoSetup to create an installer.
         """
-        self._print(u'Running Inno Setup...')
+        self._print('Running Inno Setup...')
         os.chdir(self.script_path)
-        innosetup = Popen((self.innosetup,
-            os.path.join(self.script_path, u'OpenLP-2.0.iss'), u'/q'))
+        innosetup = Popen((self.innosetup, os.path.join(self.script_path, 'OpenLP-2.0.iss'), '/q'))
         code = innosetup.wait()
         if code != 0:
-            raise Exception(u'Error running Inno Setup')
+            raise Exception('Error running Inno Setup')
 
     def run_portableapp_builder(self):
         """
@@ -566,105 +538,92 @@
         2  Builds the PortableApps Launcher
         3  Builds the PortableApps Install
         """
-        self._print(u'Running PortableApps Builder...')
-        self._print(u'  Clearing old files')
+        self._print('Running PortableApps Builder...')
+        self._print('  Clearing old files')
         # Remove previous contents of portableapp build directory.
         if os.path.exists(self.portable_path):
             rmtree(self.portable_path)
-        self._print(u'  Creating PortableApps build directory')
+        self._print('  Creating PortableApps build directory')
         # Copy the contents of the OpenLPPortable directory to the portable
         # build directory.
-        dir_util.copy_tree(os.path.join(self.script_path, u'OpenLPPortable'),
-            self.portable_path)
+        dir_util.copy_tree(os.path.join(self.script_path, 'OpenLPPortable'), self.portable_path)
         self.check_portableapp_directory()
         self.create_portableapps_appinfo_file()
         # Copy distribution files to portableapp build directory.
-        self._print(u'  Copying distribution files')
-        portable_app_path = os.path.join(self.portable_path, u'App', u'OpenLP')
+        self._print('  Copying distribution files')
+        portable_app_path = os.path.join(self.portable_path, 'App', 'OpenLP')
         dir_util.copy_tree(self.dist_path, portable_app_path)
         # Copy help files to portableapp build directory.
-        self._print(u'  Copying help files')
-        dir_util.copy_tree(self.helpfile_path,
-            os.path.join(portable_app_path, u'help'))
+        self._print('  Copying help files')
+        dir_util.copy_tree(self.helpfile_path, os.path.join(portable_app_path, 'help'))
         # Build the launcher.
-        self._print(u'  Building PortableApps Launcher')
-        portableapps = Popen((self.portablelauncher, self.portable_path),
-            stdout=PIPE)
+        self._print('  Building PortableApps Launcher')
+        portableapps = Popen((self.portablelauncher, self.portable_path), stdout=PIPE)
         code = portableapps.wait()
         if code != 0:
-            raise Exception(u'Error creating PortableAppa Launcher')
+            raise Exception('Error creating PortableAppa Launcher')
         # Build the portable installer.
-        self._print(u'  Building PortableApps Installer')
-        portableapps = Popen((self.portableinstaller, self.portable_path),
-            stdout=PIPE)
+        self._print('  Building PortableApps Installer')
+        portableapps = Popen((self.portableinstaller, self.portable_path), stdout=PIPE)
         code = portableapps.wait()
         if code != 0:
-            raise Exception(u'Error running PortableApps Installer')
-        portable_app = os.path.abspath(os.path.join(self.portable_path, u'..',
-            u'OpenLPPortable_%s.paf.exe' % self.version))
+            raise Exception('Error running PortableApps Installer')
+        portable_app = os.path.abspath(os.path.join(self.portable_path, '..',
+                                                    'OpenLPPortable_%s.paf.exe' % self.version))
         if os.path.exists(portable_app):
-            move(portable_app, os.path.abspath(
-                os.path.join(self.dist_path, u'..')))
-            self._print(u'  PortableApp build complete')
+            move(portable_app, os.path.abspath(os.path.join(self.dist_path, '..')))
+            self._print('  PortableApp build complete')
         else:
-            raise Exception(u'PortableApp failed to build')
+            raise Exception('PortableApp failed to build')
 
     def build_pptviewlib(self):
         """
         Build the PowerPoint Viewer DLL using Visual Studio.
         """
-        self._print(u'Building PPTVIEWLIB.DLL...')
-        vcbuild = Popen((self.vcbuild, u'/rebuild',
-            os.path.join(self.pptviewlib_path, u'pptviewlib.vcproj'),
-            u'Release|Win32'))
+        self._print('Building PPTVIEWLIB.DLL...')
+        vcbuild = Popen((self.vcbuild, '/rebuild', os.path.join(self.pptviewlib_path, 'pptviewlib.vcproj'),
+                         'Release|Win32'))
         code = vcbuild.wait()
         if code != 0:
-            raise Exception(u'Error building pptviewlib.dll')
-        copy(os.path.join(self.pptviewlib_path, u'Release',
-            u'pptviewlib.dll'), self.pptviewlib_path)
+            raise Exception('Error building pptviewlib.dll')
+        copy(os.path.join(self.pptviewlib_path, 'Release', 'pptviewlib.dll'), self.pptviewlib_path)
 
     def main(self):
         """
         The main function to run the Windows builder.
         """
-        self._print_verbose(u'OpenLP main script: ......%s',
-            self.openlp_script)
-        self._print_verbose(u'Script path: .............%s',
-            os.path.split(os.path.abspath(__file__))[0])
-        self._print_verbose(u'Branch path: .............%s', self.branch_path)
-        self._print_verbose(u'Source path: .............%s', self.source_path)
-        self._print_verbose(u'Dist path: ...............%s', self.dist_path)
-        self._print_verbose(u'Portable path: ...........%s',
-            self.portable_path)
-        self._print_verbose(u'PyInstaller: .............%s', self.pyinstaller)
-        self._print_verbose(u'Documentation branch path:%s', self.docs_path)
-        self._print_verbose(u'Help file build path: ....%s',
-            self.helpfile_path)
-        self._print_verbose(u'Inno Setup path: .........%s', self.innosetup)
-        self._print_verbose(u'PortableApp Launcher......%s',
-            self.portablelauncher)
-        self._print_verbose(u'PortableApp Installer.....%s',
-            self.portableinstaller)
-        self._print_verbose(u'Windows resources: .......%s', self.winres_path)
-        self._print_verbose(u'VCBuild path: ............%s', self.vcbuild)
-        self._print_verbose(u'PPTVIEWLIB path: .........%s',
-            self.pptviewlib_path)
-        self._print_verbose(u'')
+        self._print_verbose('OpenLP main script: ......%s', self.openlp_script)
+        self._print_verbose('Script path: .............%s', os.path.split(os.path.abspath(__file__))[0])
+        self._print_verbose('Branch path: .............%s', self.branch_path)
+        self._print_verbose('Source path: .............%s', self.source_path)
+        self._print_verbose('Dist path: ...............%s', self.dist_path)
+        self._print_verbose('Portable path: ...........%s', self.portable_path)
+        self._print_verbose('PyInstaller: .............%s', self.pyinstaller)
+        self._print_verbose('Documentation branch path:%s', self.docs_path)
+        self._print_verbose('Help file build path: ....%s', self.helpfile_path)
+        self._print_verbose('Inno Setup path: .........%s', self.innosetup)
+        self._print_verbose('PortableApp Launcher......%s', self.portablelauncher)
+        self._print_verbose('PortableApp Installer.....%s', self.portableinstaller)
+        self._print_verbose('Windows resources: .......%s', self.winres_path)
+        self._print_verbose('VCBuild path: ............%s', self.vcbuild)
+        self._print_verbose('PPTVIEWLIB path: .........%s', self.pptviewlib_path)
+        self._print_verbose('')
         if not self.args.skip_update:
             self.update_code()
         self.build_pptviewlib()
         self.run_pyinstaller()
         self.write_version_file()
+        self.copy_default_theme()
         self.copy_plugins()
         self.copy_media_player()
         if os.path.exists(self.manual_path):
             self.run_sphinx()
             self.run_htmlhelp()
         else:
-            self._print(u'')
-            self._print(u'WARNING: Documentation trunk not found. Windows')
-            self._print(u'         Help file will not be included in build')
-            self._print(u'')
+            self._print('')
+            self._print('WARNING: Documentation trunk not found. Windows')
+            self._print('         Help file will not be included in build')
+            self._print('')
         self.copy_windows_files()
         if not self.args.skip_translations:
             self.update_translations()
@@ -673,7 +632,8 @@
         self.run_innosetup()
         if self.args.portable:
             self.run_portableapp_builder()
-        self._print(u'Done.')
-
-if __name__ == u'__main__':
+        self._print('Done.')
+
+
+if __name__ == '__main__':
     WindowsBuilder().main()