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

Delta Between Two Patch Sets: packagerChrome.py

Issue 29333368: Issue 3498 - Improve file mapping / skipping logic (Closed)
Left Patch Set: Pass the skip parameter for both Gecko Files.read calls Created Jan. 12, 2016, 1:24 p.m.
Right Patch Set: Fixed typo in Gecko packager Created Jan. 13, 2016, 11:10 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « packager.py ('k') | packagerGecko.py » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 # coding: utf-8 1 # coding: utf-8
2 2
3 # This Source Code Form is subject to the terms of the Mozilla Public 3 # This Source Code Form is subject to the terms of the Mozilla Public
4 # License, v. 2.0. If a copy of the MPL was not distributed with this 4 # License, v. 2.0. If a copy of the MPL was not distributed with this
5 # file, You can obtain one at http://mozilla.org/MPL/2.0/. 5 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
6 6
7 import sys 7 import sys
8 import os 8 import os
9 import re 9 import re
10 import json 10 import json
(...skipping 308 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 params = { 319 params = {
320 'type': type, 320 'type': type,
321 'baseDir': baseDir, 321 'baseDir': baseDir,
322 'releaseBuild': releaseBuild, 322 'releaseBuild': releaseBuild,
323 'version': version, 323 'version': version,
324 'experimentalAPI': experimentalAPI, 324 'experimentalAPI': experimentalAPI,
325 'devenv': devenv, 325 'devenv': devenv,
326 'metadata': metadata, 326 'metadata': metadata,
327 } 327 }
328 328
329 skip = metadata.options('mapping') if metadata.has_section('mapping') else [] 329 mapped = metadata.items('mapping') if metadata.has_section('mapping') else []
330 files = Files(getPackageFiles(params), getIgnoredFiles(params), 330 files = Files(getPackageFiles(params), getIgnoredFiles(params),
331 process=lambda path, data: processFile(path, data, params)) 331 process=lambda path, data: processFile(path, data, params))
332 332
333 if metadata.has_section('mapping'): 333 files.readMappedFiles(mapped)
334 files.readMappedFiles(metadata.items('mapping')) 334 files.read(baseDir, skip=[opt for opt, _ in mapped])
335 files.read(baseDir, skip=skip)
336 335
337 if metadata.has_section('convert_js'): 336 if metadata.has_section('convert_js'):
338 convertJS(params, files) 337 convertJS(params, files)
339 338
340 if metadata.has_section('convert_img'): 339 if metadata.has_section('convert_img'):
341 from imageConversion import convertImages 340 from imageConversion import convertImages
342 convertImages(params, files) 341 convertImages(params, files)
343 342
344 if metadata.has_section('preprocess'): 343 if metadata.has_section('preprocess'):
345 files.preprocess( 344 files.preprocess(
(...skipping 23 matching lines...) Expand all
369 files['qunit/index.html'] = createScriptPage(params, 'testIndex.html.tmpl', 368 files['qunit/index.html'] = createScriptPage(params, 'testIndex.html.tmpl',
370 ('general', 'testScripts')) 369 ('general', 'testScripts'))
371 370
372 zipdata = files.zipToString() 371 zipdata = files.zipToString()
373 signature = None 372 signature = None
374 pubkey = None 373 pubkey = None
375 if keyFile != None: 374 if keyFile != None:
376 signature = signBinary(zipdata, keyFile) 375 signature = signBinary(zipdata, keyFile)
377 pubkey = getPublicKey(keyFile) 376 pubkey = getPublicKey(keyFile)
378 writePackage(outFile, pubkey, signature, zipdata) 377 writePackage(outFile, pubkey, signature, zipdata)
LEFTRIGHT

Powered by Google App Engine
This is Rietveld