Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: packagerEdge.py

Issue 29357701: Issue 4548 - Split out Safari and add Edge release automation (Closed)
Patch Set: Addressed nits Created Oct. 25, 2016, 10:35 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build.py ('k') | releaseAutomation.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packagerEdge.py
diff --git a/packagerEdge.py b/packagerEdge.py
index c63064e8fbefaaec4482fbcfd2808d89e0a0668e..1fcf6f3fd47ba0900d0429bfffe8d845947be964 100644
--- a/packagerEdge.py
+++ b/packagerEdge.py
@@ -11,6 +11,8 @@ import zipfile
import packager
import packagerChrome
+# These functions are unused here, but releaseAutomation.py expects them.
+from packager import readMetadata, getDefaultFileName
# Files and directories expected inside of the .APPX archive.
MANIFEST = 'AppxManifest.xml'
@@ -93,7 +95,7 @@ def create_appx_manifest(params, files, release_build=False):
for size in ['44', '50', '150']:
path = '{}/logo_{}.png'.format(ASSETS_DIR, size)
if path not in files:
- raise KeyError(path + 'is not found in files')
+ raise KeyError(path + ' is not found in files')
params['logo_' + size] = path.replace('/', '\\')
template = _get_template_for(MANIFEST)
« no previous file with comments | « build.py ('k') | releaseAutomation.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld