diff --git a/osx/macosx-builder.py b/osx/macosx-builder.py index bb1a895..7ebc22d 100644 --- a/osx/macosx-builder.py +++ b/osx/macosx-builder.py @@ -118,7 +118,7 @@ import subprocess 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 @@ def _which(command): """ 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 @@ class MacosxBuilder(object): 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 @@ class MacosxBuilder(object): """ if len(args) > 0: text = text % tuple(args) - print text + print(text) def _print_verbose(self, text, *args): """ @@ -173,7 +174,7 @@ class MacosxBuilder(object): 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 @@ class MacosxBuilder(object): 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 @@ class MacosxBuilder(object): """ 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 @@ class MacosxBuilder(object): 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 @@ class MacosxBuilder(object): """ 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 @@ class MacosxBuilder(object): 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) + 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(u'Error reverting the code') - self._print(u'Updating the code...') - bzr = Popen((u'bzr', u'update'), stdout=PIPE) + 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(u'Error updating the code') + 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'), 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 @@ class MacosxBuilder(object): """ 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 @@ class MacosxBuilder(object): # 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 @@ class MacosxBuilder(object): 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 @@ class MacosxBuilder(object): 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 @@ class MacosxBuilder(object): """ 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 @@ class MacosxBuilder(object): 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() diff --git a/pyinstaller-hooks/rthook_openlp_pyqt4.py b/pyinstaller-hooks/rthook_openlp_pyqt4.py index 44b9956..d49b3fa 100644 --- a/pyinstaller-hooks/rthook_openlp_pyqt4.py +++ b/pyinstaller-hooks/rthook_openlp_pyqt4.py @@ -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) diff --git a/windows/windows-builder.py b/windows/windows-builder.py index 919466e..3b41562 100644 --- a/windows/windows-builder.py +++ b/windows/windows-builder.py @@ -129,7 +129,7 @@ import sys 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 @@ class WindowsBuilder(object): 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 @@ class WindowsBuilder(object): """ if len(args) > 0: text = text % tuple(args) - print text + print(text) def _print_verbose(self, text, *args): """ @@ -167,26 +168,20 @@ class WindowsBuilder(object): """ 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 @@ class WindowsBuilder(object): 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 @@ class WindowsBuilder(object): 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 @@ class WindowsBuilder(object): 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) + 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(u'Error reverting the code') - self._print(u'Updating the code...') - bzr = Popen((u'bzr', u'update'), stdout=PIPE) + 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(u'Error updating the code') + 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'), 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 @@ class WindowsBuilder(object): 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 @@ class WindowsBuilder(object): """ 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 @@ class WindowsBuilder(object): """ 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 @@ class WindowsBuilder(object): 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 @@ class WindowsBuilder(object): self.run_innosetup() if self.args.portable: self.run_portableapp_builder() - self._print(u'Done.') + self._print('Done.') -if __name__ == u'__main__': + +if __name__ == '__main__': WindowsBuilder().main()