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

Delta Between Two Patch Sets: build.py

Issue 29365816: Issue 4677 - Add support for x86 builds in adblockbrowser-build (Closed)
Left Patch Set: Not changing previous exit codes per Anton's suggestion Created Dec. 1, 2016, 1:59 p.m.
Right Patch Set: Adding missing MOZILLA_OFFICIAL config Created Dec. 28, 2016, 6:08 p.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 | « adblockbrowser-cfg.py ('k') | config.py.sample » ('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 #!/usr/bin/env python 1 #!/usr/bin/env python
2 2
3 import glob 3 import glob
4 import json 4 import json
5 import os 5 import os
6 import re 6 import re
7 import shutil 7 import shutil
8 import string 8 import string
9 import subprocess 9 import subprocess
10 import sys 10 import sys
(...skipping 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 print >>sys.stderr, "Invalid architecture, check config.py" 139 print >>sys.stderr, "Invalid architecture, check config.py"
140 sys.exit(6) 140 sys.exit(6)
141 141
142 if do_build: 142 if do_build:
143 apk_path = build(architecture, distribution_mode, build_mode, 143 apk_path = build(architecture, distribution_mode, build_mode,
144 config.ANDROID_SDK_PATH, config.ANDROID_NDK_PATH) 144 config.ANDROID_SDK_PATH, config.ANDROID_NDK_PATH)
145 if do_sign: 145 if do_sign:
146 sign(apk_path, config.ANDROID_KEYSTORE_PATH, config.ANDROID_KEY_NAME) 146 sign(apk_path, config.ANDROID_KEYSTORE_PATH, config.ANDROID_KEY_NAME)
147 else: 147 else:
148 print apk_path 148 print apk_path
LEFTRIGHT

Powered by Google App Engine
This is Rietveld